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