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.

Comments

nitvirus created an issue. See original summary.

nitvirus’s picture

Assigned: nitvirus » Unassigned
Status: Needs work » Needs review
StatusFileSize
new839 bytes

Adding patch for both main and test module

manuel garcia’s picture

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

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

Status: Needs work » Needs review
StatusFileSize
new1.29 KB
new372 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

Status: Needs review » Fixed

Thanks!

manuel garcia’s picture

Category: Support request » Task

Status: Fixed » Closed (fixed)

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