Problem/Motivation

The permission 'administer gdpr settings' is tied to admin/config/gdpr/task-email. But that permission doesn't exist. Which means only user=1 can access that config page.

Proposed resolution

Add the permissions

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

CommentFileSizeAuthor
#3 missing-permission-for-email-tasks-config-page-3026951-#3.patch1.24 KBkkalaskar
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

heddn created an issue. See original summary.

kkalaskar’s picture

kkalaskar CreditAttribution: kkalaskar as a volunteer and at Asentech LLC commented
Assigned: Unassigned » kkalaskar
kkalaskar’s picture

kkalaskar CreditAttribution: kkalaskar as a volunteer and at Asentech LLC commented
FileSize
missing-permission-for-email-tasks-config-page-3026951-#3.patch1.24 KB

Hi,
I Added the permission under the hook_permission and update the access arguments.
Please have a look and commit the patch.
I applied and tested this patch working fine for me.
Download the patch into modules/contrib/gdpr folder and run the apply command.

git apply -v missing-permission-for-email-tasks-config-page-3026951-#3.patch

After successfully applying patch don't forget to clear cache and then you need to set a permission Administer task emails at /admin/people/permissions

kkalaskar’s picture

kkalaskar CreditAttribution: kkalaskar as a volunteer and at Asentech LLC commented
Status: Active » Needs review

Status: Needs review » Needs work

The last submitted patch, 3: missing-permission-for-email-tasks-config-page-3026951-#3.patch, failed testing. View results

kkalaskar’s picture

kkalaskar CreditAttribution: kkalaskar as a volunteer and at Asentech LLC commented
Status: Needs work » Needs review

  • lbesenyei committed 73824da on 7.x-1.x authored by kkalaskar 
    Issue #3026951 by kkalaskar, heddn: Missing permission for email tasks...
lbesenyei’s picture

lbesenyei CreditAttribution: lbesenyei at Brainsum for Brainsum commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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