The module currently only provides one permission to manage protected pages and the module's settings as well as email functionality.
I would like to change this and introduce new permissions so we can e.g. define that users are not able to send email nor change the module settings - but they're able to manage which pages should be protected (= content).

I'll attach a patch for this soon.

CommentFileSizeAuthor
#9 split-up-permissions-2996312-9.patch6.33 KBarkener
#8 split-up-permissions-2996312-8.patch4.22 KBarkener
#5 split-up-permissions-2996312-5.patch6.25 KBdpacassi
#2 split-up-permissions-2996312-2.patch4.19 KBdpacassi

Comments

dpacassi created an issue. See original summary.

dpacassi’s picture

dpacassi
Primary language English
Location Zürich, Switzerland
commented
StatusFileSize
new split-up-permissions-2996312-2.patch4.19 KB

See attached patch for updated permissions.

dpacassi’s picture

dpacassi
Primary language English
Location Zürich, Switzerland
commented
Status: Active » Needs review

Status: Needs review » Needs work

The last submitted patch, 2: split-up-permissions-2996312-2.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

dpacassi’s picture

dpacassi
Primary language English
Location Zürich, Switzerland
commented
StatusFileSize
new split-up-permissions-2996312-5.patch6.25 KB

Updated tests and also provided an update hook to migrate the old permission to the new ones.

dpacassi’s picture

dpacassi
Primary language English
Location Zürich, Switzerland
commented
Status: Needs work » Needs review
bkosborne’s picture

bkosborne
Primary language English
Location New Jersey, USA
commented

+1 to this. I'm evaluating this module and this email functionality is something I won't use and would be great to disable it!

arkener’s picture

arkener commented
StatusFileSize
new split-up-permissions-2996312-8.patch4.22 KB

+1, it would be great to be able to separate the permissions for the module settings and the ability to moderate protected pages. Rerolling the patch against 8.x-1.x.

arkener’s picture

arkener commented
StatusFileSize
new split-up-permissions-2996312-9.patch6.33 KB

Uploading the correct rerolled patch...