Problem/Motivation

The Drupal\Tests\media\Kernel\OEmbedIframeControllerTest fails, because in the method Drupal\media\Controller\OEmbedIframeController::render() the second parameter of the method Symfony\Component\HttpFoundation\ParameterBag::getInt() is set to NULL and the second parameter need to be an integer and 0 is its default value.

Proposed resolution

The problem can be fixed by not setting the second parameter. The default value of 0 is fine.

Remaining tasks

TBD

User interface changes

None

API changes

None

Data model changes

None

Release notes snippet

TBD

CommentFileSizeAuthor
#2 3248014-2.patch849 bytesdaffie

Comments

daffie created an issue. See original summary.

daffie’s picture

daffie commented
Status: Active » Needs review
StatusFileSize
new 3248014-2.patch849 bytes

The fix.

andypost’s picture

andypost
he/him
Primary language Russian
commented
Status: Needs review » Reviewed & tested by the community

Yes, that's enough https://github.com/symfony/symfony/blob/5.4/src/Symfony/Component/HttpFo...

  • catch committed 507fc5f on 9.4.x 
    Issue #3248014 by daffie, andypost: [Symfony 6] The Drupal\Tests\media\...
catch’s picture

catch
he/him
Primary language English
commented
Version: 9.4.x-dev » 9.3.x-dev
Status: Reviewed & tested by the community » Fixed

Committed/pushed to 9.4.x and cherry-picked to 9.3.x, thanks!

  • catch committed ce8e734 on 9.3.x 
    Issue #3248014 by daffie, andypost: [Symfony 6] The Drupal\Tests\media\...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.