Erorr displayed from update manager:

Call to deprecated method getStorage() of class Drupal\Core\Entity\EntityManager. Deprecated in drupal:8.0.0 and is removed from drupal:9.0.0. Use Drupal\Core\Entity\EntityTypeManagerInterface::getStorage() instead.

in line 42 at modules/contrib/imageapi_optimize/src/Form/ImageAPIOptimizePipelineEditForm.php

Interface Drupal\imageapi_optimize\ImageAPIOptimizeProcessorInterface extends deprecated interface Drupal\Component\Plugin\ConfigurablePluginInterface: Drupal\Component\Plugin\ConfigurablePluginInterface is deprecated in Drupal 8.7.0 and will be removed before Drupal 9.0.0. You should implement ConfigurableInterface and/or DependentPluginInterface directly as needed. If you implement ConfigurableInterface you may choose to implement ConfigurablePluginInterface in Drupal 8 as well for maximum compatibility, however this must be removed prior to Drupal 9.
In modules/contrib/imageapi_optimize/src/ImageAPIOptimizeProcessorInterface.php

I see that is needed update testcase class too:

Class PHPUnit\Framework\TestCase not found.

In all tests and also the yml info file:

Add core_version_requirement: ^8 || ^9 to designate that the module is compatible with Drupal 9

I need to learn how to make patches and update here. I tried before but I don't have too much time now, but it's ok for reporting.

Thanks for the module!

CommentFileSizeAuthor
#5 3226168-d9.patch545 bytesjcisio
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:

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

renguer0 created an issue. See original summary.

renguer0’s picture

Issue summary: View changes

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

anfor’s picture

Status: Active » Reviewed & tested by the community

Everything has already been done in this issue : https://www.drupal.org/project/imageapi_optimize/issues/3117904

jcisio’s picture

Version: 4.0.0-beta1 » 4.x-dev
Category: Bug report » Task
Status: Reviewed & tested by the community » Needs review
FileSize
545 bytes

The fact that we changed tests for class renames in 9.2 means that we don't really support Drupal 8. This patch makes sure that it is D9+ only.

  • jcisio committed bf763e5 on 4.x
    Issue #3226168 by anfor, jcisio: Drupal 9 compatibility
    
jcisio’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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