pimcore / pimcore

Core Framework for the Open Source Data & Experience Management Platform (PIM, MDM, CDP, DAM, DXP/CMS & Digital Commerce)
http://www.pimcore.com
Other
3.35k stars 1.43k forks source link

[Bug]: Upload of new Excel File Version leads to wrong assets customSettings and big logs #15221

Open ascheider opened 1 year ago

ascheider commented 1 year ago

Pimcore version

10.5.19

Steps to reproduce

Upload excel file to the assets and open up the table assets in the database cs Custom settings are correct and empty at this point. Upload new excel version and refresh assets table cs not empty Custom settings are not empty anymore. Execute core messenger: bin/console messenger:consume pimcore_core --time-limit=600 Messenger will also add document_page_count to the customSettings which only exists in pdf. cs page count At this point messenger will fail to proccess the messages and write the prod.log whole time, until no disc space is available.

Stack trace:

[2023-05-17T08:29:33.467839+02:00] pimcore.ERROR: Couldn't load document /ImportFiles/230508_deepl-Pimcore-test-file.xlsx only PDF documents are currently supported [] [] [2023-05-17T08:29:33.468597+02:00] pimcore.ERROR: Couldn't create image-thumbnail of document /ImportFiles/230508_deepl-Pimcore-test-file.xlsx [] [] [2023-05-17T08:29:33.468814+02:00] pimcore.ERROR: Couldn't load document /ImportFiles/230508_deepl-Pimcore-test-file.xlsx only PDF documents are currently supported [] [] [2023-05-17T08:29:33.473918+02:00] pimcore.ERROR: Couldn't load document /ImportFiles/230508_deepl-Pimcore-test-file.xlsx only PDF documents are currently supported [] [] [2023-05-17T08:29:33.474428+02:00] pimcore.ERROR: Exception: Couldn't load document /ImportFiles/230508_deepl-Pimcore-test-file.xlsx only PDF documents are currently supported in /pimcore/vendor/pimcore/pimcore/lib/Document/Adapter/Ghostscript.php:105 Stack trace: #0 /pimcore/vendor/pimcore/pimcore/models/Asset/Document.php(63): Pimcore\Document\Adapter\Ghostscript->load() #1 /pimcore/vendor/pimcore/pimcore/lib/Messenger/Handler/AssetUpdateTasksHandler.php(63): Pimcore\Model\Asset\Document->processPageCount() #2 /pimcore/vendor/pimcore/pimcore/lib/Messenger/Handler/AssetUpdateTasksHandler.php(45): Pimcore\Messenger\Handler\AssetUpdateTasksHandler->processDocument() #3 /pimcore/vendor/symfony/messenger/Middleware/HandleMessageMiddleware.php(95): Pimcore\Messenger\Handler\AssetUpdateTasksHandler->__invoke() #4 /pimcore/vendor/symfony/messenger/Middleware/SendMessageMiddleware.php(73): Symfony\Component\Messenger\Middleware\HandleMessageMiddleware->handle() #5 /pimcore/vendor/symfony/messenger/Middleware/FailedMessageProcessingMiddleware.php(34): Symfony\Component\Messenger\Middleware\SendMessageMiddleware->handle() #6 /pimcore/vendor/symfony/messenger/Middleware/DispatchAfterCurrentBusMiddleware.php(68): Symfony\Component\Messenger\Middleware\FailedMessageProcessingMiddleware->handle() #7 /pimcore/vendor/symfony/messenger/Middleware/RejectRedeliveredMessageMiddleware.php(48): Symfony\Component\Messenger\Middleware\DispatchAfterCurrentBusMiddleware->handle() #8 /pimcore/vendor/symfony/messenger/Middleware/AddBusNameStampMiddleware.php(37): Symfony\Component\Messenger\Middleware\RejectRedeliveredMessageMiddleware->handle() #9 /pimcore/vendor/symfony/messenger/MessageBus.php(77): Symfony\Component\Messenger\Middleware\AddBusNameStampMiddleware->handle() #10 /pimcore/vendor/symfony/messenger/RoutableMessageBus.php(54): Symfony\Component\Messenger\MessageBus->dispatch() #11 /pimcore/vendor/symfony/messenger/Worker.php(158): Symfony\Component\Messenger\RoutableMessageBus->dispatch() #12 /pimcore/vendor/symfony/messenger/Worker.php(107): Symfony\Component\Messenger\Worker->handleMessage() #13 /pimcore/vendor/symfony/messenger/Command/ConsumeMessagesCommand.php(225): Symfony\Component\Messenger\Worker->run() #14 /pimcore/vendor/symfony/console/Command/Command.php(298): Symfony\Component\Messenger\Command\ConsumeMessagesCommand->execute() #15 /pimcore/vendor/symfony/console/Application.php(1058): Symfony\Component\Console\Command\Command->run() #16 /pimcore/vendor/symfony/framework-bundle/Console/Application.php(96): Symfony\Component\Console\Application->doRunCommand() #17 /pimcore/vendor/symfony/console/Application.php(301): Symfony\Bundle\FrameworkBundle\Console\Application->doRunCommand() #18 /pimcore/vendor/symfony/framework-bundle/Console/Application.php(82): Symfony\Component\Console\Application->doRun() #19 /pimcore/vendor/symfony/console/Application.php(171): Symfony\Bundle\FrameworkBundle\Console\Application->doRun() #20 /pimcore/bin/console(49): Symfony\Component\Console\Application->run() #21 {main} [] []

Expected / actual behavior

If i upload the new excel version the customSettings must stay empty like before

github-actions[bot] commented 1 year ago

Thanks a lot for reporting the issue. We did not consider the issue as "Priority" or "Backlog", so we're not going to work on that anytime soon. Please create a pull request to fix the issue if this is a bug report. We'll then review it as quickly as possible. If you're interested in contributing a feature, please contact us first here before creating a pull request. We'll then decide whether we'd accept it or not. Thanks for your understanding.