- Issue created by @trackleft2
- Merge request !12Issue #3502075 Support masquerade module version 2.0.0-rc5 and greater. β (Merged) created by trackleft2
Confirmed that the functionality is broken, and that this MR fixes it. Thanks.
The module literally doesn't work without this fix.
By the way, it'd be awesome if you could review π LogicException when logging out while masquerading Needs review
- πΊπΈUnited States trackleft2 Tucson, AZ πΊπΈ
Will review the other MR, thanks for reviewing this one.
FYI, when I attempted to move the functionality of this merge request into the masquerade module for β¨ Update log messages to reflect that a user was masquerading Needs review
I wasn't able to make it work. This is also why the MR on that issue is so different from this one.This leads me to believe that I set my test site up incorrectly when I made this merge request (using an older version of masquerade module) I do want to test this again ensuring the latest version of masquerade and the latest version of core it runs on.
Ideally, you wouldn't need to do any setup. You should just be able to use DDEV to install the modules and spin up a site, then run
ddev phpunit
from https://github.com/ddev/ddev-drupal-contrib to run all the unit tests.-
trackleft2 β
committed 27ba1671 on 2.0.x
Issue #3502075 Support masquerade module version 2.0.0-rc5 and greater.
-
trackleft2 β
committed 27ba1671 on 2.0.x
Automatically closed - issue fixed for 2 weeks with no activity.