I don't even have any images yet. getting the following in my error_log (typed, so there could be a typo, sorry)

[client 10.0.0.89] PHP Parse error:  parse error, unexpected ']', expecting '(' in /srv/www/html/sites/all/modules/exif/exif.module on line 48, referer: http://10.0.0.81/?q=admin/build/modules

This is a test site. I have the module working on some other site...

Causes blank pages on my site when enabled. Had to delete the module directory (or move) to recover.

Comments

David Lesieur’s picture

David Lesieur commented

I can't reproduce this bug. Also, if it works on other sites of yours, you might want to look into what's different in their setup. Did you have PEL installed?

hyperlogos’s picture

hyperlogos commented

Yes, I do have PEL installed.

Does either exif or PEL require PHP 5? I think my other site is PHP5.

David Lesieur’s picture

David Lesieur commented

Yes, PEL requires PHP 5.

hyperlogos’s picture

hyperlogos commented

Well, that explains it. Perhaps you could put a note about your module requiring php5?

David Lesieur’s picture

David Lesieur commented
Status: Active » Fixed

There already is a note, both on the project's page and in the README file... :-)

David Lesieur’s picture

David Lesieur commented
Category: bug » support
hyperlogos’s picture

hyperlogos commented

Oh there it is :/

Maybe for people like me you could make it say it requires PHP5 when you try to enable it :D

David Lesieur’s picture

David Lesieur commented
Title: error when enabling exif » Show an error message when trying to install on PHP < 5.0
Category: support » feature
Priority: Critical » Normal
Status: Fixed » Active

Good idea! I'm turning this into a feature request, it will remind me (or anyone interested) to implement it.

jphautin’s picture

jphautin commented
Issue summary: View changes
Status: Active » Closed (won't fix)