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 |
Comments
Comment #1
Quarantine CreditAttribution: Quarantine commentedSubscribing.
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 commentedGlad 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 commentedHey 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.
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 commentedThis 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.Comment #5
Anonymous (not verified) CreditAttribution: Anonymous commentedThis is fixed with commit http://drupalcode.org/project/microdata.git/commit/c94d227
A follow-up issue has been created, #1319822: Output microdata on the term page