Support microdata for taxonomy_term [#1282444]

The problem with taxonomy as opposed to other core fields is that it is basically a reference field; however, most vocabularies treat them as literal properties, not as items.

If we were using the Drupal mental model to expose the data, the mapping would be on the taxonomy term entity configuration form (which I don't think exists anyway). If we use the mental model that microdata vocabs generally use, the mapping would be on the field itself.

I think we should have a specialized mapping form for taxonomy fields. It should give all of the taxonomy term's properties on the field (instead of the entity) and give a checkbox for people to make this an item instead if people want to. This is a special case that doesn't make sense when you actually have a proper conceptual model of how the code works, but should be more intuitive for the end users.

Comment	File	Size	Author
#2	1282444-taxonomy-term-2.patch	9.84 KB	linclark

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

Quarantine CreditAttribution: Quarantine commented 20 September 2011 at 08:41

Subscribing.

I'm not a coder so I won't be of much help, but I'll be following the development of this module closely. :) I'll be testing this module on a site that I'm working on and incorporate some hard-coded code via the theme files for my immediate use (for instance, for my Ubercart site, I'm including a itemprop="price" property for the price).

Comment #2

Anonymous (not verified) CreditAttribution: Anonymous commented 20 October 2011 at 20:57

File	Size
1282444-taxonomy-term-2.patch	9.84 KB

Glad to hear you're following, Quarantine :)

This gets the functionality working. Still needs tests, cleanup, and comments before I commit it though.

Comment #3

Quarantine CreditAttribution: Quarantine commented 23 October 2011 at 17:44

Hey lin!

This is my first time patching, and I'm running into some trouble to get it to work. My attempt to patch it is resulting in errors - I'm doing it right in my Ubuntu server.

root@server:/var/www/URL.com/test/sites/all/modules/microdata# patch -b < 1282444-taxonomy-term-2.patch
patching file microdata.module
Hunk #1 FAILED at 93.
Hunk #2 FAILED at 220.
Hunk #3 FAILED at 329.
Hunk #4 FAILED at 361.
Hunk #5 FAILED at 397.
Hunk #6 FAILED at 486.
Hunk #7 FAILED at 540.
Hunk #8 FAILED at 639.
Hunk #9 FAILED at 827.
Hunk #10 FAILED at 853.
10 out of 10 hunks FAILED -- saving rejects to file microdata.module.rej
root@server:/var/www/URL.com/test/sites/all/modules/microdata#

I thought of patching it manually but it seems too huge of a patch to do it manually..

Comment #4

Anonymous (not verified) CreditAttribution: Anonymous commented 23 October 2011 at 21:20

This probably means you need to do a git pull to get an up to date copy of the code before trying to apply the patch. However, I plan to commit it very soon, so you can also just wait until then if you like.