Ran upgrade status on the module and found this warning.

Add core_version_requirement: ^8 || ^9 to imagezoom.info.yml to designate that the module is compatible with Drupal 9. See https://www.drupal.org/node/3070687.

CommentFileSizeAuthor
#5 interdiff_2_5.txt359 bytesrahulrasgon
#5 3134137-5.patch827 bytesrahulrasgon
#2 3134137-2.patch344 byteskarishmaamin

Issue fork imagezoom-3134137

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

karishmaamin created an issue. See original summary.

karishmaamin’s picture

karishmaamin commented
Status: Active » Needs review
StatusFileSize
new 3134137-2.patch344 bytes
nitvirus’s picture

nitvirus commented
Issue tags: +Drupal 9 compatibility, +Drupal 9 readiness

adding specific tags

rahulrasgon’s picture

rahulrasgon commented
Assigned: Unassigned » rahulrasgon
Status: Needs review » Needs work
rahulrasgon’s picture

rahulrasgon commented
StatusFileSize
new 3134137-5.patch827 bytes
new interdiff_2_5.txt359 bytes

Adding core_version_requirement: ^8 || ^9 to submodule as well.

Please review the updated patch.

rahulrasgon’s picture

rahulrasgon commented
Assigned: rahulrasgon » Unassigned
Status: Needs work » Needs review
valdes14’s picture

valdes14 commented

Thanks a lot for the patch in #5. It works when applied to a D8 install for upgrading to D9, but Imagezoom cannot be installed on a fresh D9 as the patch is not included in the module yet.

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

AstonVictor opened merge request !1

astonvictor’s picture

astonvictor commented
Status: Needs review » Fixed

Created and merged a new MR.

Fixed deprecated functions.

Status: Fixed » Closed (fixed)

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