Problem/Motivation

The last release of this module was created almost 2 years ago, and it does not support Drupal 9. We need a new release of the module that can be installed in Drupal 9.

Steps to reproduce

Try to install version 1.2 of this module in Drupal 9.

Proposed resolution

Create a 1.3 release.

Remaining tasks

Tag the new version of the module, and push it.

Comments

jcnventura created an issue. See original summary.

avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Title: Please create a 1.3 release » Please create a new release compatible with Drupal 9
Issue tags: +Drupal 9 compatibility

Version 8.x-1.3 has been created on December. I cannot say if it's compatible with Drupal 9, though.

avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Related issues: +#3198542: Unecessary getConnection method

Actually, the latest developed code doesn't seem compatible with latest Drupal 8 versions either.

vipin.mittal18’s picture

vipin.mittal18
He/Him
Primary language English
Location Greater Noida, India
commented

I am aligned with you kiamlaluno.

Though maintainer has added core version requirement in the info file, still I can see Drupal9 practices are not followed like creating an object in the controller class instead of using dependency injection.

thalles’s picture

thalles
Primary language Portuguese, Brazil
Location Teófilo Otoni - MG
commented

Sorry! I will provide this!

thalles’s picture

thalles
Primary language Portuguese, Brazil
Location Teófilo Otoni - MG
commented

I ran phpcs with drupal / coder and drupal-check and found no problem.
If you know any issue, can you please share them here?

thalles’s picture

thalles
Primary language Portuguese, Brazil
Location Teófilo Otoni - MG
commented
Status: Active » Postponed (maintainer needs more info)
avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented

To be compatible with Drupal 9, at least #3198542: Unecessary getConnection method should be fixed.The code is calling a method that doesn't exist, which was removed from Drupal 8 time ago.

I didn't check for other issues; I cannot exclude there are other incompatibilities.

thalles’s picture

thalles
Primary language Portuguese, Brazil
Location Teófilo Otoni - MG
commented

Thanks!

thalles’s picture

thalles
Primary language Portuguese, Brazil
Location Teófilo Otoni - MG
commented
Status: Postponed (maintainer needs more info) » Fixed

Done!

Status: Fixed » Closed (fixed)

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