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

CommentFileSizeAuthor
#5 interdiff-3136402-2-5.txt372 bytesmrinalini9
#5 3136402-5.patch1.29 KBmrinalini9
#2 3136402-2.patch839 bytesnitvirus

Comments

nitvirus created an issue. See original summary.

nitvirus’s picture

nitvirus commented
Assigned: nitvirus » Unassigned
Status: Needs work » Needs review
StatusFileSize
new 3136402-2.patch839 bytes

Adding patch for both main and test module

manuel garcia’s picture

manuel garcia commented
Issue tags: +Drupal 9 compatibility

Thanks for the path @nitvirus

I have queued test for PHP 7.3 & MySQL 5.7, D9.0 to see what happens with Drupal 9.

manuel garcia’s picture

manuel garcia commented
Status: Needs review » Needs work

There is more work to be done here:

1) Drupal\Tests\view_profiles_perms\Functional\ViewProfilesPermsTest::testViewProfilePerms
Exception: Drupal\Tests\BrowserTestBase::$defaultTheme is required. See https://www.drupal.org/node/3083055, which includes recommendations on which theme to use.
mrinalini9’s picture

mrinalini9 commented
Status: Needs work » Needs review
StatusFileSize
new 3136402-5.patch1.29 KB
new interdiff-3136402-2-5.txt372 bytes

Updated patch by fixing test case failure issues in #2, please review.

  • Manuel Garcia committed f320dd7 on 8.x-1.x 
    Issue #3136402 by mrinalini9, nitvirus, Manuel Garcia: Drupal 9...
manuel garcia’s picture

manuel garcia commented
Status: Needs review » Fixed

Thanks!

manuel garcia’s picture

manuel garcia commented
Category: Support request » Task

Status: Fixed » Closed (fixed)

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