Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
http://drupal.org/project/video
Problem/Motivation
For HTML elements with a src attribute, such as video, or a data attribute, such as object, the microdata attributes have to be placed within the element itself. This means that microdata module will need to pass the microdata attributes to the templates, which are placed in sites/all/modules/video/theme.
Proposed resolution
- Add the property information that is required by Entity API. Properties should potentially include some metadata. For example, http://schema.org/VideoObject exposes the encodingFormat (i.e. mpeg4)
- Use hook_microdata_value_types_alter(&$types) to change the value type to 'item_option'.
- Figure out how to get microdata attributes from the formatter code to the template and place it on the video tag.
- If metadata is included, add that to the templates in meta tags, conditional on whether microdata exists for them.
- Add a suggested mapping based on Schema.org
Remaining tasks
Everything ;)
User interface changes
Microdata box and suggestions radio will be added to the bottom of the field settings page.
Comment | File | Size | Author |
---|---|---|---|
#1 | 1606580-1-video_microdata.patch | 4.2 KB | colette |
Comments
Comment #1
colette CreditAttribution: colette commentedI wrote the first patch for the video module, publishing microdata for the URL and thumbnail of the video.
Comment #2
colette CreditAttribution: colette commentedI moved the issue to the Video module queue.
Comment #3
colette CreditAttribution: colette commentedI just realized that the issue was assigned to someone else, now it's fixed. I also switched the status to "needs review", waiting for some feedback from the module developers.
Comment #5
Jorrit CreditAttribution: Jorrit commentedI'll take a look at it later, thanks for the patch. Let's see if it still applies to the most recent branch.
Comment #6
Jorrit CreditAttribution: Jorrit commented#1: 1606580-1-video_microdata.patch queued for re-testing.
Comment #8
brycefisherfleig CreditAttribution: brycefisherfleig commentedHi Colette,
Thanks for your contribution to this module! I'm not sure if you're still working on this issue, but it looks like the current patch probably just does not apply to the current head of video-7.x-2.x-dev. Could you try again to submit a patch? Let me know if I should unassign you from this issue instead. Thanks!
Cheers
Bryce
Comment #9
brycefisherfleig CreditAttribution: brycefisherfleig commented