Problem/Motivation

Deprecation warning being shown on line 352, method getRoutesByPath().
File: core/lib/Drupal/Core/Routing/RouteProvider.php

Proposed resolution

Adding compatibility with PHP 8.1

CommentFileSizeAuthor
#8 3370580-7.patch1.59 KBsuryabhi
#7 3370580-6.patch866 bytessuryabhi

Issue fork drupal-3370580

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:

About issue forks

Comments

esdrasterrero created an issue. See original summary.

esdrasterrero opened merge request !4271

esdrasterrero’s picture

esdrasterrero commented
Status: Active » Needs review
smustgrave’s picture

smustgrave commented
Version: 10.0.x-dev » 11.x-dev
Assigned: esdrasterrero » Unassigned
Status: Needs review » Needs work
Issue tags: +Needs steps to reproduce, +Needs tests

Thank you for taking an interest

First you shouldn't assign tickets to yourself unless your a maintainer, simple comment should do https://www.drupal.org/docs/develop/issues/issue-procedures-and-etiquett...

Also this will need steps to reproduce. Need to investigate why this null vs just an isset or empty check. Could be masking a larger problem.

11.x is current development branch

suryabhi made their first commit to this issue’s fork.

suryabhi opened merge request !4280

suryabhi’s picture

suryabhi commented
StatusFileSize
new 3370580-6.patch866 bytes

Kindly review the Patch

suryabhi’s picture

suryabhi commented
StatusFileSize
new 3370580-7.patch1.59 KB

Build issue fixed patch.

suryabhi’s picture

suryabhi commented
cilefen’s picture

cilefen commented

Contributors: please do not ignore comment #4.

suryabhi’s picture

suryabhi commented
Status: Needs work » Needs review
smustgrave’s picture

smustgrave commented
Status: Needs review » Needs work

Was previously tagged for steps to reproduce and tests.

Also with these kind of issues we need to figure out why that variable is empty vs just putting an issuet() or !empty() check.

Thanks!

Version: 11.x-dev » main

Drupal core is now using the main branch as the primary development branch. New developments and disruptive changes should now be targeted to the main branch.

Read more in the announcement.