Closed (fixed)
Project:
FileField
Version:
6.x-1.0-beta1
Component:
Code
Priority:
Critical
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
28 Jun 2008 at 14:11 UTC
Updated:
13 Jul 2008 at 22:14 UTC
D6 (6.2) can't use the filefield_meta module.
The filefield_meta.info dependencies should be
dependencies[] = content
dependencies[] = filefield
dependencies[] = getid3
not
dependencies = content filefield getid3
Comments
Comment #1
seaneffel commentedYes, I just spotted this in 6.2 on the modules page:
Filefield-Meta, 6.x-1.0-beta1, "This version is incompatible with the 6.2 version of Drupal core."
Made the changes in the .info file like above and its all fixed.
Comment #2
jpetso commentedThere's a reason why it's not enabled yet, which is because it doesn't work on the current filefield infrastructure which changed quite a bit from Drupal 5 to 6. Porting should be relatively easy, but there's more important stuff to do for me at the moment.
Comment #3
jpetso commentedOk, I "fixed" the dependencies (to be released with beta2 release, for your entertainment) but didn't port filefield_meta to a working state so far. In its current state, it should pretty much be a no-op, not doing anything because its hook is not invoked.
I'm also not quite sure what to do with it, as the widget integration won't work in the current form - probably better to have a separate module for video files providing a preview widget and custom layout for video needs, instead of that crappy "width: x, height: y, duration: z" text addendum to the generic widget. Such a widget can be done much more easily on the D6 filefield, but realistically seen I can't provide video support next to image support in the timeframe that I've got left for filefield (which is like, just a few hours or so).
I guess the way forward would be to make filefield_meta a pure demonstration obejct - throw out the form altering code, port the metadata storage to hook_file(), and maybe implement a simple widget and formatter (with lots of API documentation) that shows how to extend filefield like filefield_image does too. Not on my list of priorities though, I'll leave that as an exercise for the interested reader :P
Comment #4
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.