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.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jphautin’s picture

Assigned: Unassigned » jphautin
Category: bug » support

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

Marc-Antoine’s picture

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

Marc-Antoine’s picture

Category: support » bug

In fact I'm pretty sure it's a bug

Cyberschorsch’s picture

This 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)

Cyberschorsch’s picture

Status: Active » Needs review
ALABETAA’s picture

Hello,

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,

TinaRey’s picture

Same 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

MattO77’s picture

Same issue here using Media 2.x. Waiting for fix?

firfin’s picture

Version: 7.x-1.2 » 7.x-1.x-dev
Assigned: jphautin » firfin
Status: Needs review » Needs work

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

firfin’s picture

Assigned: firfin » Unassigned
Status: Needs work » Needs review
FileSize
19.59 KB

Ok, 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.

jphautin’s picture

Assigned: Unassigned » jphautin
jphautin’s picture

Hello,

I was not able to apply any patchs.
So I correct current dev version following Cyberschorsch modifications.
Please review

jphautin’s picture

Status: Needs review » Fixed
firfin’s picture

Patch 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!

A-snowboard’s picture

Hello,

(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

firfin’s picture

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

A-snowboard’s picture

Hello,

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.

Status: Fixed » Closed (fixed)

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