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

rapsli’s picture

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

jphautin’s picture

Issue summary: View changes
Status: Needs review » Closed (won't fix)

Drupal6 EOL