Right now, video fields are not visible through entity metadata wrappers because there's not property_type in the hook_field_info() data.

CommentFileSizeAuthor
#1 video-entity_metadata_wrappers_integration-2309595-1.patch329 bytesgarphy
#3 video-entity_metadata_wrappers_integration-2309595-3.patch1.73 KBgarphy
#4 interdiff.txt2.24 KBgarphy
#4 video-entity_metadata_wrappers_integration-2309595-4.patch1.56 KBgarphy

Comments

garphy’s picture

garphy
Primary language French
Location Montreal, Canada
commented
StatusFileSize
new video-entity_metadata_wrappers_integration-2309595-1.patch329 bytes

This first basic patch integrate video fields as "file". This is not ideal as we eventually want to gather all the data (file derivatives, ...).

garphy’s picture

garphy
Primary language French
Location Montreal, Canada
commented
Status: Active » Needs work
garphy’s picture

garphy
Primary language French
Location Montreal, Canada
commented
StatusFileSize
new video-entity_metadata_wrappers_integration-2309595-3.patch1.73 KB

This one renders the original file data in a "file" property (as file & image field do) and renders thumbnail file data & playable files data in specific properties (in "file" format, too).
Still WIP.

garphy’s picture

garphy
Primary language French
Location Montreal, Canada
commented
Status: Needs work » Needs review
StatusFileSize
new video-entity_metadata_wrappers_integration-2309595-4.patch1.56 KB
new interdiff.txt2.24 KB
  • Renamed the property_callback name to match module's name
  • Reworked callbacks declaration (finally understood how it's supposed to work
  • Now entity_metadata_field_default_property_callback is fired and multiple value fields also work

    • Entity/Fields/Wrappers APIs are one the most convoluted ones I've ever seen in Drupal. I'll be glad when we all switch these kind of integration on D8 unified Entity API (</rant>)...

    I think I would benefit of some reviews now.

skwashd’s picture

skwashd commented
Status: Needs review » Reviewed & tested by the community

This is working for me when used with RestWS.

hypertext200’s picture

hypertext200
he/him
Primary language Singhalese
Location New York City
commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.