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

Issue fork imageapi_optimize-3226168

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

renguer0 created an issue. See original summary.

renguer0’s picture

renguer0 commented
Issue summary: View changes

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

anfor’s picture

anfor commented
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

jcisio commented
Version: 4.0.0-beta1 » 4.x-dev
Category: Bug report » Task
Status: Reviewed & tested by the community » Needs review
StatusFileSize
new 3226168-d9.patch545 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

jcisio commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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