Check the compatibility of Lazy-load module for upcoming Drupal 9.

CommentFileSizeAuthor
#9 drupal_9_readiness-3094992-9.patch467 bytesjannakha
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

osman created an issue. See original summary.

  • osman committed 58aa7af on 8.x-3.x 
    Issue #3094992: Drupal 9 readiness
osman’s picture

osman
Location San Francisco, CA
CreditAttribution: osman as a volunteer commented

Tried with ed5986f5, works normal, and no issue has been observed.

Kristen Pol’s picture

Kristen Pol
she/her
Primary language English
Location Santa Cruz, CA, USA
CreditAttribution: Kristen Pol at Hook 42 commented
Issue tags: +Drupal 9 readiness
Kristen Pol’s picture

Kristen Pol
she/her
Primary language English
Location Santa Cruz, CA, USA
CreditAttribution: Kristen Pol at Hook 42 commented
Title: Drupal 9 readiness » Drupal 9 readiness / compatibility
Issue summary: View changes
Issue tags: -Drupal 9 readiness +Drupal 9 compatibility

Per a Slack discussion with Gábor Hojtsy regarding usage of D9 tags (Drupal 9, Drupal 9 compatibility, Drupal 9 readiness, etc.), "Drupal 9 compatibility" should be used for contributed projects that need updating and "Drupal 9" was the old tag for D8 issues before the D9 branch was ready. Doing title and tag cleanup here based on that discussion.

Pavel_SPN’s picture

Pavel_SPN CreditAttribution: Pavel_SPN at EPAM Systems commented

Hi, no compatibility issues with Drupal 9. Drupal-check test passed without errors.

  • osman committed e4c2146 on 8.x-3.x 
    Issue #3094992 by osman: Drupal 8.8.0 compatibility

@see https://www....
osman’s picture

osman
Location San Francisco, CA
CreditAttribution: osman as a volunteer commented

e4c2146 adds compatibility with Drupal core v8.8.0.

\Drupal\Core\Path\AliasManagerInterface is deprecated in drupal:8.8.0 and is removed from drupal:9.0.0. Use \Drupal\path_alias\AliasManagerInterface.

See the change record: https://www.drupal.org/node/3092086

jannakha’s picture

jannakha CreditAttribution: jannakha as a volunteer and at Tomato Elephant Studio commented
Status: Active » Needs review
Issue tags: +Drupal 9 porting weekend
FileSize
drupal_9_readiness-3094992-9.patch467 bytes

Drupal 9 compatibility checks done:
- status_upgrade - composer.json issue
- drupal/rector - return no errors (https://github.com/palantirnet/drupal-rector)

jungle’s picture

jungle
He/Him
Location Chongqing, China
CreditAttribution: jungle as a volunteer commented
Status: Needs review » Reviewed & tested by the community

Thanks, @JannaKha!

Double-checked. #9 is the only one left, and the fix is straight and good.

  • osman committed a4f6420 on 8.x-3.x authored by JannaKha 
    Issue #3094992 by JannaKha, osman: Drupal 9 readiness / compatibility
osman’s picture

osman
Location San Francisco, CA
CreditAttribution: osman as a volunteer commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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