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

CommentFileSizeAuthor
#6 3131683-6.patch1.09 KBmistrae
#5 verf-d9-3131683-5.patch708 bytessutharsan
#5 interdiff-3131683-2-5.txt569 bytessutharsan
#2 3131683-2.patch292 bytesyogeshmpawar

Comments

yogeshmpawar created an issue. See original summary.

yogeshmpawar’s picture

yogeshmpawar
He/Him/His
Primary language English
Location Pune(MH), INDIA 🇮🇳
commented
Assigned: yogeshmpawar » Unassigned
Status: Active » Needs review
Issue tags: +Drupal 9 compatibility, +Drupal 9 readiness, +Drupal 9 porting day
StatusFileSize
new 3131683-2.patch292 bytes

Here's the patch for Drupal 9 readiness.

kristen pol’s picture

kristen pol
she/her
Primary language English
Location Santa Cruz, CA, USA
commented
Title: Drupal 9 readiness » Drupal 9 compatibility
Issue tags: -Drupal 9 readiness

Per a Slack discussion with Gábor Hojtsy last year 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. Please use "Drupal 9 compatibility" for titles and tags on other issues. Thanks.

sutharsan’s picture

sutharsan commented
Assigned: Unassigned » sutharsan
Status: Needs review » Needs work

Upgrade Status report after applying the patch

================================================================================
Views Entity Reference Filter, --
Scanned on Mon, 06/01/2020 - 21:12

FILE: modules/d9/verf/verf.module

STATUS         LINE                           MESSAGE                           
--------------------------------------------------------------------------------
Fix now        46   Call to deprecated method isSubclassOf() of class           
                    Drupal\Core\Entity\EntityTypeInterface. Deprecated in       
                    drupal:8.3.0 and is removed from drupal:9.0.0. Use          
                    Drupal\Core\Entity\EntityTypeInterface::entityClassImplement
                    s() instead.                                                
--------------------------------------------------------------------------------
sutharsan’s picture

sutharsan commented
Assigned: sutharsan » Unassigned
Status: Needs work » Needs review
StatusFileSize
new interdiff-3131683-2-5.txt569 bytes
new verf-d9-3131683-5.patch708 bytes

EntityTypeInterface::isSubclassOf() replaced.

mistrae’s picture

mistrae commented
StatusFileSize
new 3131683-6.patch1.09 KB

Adding core_version_requirement to test submodule

Ravi_3596’s picture

Ravi_3596 commented
Status: Needs review » Reviewed & tested by the community

Patch #6 works perfectly fine for me !!

ggh’s picture

ggh commented

When can we expect a stable release for this module ported for Drupal 9,Can we have some timelines

  • legolasbo committed 2106121 on 8.x-1.x authored by Mistrae 
    Issue #3131683 by Sutharsan, yogeshmpawar, Mistrae, legolasbo: Drupal 9...
legolasbo’s picture

legolasbo
he/him
Primary language Dutch
Location Middelburg
commented
Status: Reviewed & tested by the community » Fixed

Committed and pushed to 8.x-1.x. Thanks!

Status: Fixed » Closed (fixed)

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