Contrib.social

Call to deprecated method loadRevision() of class Drupal\Core\Entity\EntityStorageInterface:

Open on Drupal.org →
Created on 25 July 2023, 11 months ago
Updated 16 February 2024, 4 months ago

Problem/Motivation

I have got a deprecation in this module.
The class of loadRevision() method is changed in Drupal 10 See. https://www.drupal.org/node/3294237 →
It will removed in Drupal 11.

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

Steps to reproduce

Proposed resolution

Replace
\Drupal\Core\Entity\EntityStorageInterface::loadRevision()
to
\Drupal\Core\Entity\RevisionableStorageInterface::loadRevision()

Remaining tasks

User interface changes

API changes

Data model changes

🐛 Bug report
Status

Closed: won't fix

Version

1.0

Component

Code

Created by

🇮🇳India keshav.k

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

  • Issue created by @keshav.k
  • Comment 11 months ago →
    🇮🇳India keshav.k
  • First commit to issue fork.
  • Open in Jenkins → Open on Drupal.org →
    Core: 9.5.x + Environment: PHP 7.4 & MySQL 5.7
    last update 11 months ago
    17 pass, 148 fail
  • @sakthi_dev opened merge request.
  • Status changed to Needs review 11 months ago
  • Comment 11 months ago →
    🇮🇳India sakthi_dev

    Please review.

  • Comment 11 months ago →
    🇩🇪Germany marcoliver Neuss, NRW, Germany

    While the MR indeed updates the interfaces used directly in the module, I believe the MR is currently not ready to merge simply for the fact, that right now Paragraphs supports Core => 9.3 and 10.x. But the change this MR addresses was only introduced in 10.1.x, so merging this into 8.x-1.x would render this branch unusable for any Core version below 10.1.x.

    So maybe a dedicated branch for 11-compatibility might be needed here?

  • Status changed to Closed: won't fix 10 months ago
  • Comment 10 months ago →
    🇨🇭Switzerland Berdir Switzerland

    this is a false positive in phpstan drupal integration that has been fixed since now.

  • Comment 4 months ago →
    🇭🇺Hungary Pasqualle 🇭🇺 Budapest

    Why this is "won't fix". Should not it be in the postponed state until paragraphs supports Drupal 11?

contrib.social Blog FAQ Discussions
Production build 0.69.0 2024