Closed (won't fix)
Project:
Exif
Version:
6.x-1.4
Component:
Code
Priority:
Major
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
18 Apr 2011 at 03:58 UTC
Updated:
28 Feb 2016 at 18:15 UTC
Jump to comment: Most recent
The function _exif_get_image_path incorrectly assumes that if a content type is named 'image' it was provided by the image module.
The test in this function (line 224 of current source) is:
if ($node->type == 'image') {
return $node->images[IMAGE_ORIGINAL];
}
This should be changed to:
if ($node->type == 'image' && is_array($node->images)) {
return $node->images[IMAGE_ORIGINAL];
}
Without this change, the EXIF module will not work for anyone who builds an 'image' content type using CCK/imagefield rather than using the dated image module. Hence the major impact.
Comments
Comment #1
rapsli commentedThanks. This does look good. I don't have the time anymore to maintain the module. If you want to maintain this version of the module... let me know.
Comment #2
jphautin commentedDrupal6 EOL