fix: array thumbnail parameter in getImageThumbnailAction

[marcopixel]

Changes in this pull request

Fix getImageThumbnailAction crash when thumbnail query param is an array.

PR #39 replaced $request->get() with $request->query->get() across controllers to address the Symfony 7.4 deprecation. That was correct in general, but thumbnail is a special case: it accepts both a string (named config) and an array (inline config). InputBag::get() throws BadRequestException for non-scalar values since Symfony 5.1+, so the array case broke.

This can be easily triggered by declaring a dynamic configuration inside twig:

{{ opendxp_image('test_image', {
        'thumbnail': {
            'width': 500,
            'aspectratio': true,
            'interlace': true,
            'quality': 85,
            'format': 'png'
        }
}) }}

Symfony\Component\HttpFoundation\Exception\BadRequestException:
Input value "thumbnail" contains a non-scalar value.

  at vendor/symfony/http-foundation/InputBag.php:46
  at Symfony\Component\HttpFoundation\InputBag->get('thumbnail')
     (vendor/open-dxp/admin-bundle/src/Controller/Admin/Asset/AssetController.php:1171)
  at OpenDxp\Bundle\AdminBundle\Controller\Admin\Asset\AssetController->getImageThumbnailAction(object(Request))
     (vendor/symfony/http-kernel/HttpKernel.php:183)
  at Symfony\Component\HttpKernel\HttpKernel->handleRaw(object(Request), 1)
     (vendor/symfony/http-kernel/HttpKernel.php:76)
  at Symfony\Component\HttpKernel\HttpKernel->handle(object(Request), 1, true)
     (vendor/symfony/http-kernel/Kernel.php:193)
  at Symfony\Component\HttpKernel\Kernel->handle(object(Request))
     (vendor/symfony/runtime/Runner/Symfony/HttpKernelRunner.php:35)
  at Symfony\Component\Runtime\Runner\Symfony\HttpKernelRunner->run()
     (vendor/autoload_runtime.php:32)
  at require_once('/var/www/html/vendor/autoload_runtime.php')
     (public/index.php:17)               

Fix:
Read via ->all()['thumbnail'] ?? null which bypasses the scalar restriction and handles both value shapes without throwing.

Changes:

• AssetController::getImageThumbnailAction: replace ->get('thumbnail') with ->all()['thumbnail'] ?? null
• Add GetImageThumbnailQueryParamTest

[solverat]

thanks @marcopixel!