The 7.x-1.x branch has tests, so let's take advantage of them and make sure the module works as expected.

CommentFileSizeAuthor
#2 exif-n3268425-2.patch417 bytesdamienmckenna

Comments

DamienMcKenna created an issue. See original summary.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
StatusFileSize
new exif-n3268425-2.patch417 bytes

WIP taken from #1336004.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented

The tests start failing with this error:

Exception Warning    exif.module        184 _exif_get_file_uri_and_language()  
    Undefined property: stdClass::$field_image
Exception Warning    exif.module        174 _exif_get_image_fields_metadata()  
    Undefined variable $image_descriptor
Exception Warning    exif.module        174 _exif_get_image_fields_metadata()  
    Trying to access array offset on value of type null

That's repeated a bunch of times, likely for each of the fields added to the content type.

  • DamienMcKenna committed 5b10a6d on 7.x-1.x 
    Issue #3268425 by DamienMcKenna: Add missing field admin permission for...
damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Active » Needs work

I committed the first patch, need to keep working on the rest.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Needs work » Closed (outdated)

D7 is no longer supported.

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.