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.

Comments

mwheinz’s picture

Assigned: Unassigned » mwheinz
mwheinz’s picture

Priority: Normal » Minor

Downgrading 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.

mwheinz’s picture

Status: Active » Fixed

Patch accepted (quite some time ago, actually).

bkat’s picture

Did you accept the patch as is? If so, you should document the dependencies on the cck field names for video and embedded video.

Status: Fixed » Closed (fixed)

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