Confirmed the bug affects CMS 4 as well, so fixing it there.
Manual testing steps
Copy the SecurityAdmin from CMS 5 and plonk it into CMS 4, then try to import a member. If you add it as a separate class you'll want to set a different url_segment and menu_title so you know what one you're looking at. Note that there's syntax in there that needs to run on PHP 8.1 - but that doesn't affect the actual code fix which as per CI is compatible with 7.4
[x] All commits are relevant to the purpose of the PR (e.g. no debug statements, unrelated refactoring, or arbitrary linting)
Small amounts of additional linting are usually okay, but if it makes it hard to concentrate on the relevant changes, ask for the unrelated changes to be reverted, and submitted as a separate PR.
Description
Confirmed the bug affects CMS 4 as well, so fixing it there.
Manual testing steps
Copy the
SecurityAdmin
from CMS 5 and plonk it into CMS 4, then try to import a member. If you add it as a separate class you'll want to set a differenturl_segment
andmenu_title
so you know what one you're looking at. Note that there's syntax in there that needs to run on PHP 8.1 - but that doesn't affect the actual code fix which as per CI is compatible with 7.4Issues
Pull request checklist