Ingore loadRevision deprecation

https://drupal.slack.com/archives/C033S2JUMLJ/p1717751306376909

:)

These don't seem to appear in the deprecation status info though? These two appear but they will not match the strpos() proposed:

Call to deprecated method loadRevision() of interface Drupal​\​Core​\​Entity​\​EntityStorageInterface. Deprecated in drupal:10.1.0 and is removed from drupal:11.0.0. Use Drupal​\​Core​\​Entity​\​RevisionableStorageInterface::loadRevision instead.

Call to deprecated method deleteRevision() of interface Drupal​\​Core​\​Entity​\​EntityStorageInterface. Deprecated in drupal:10.1.0 and is removed from drupal:11.0.0. Use Drupal​\​Core​\​Entity​\​RevisionableStorageInterface::deleteRevision instead.

Is deleteRevision not affected by the same situation?

Search at https://dev.acquia.com/drupal11/deprecation_status/errors?error=%2ARevis...

Oops missed delte indeed. Same set.

Silly if I mistyped the strpos, will fix.

Yeah the "mistype" is the lack of method at the end of the strpos, the errors have the respective methods, while you match for RevisionableStorageInterface instead which does not appear as is.

Ok I made a suggestion for what needed changing. Marking it "ignore" will still make it show up in the reported list in Upgrade Status but it will be lower priority. Is that intended? (Similar to issues in future Drupal versions).

Yeah, report it, but then it wont hold back upgrades since the chance for it to actually be an issue is minimal.

Great, thanks!

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