Problem/Motivation

Now that Drupal 11 is in beta, the API is hopefully stable enough to add support.

Issue fork rrssb-3453057

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

ptmkenny created an issue. See original summary.

ptmkenny opened merge request !16

ptmkenny’s picture

ptmkenny commented

Upgrade Status module scan notes only one issue, which is fixed in #3453055: entityQuery on settings form is missing an access check. (The other issue is a D12 deprecation.)

Scanned on Fri, 06/07/2024 - 03:30.
1 warning found.
Check manually
Errors without Drupal source version numbers including parse errors and use of APIs from dependencies.
File name	Line	Error
web/modules/rrssb/src/Form/RRSSBSettingsForm.php	244	Relying on entity queries to check access by default is deprecated in drupal:9.2.0 and an error will be thrown from drupal:10.0.0. Call \Drupal\Core\Entity\Query\QueryInterface::accessCheck() with TRUE or FALSE to specify whether access should be checked.
Ignore
Deprecated API use for APIs removed in future Drupal major versions is not required to fix yet.
File name	Line	Error
web/modules/rrssb/rrssb.module	347	Fetching deprecated class constant EXISTS_REPLACE of interface Drupal\Core\File\FileSystemInterface. Deprecated in drupal:10.3.0 and is removed from drupal:12.0.0. Use Drupal\Core\File\FileExists::Replace instead.
ptmkenny’s picture

ptmkenny commented
Status: Active » Needs review

Unit tests pass on D11 so once #3453055: entityQuery on settings form is missing an access check gets committed this should be ready. Note that this MR drops support for D9.

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

adamps’s picture

adamps commented
Status: Needs review » Fixed

Thanks. I'll commit this anyway. I believe #3453055 might not be needed, however if I'm wrong we can easily commit it after.

Status: Fixed » Closed (fixed)

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