I finally took the plunge and upgraded my video module to 6.x.4 which also meant I had to use emfield/emvideo for handling of youtube videos.
Of course this broke integration of acidfree with the video module.
Too get this working, I did the following:
1. removed the dependency on video_image in acidfree_video.info so that module doesn't exist in video-6.x-4
2. removed a call to a function named dm() in acidfree_video.module. This was occurring whenever acidfree tries to create a fake thumbnail for a video
3. fix up theme_acidfree_print_thumb_video. I ended up overriding this in my theme to avoid a bug update to the acidfree code but the function is identical. Also my code isn't super generic as it relies on the cck field names I gave for the video_upload and the embedded video.
If you want to see the changes in action, visit the following URLs:
1. http://womensfooty.com/training/afl-coaching-style-play - album containing just video_upload nodes
2. http://womensfooty.com/albums/afl-skills-videos - album containing a mixture of emvideo and video_upload nodes.
| Comment | File | Size | Author |
|---|---|---|---|
| theme_acidfree_print_thumb_video.php_.txt | 1.74 KB | bkat | |
| acidfree_video.module.patch | 188 bytes | bkat |
Comments
Comment #1
mwheinz commentedComment #2
mwheinz commentedDowngrading this because it's an enhancement rather than broken functionality. Thanks for the patch - I'm working my way through all the outstanding issues right now and trying to get a handle on all the outstanding issues.
Comment #3
mwheinz commentedPatch accepted (quite some time ago, actually).
Comment #4
bkat commentedDid you accept the patch as is? If so, you should document the dependencies on the cck field names for video and embedded video.