When setting exif to use the external exiftool program everything seems alright until it needs to load the drop-down with available tag names.
In the log if find
PHP message: Error: Class 'Drupal\exif\SimpleExifToolFacade' not found
While The file "modules/exif/src/SimpleExiftoolFacade.php" does exist.
As we need the IPTC Keywords tag I don't think we really need the exiftool backend. If there is a workaround please let me know
| Comment | File | Size | Author |
|---|---|---|---|
| #8 | exif-d8-class-not-found-2941644-8.patch | 175 bytes | hosef |
| #7 | class_not_found-2941644-7.patch | 578 bytes | Matt_TOTW |
| #6 | class_not_found-2941644-6.patch | 573 bytes | tanc |
Comments
Comment #2
tancWe're also seeing this issue but I'm unable to replicate it locally which makes debugging it really hard. I'm only seeing it on a server, running PHP7.0 (I'm also running PHP7.0 locally). In this case there is no dropdown on our node form. Switching to using PHP extension instead of exiftool bypasses the class loading issue.
Comment #3
karel010 commented@tanc
I see I made an error in the last paragraph. I of course meant that we really need the exiftool or a way to have the PHP extension use IPTC.
We recently set up an environment local d8 development so I will see if we have the same problem.
Do you have things installed with composer or without?
Comment #4
tancWith composer. Its an extremely frustrating error as I can't replicate it locally to debug. I've tried all sorts of things, regenerating the autoloader, using longhand syntax to include the classes, use statements. Nothing seems to get the class recognised.
Comment #5
tancThe only progress I've made is by including the class file directly in the getExifInterface() method:
Comment #6
tancHere is a patch. I don't think this is a good solution but we need to move forward so uploading this here for composer patching purposes.
Comment #7
Matt_TOTW commentedThanks @tanc!
We need to include_once, then all good.
Comment #8
hosef commentedThis issue is caused because the file name is not capitalized correctly, and the autoloader cannot find it because it is case sensitive.
Here is a patch that renames the file.
Comment #9
karel010 commented@hosef good catch!
Comment #10
jphautin commented@hosef thanks. integrated in dev branch