Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Hello.
I have a problem when I try to add a field to display the metadata info.
When I create the cck field, in the section where I configure what metadata display, the description ask me to select the image, but the field is empty (I have a cck field of type "media" (media contrib module). There is no field to select, so when I try to save the changes, an error is displayed. (See screenshoot).
Any idea what happens this?
Thanks a lot.
Comment | File | Size | Author |
---|---|---|---|
#10 | exif-make-media-files-selectable-1366014-10.patch | 19.59 KB | firfin |
#4 | exif-1366014Cyberschorsch.patch | 1.65 KB | Cyberschorsch |
issue_image_field.png | 78.09 KB | escuriola |
Comments
Comment #1
jphautin CreditAttribution: jphautin commentedDo you have an image field in your content type ?
You must add an image field to the content type before beeing able to add any metadata field.
Comment #2
Marc-Antoine CreditAttribution: Marc-Antoine commentedSame here!
I'm using
Drupal 7.10
Exif 7.x-1.x dev
Media 7.x-2.x
And of course I have an image field before my image metadata text field.
Comment #3
Marc-Antoine CreditAttribution: Marc-Antoine commentedIn fact I'm pretty sure it's a bug
Comment #4
CyberschorschThis is definitly a bug with the media module integration. The problem is, that the module is ignoring the widget type media_generic of the media module.
I just created a quick patch. Seems to be running on my install (Media 1.0 RC2)
Comment #5
CyberschorschComment #6
ALABETAA CreditAttribution: ALABETAA commentedHello,
I have the same problem with Media 7.x-1.0+7-dev. I cannot apply the patch of post #4 because it seems already applied on exif module 7.x-1.2.
I have not installed Media 7.x-2.x because it seems to be unstable.
What must I do ?
Thanks,
Comment #7
TinaRey CreditAttribution: TinaRey commentedSame here, any news for this? I've got Media now to try a few thing but even without media it didn't work. Tried both putting the field onto the node (which has several image fields) and onto the media-image file; neither works, the database stays empty... I can't find any errors anywhere either; it's as if the module wasn't even trying to do anything...
Drupal 7.15
Exif 7.x-1.2 (both 'Refresh on node update' and 'Write empty image data' enabled, so there should at least be some empty line in the db, right?)
(Media 7.x-1.2) > doesn't work without media either
Comment #8
MattO77 CreditAttribution: MattO77 commentedSame issue here using Media 2.x. Waiting for fix?
Comment #9
firfin CreditAttribution: firfin commented@RaspberryBlack : It seems you have -at least partly- some other issue(s). Please don't confuse this one. This issue is about not being able to select images from the 'media' or 'file' type. Core 'Image' from seems to work for most people.
@ALABETAA : Why do you think this patch has already been applied?
@Cyberschorsch : thanks for the patch! It seems very promising. Unfortunately it does not apply for latest dev. It also doesn't quite seem to follow drupal git standards? Have you seen Configuring Netbeans?
I will re-roll the patch for latest dev.
Comment #10
firfin CreditAttribution: firfin commentedOk, patch is attached. This patch applies cleanly on the dev version.
It contains Cyberschorsch' alterations from #4.
After it is applied all image files show up. Not just Core Image, but also the file_entity images using the media selector widget. Haven't seen any problems so far.
Furthermore it contains about 80 minor changes that coder module came up with. Mostly lack of spacing, use of tabs instead of spaces, etc. No logic or coding errors. So basically nothing changed except it for the alterations mentioned above. And the fact almost completely passes coder now. After running this the exif.module is almost 100% drupal coding standards compliant :-)
Please test and review.
Comment #11
jphautin CreditAttribution: jphautin commentedComment #12
jphautin CreditAttribution: jphautin commentedHello,
I was not able to apply any patchs.
So I correct current dev version following Cyberschorsch modifications.
Please review
Comment #13
jphautin CreditAttribution: jphautin commentedComment #14
firfin CreditAttribution: firfin commentedPatch was for dev version before #1956946: Illegal offset type in isset or empty .. field_ui_widget_type_options() landed. That is probably why it did not apply.
But more important, thanks for putting this fix into dev version jphautin! It is in latest dev now right?
And cyberschorsch thanks for getting this rolling!
Newest dev version had this included and works!
Comment #15
A-snowboard CreditAttribution: A-snowboard commentedHello,
(Sorry I'm french and google translate is my friend)
I have the same problem except that I use another module image management: SCALD (http://drupal.org/project/scald)
The patch corrects the problem for media 2.x?
thank you
Comment #16
firfin CreditAttribution: firfin commented@A-snowboard.
So you don't have the same problem (using scald instead of media.) It might be better to open another issue for this. Please do not reopen / restart closed issues.
The prblem here was media integration. And yes, that was fixed.
I don't believe scald integration is in exif module at all.
Comment #17
A-snowboard CreditAttribution: A-snowboard commentedHello,
I'm sorry. I hesitated to open a new question.
I will do it.
For the moment there is no integration with scald. At least with the test that I did.
I opened an issue to talk about.
Thank you.