GithubHelp home page GithubHelp logo

Comments (11)

pardahlman avatar pardahlman commented on July 17, 2024 2

Hello again,

I looked into this again, and it turns out that GD was not compiled with the flag --with-jpeg-dir=/usr/include/. After re-compiling with the flag and restarting Apache it works 🎉

However, I'm pretty certain that exif needs to be installed for CE as well. So it should be added to the list of requirements.

from pim-docs.

 avatar commented on July 17, 2024 1

I opened two MR :
akeneo/pim-community-dev#6314
#623

from pim-docs.

mickaelandrieu avatar mickaelandrieu commented on July 17, 2024

Hello @pardahlman and thank you for the feedback, can someone from @akeneo/core-team confirm that exif is / is not a PHP requirement for CE/EE versions? I'll update the docs if needed.

from pim-docs.

nidup avatar nidup commented on July 17, 2024

Looking at the code, it's only used in EE (for the Product Asset Management feature)

from pim-docs.

mickaelandrieu avatar mickaelandrieu commented on July 17, 2024

For Image magick, I've contributed it yesterday: https://github.com/akeneo/pim-docs/blob/1.5/developer_guide/installation/system_requirements/system_requirements.rst.inc#L47. Do we need - in addition - exif extension for EE ?

@pardahlman I'll be glad to merge your contribution if you add the exif extension in the System Requirements section of our documentation 👍

from pim-docs.

nidup avatar nidup commented on July 17, 2024

yep exif for EE :)

from pim-docs.

pardahlman avatar pardahlman commented on July 17, 2024

Thanks for giving this attention - I really appreciate it 👌
Do I understand you correctly that:

  1. exif is only needed for EE
  2. imagemagick is only needed for EE

I'm running 1.7.2 CE (here's the system-info) and the log entries above comes from that instance. To me, the following "PHP exif extension is required to use the ExifMetadataReader" really suggests that exif is needed, wouldn't you say? Especially since installing it mitigated the problem (but exposed a different one)

It would be interesting to hear your thought on the second log entry as well

[2017-05-14 12:16:17] request.CRITICAL: Uncaught PHP Exception Imagine\Exception\RuntimeException: "call_user_func_array() expects parameter 1 to be a valid callback, function 'imagejpeg' not found or inval
id function name" at /akeneo/vendor/imagine/imagine/lib/Imagine/Gd/Image.php line 697 {"exception":"[object] (Imagine\\Exception\\RuntimeException(code: 2): call_user_func_array() expects parameter 1 to be
a valid callback, function 'imagejpeg' not found or invalid function name at /akeneo/vendor/imagine/imagine/lib/Imagine/Gd/Image.php:697, ErrorException(code: 0): call_user_func_array() expects parameter 1
to be a valid callback, function 'imagejpeg' not found or invalid function name at /akeneo/vendor/imagine/imagine/lib/Imagine/Gd/Image.php:580)"} []

If you want to reproduce it yourselves, you can spin up an Akeneo instance using this docker-commpose (here's the file that installs the PHP extensions.

Looking forward hearing from you!

from pim-docs.

pardahlman avatar pardahlman commented on July 17, 2024

Reading the System Requirements again, it says

php5-gd No specific configuration

Don't know if it makes sense to say that it needs to be configured according to above?

from pim-docs.

 avatar commented on July 17, 2024

Hi @mickaelandrieu,
I'll take it 😄
So I reproduct the bug on my local. When I upload an image on a product with exif disable, I dont have the preview. And when exif is actived, the preview is here :)
Also, in demo.akeneo.com, the exif extension is enabled too.
So I will add php-exif in system-requirement. But I guess, I have to add the php-exif in the requirement of the package who need It.
Is it liip/imagine-bundle or imagine/Imagine or other ?

from pim-docs.

mickaelandrieu avatar mickaelandrieu commented on July 17, 2024

Hello @dnd-tyler!
It's a very good question: I don't know :) The best we can do is add it to our own composer.json: if it's Imagine the issue, as Symfony is welcoming this library, we'll probably rely on it later.

Also you may complete the Pim requirements => https://github.com/akeneo/pim-community-dev/blob/master/app/PimRequirements.php#L53

What do you think?

from pim-docs.

damien-carcel avatar damien-carcel commented on July 17, 2024

Issue has been fixed starting 2.0. Thanks for it everyone.

from pim-docs.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.