Asset should provide its own generation handling as we have have "Generate content", "Generate users", "Generate terms", we should have "Generate assets" and then entityreference support for devel generate should do the rest

CommentFileSizeAuthor
#4 asset-devel-generate-1959820-5.patch2.75 KBIRuslan
#4 asset-devel-generate-1959820-4.patch9.61 KBIRuslan
#3 1959820-4-asset_generate.patch13.14 KBSebCorbin
#2 1959820-3-asset_generate.patch13.13 KBSebCorbin
#1 1959820-2-asset_generate.patch11.34 KBSebCorbin
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

SebCorbin’s picture

SebCorbin CreditAttribution: SebCorbin commented
Status: Active » Needs review
FileSize
1959820-2-asset_generate.patch11.34 KB

And here's a patch.

Tested for Video and Image asset types so far.

Video modules other than youtube need to implement hook_emvideo_generate() that return a random video url.

SebCorbin’s picture

SebCorbin CreditAttribution: SebCorbin commented
FileSize
1959820-3-asset_generate.patch13.13 KB

And here the patch with a drush command

SebCorbin’s picture

SebCorbin CreditAttribution: SebCorbin commented
FileSize
1959820-4-asset_generate.patch13.14 KB

Modified the default thumbnail to mqdefault (better ratio and quality), also split the video url before '?' character

IRuslan’s picture

IRuslan CreditAttribution: IRuslan commented
Issue summary: View changes
Status: Needs review » Closed (fixed)
FileSize
asset-devel-generate-1959820-4.patch9.61 KB
asset-devel-generate-1959820-5.patch2.75 KB

Hi Sebastien,

We checked your patch and seems it's good (i only simply append package = "Media assets" to info file, and small fix about random asset author) . For other video modules i created ticket #2318367: Add support for devel generate video types (except youtube)

Attached new patches/committed to dev.