Problem/Motivation

From #3275864: Update to Symfony 6.1.1.

  2x: Since symfony/routing 6.1: Construction of "Symfony\Component\Routing\Exception\MissingMandatoryParametersException" with an exception message is deprecated, provide the route name and an array of missing parameters instead.
    2x in HelpTopicTest::testHelpLinks from Drupal\Tests\help_topics\Functional
 1x: Since symfony/routing 6.1: Construction of "Symfony\Component\Routing\Exception\MissingMandatoryParametersException" with an exception message is deprecated, provide the route name and an array of missing parameters instead.
    1x in EntityUrlTest::testUriRelationships from Drupal\Tests\Core\Entity

https://www.drupal.org/pift-ci-job/2364329

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

Issue fork drupal-3276187

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

catch created an issue. See original summary.

catch’s picture

Version: 9.4.x-dev » 10.0.x-dev
Issue tags: +Symfony 6.1
catch’s picture

Issue summary: View changes
mondrake’s picture

Assigned: Unassigned » mondrake

On this

mondrake’s picture

Status: Active » Needs review

No patch yet, we need to agree first. Since there's no Drupal release with Symfony 6.0 yet, and this behaviour is introduced in Symfony 6.1, IMHO the best solution is just to silence this deprecation now, then update Symfony to 6.1, then work directly on the changes with 6.1 without having to care for 6.0 BC.

Thoughts?

catch’s picture

@mondrake that makes sense to me, we only throw the exception from one place in core (UrlGenerator) + a test, so easy to make the change once we're actually on 6.1

mondrake’s picture

Status: Needs review » Postponed

Ok so let's do the deprecation in the parent then.

mondrake’s picture

Assigned: mondrake » Unassigned

mondrake’s picture

Issue tags: +Symfony 7
mondrake’s picture

Status: Postponed » Active

Parent issue was resolved, this can be worked on now.

mondrake’s picture

Status: Active » Needs work
mondrake’s picture

Status: Needs work » Needs review
daffie’s picture

Status: Needs review » Reviewed & tested by the community

The deprecation suppression has been removed and the testbot does not return a deprecation.
The 2 fixes look good to me.
For me it is RTBC.

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed 847ab04 and pushed to 10.0.x. Thanks!

  • alexpott committed 847ab04 on 10.0.x
    Issue #3276187 by mondrake, catch: Since symfony/routing 6.1:...

Status: Fixed » Closed (fixed)

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