Since this is an administrative tool, shouldn't it be available to any user that has the "Administer Role Watchdog" permission?

Or, if you want to be more granular about it, have a new role like "Edit Role Watchdog History" that can be assigned to roles?

If users don't assign the permission to other roles than administrator, the effect is the same, but if multiple roles have the permission it doesn't really make sense in my opinion that they'd have different access, which is why I've marked this as a bug rather than a feature request.

CommentFileSizeAuthor
#4 3155177-4.patch3.94 KBgaurav.kapoor

Comments

kpaxman created an issue. See original summary.

kpaxman’s picture

(sorry, this comment was in the wrong queue)

kpaxman’s picture

Checking further, this is also true for the link to delete the role watchdog record that is referenced by the view.

gaurav.kapoor’s picture

Status: Active » Needs review
StatusFileSize
new3.94 KB

Thanks for spotting this. I have modified permissions accordingly.

kpaxman’s picture

Status: Needs review » Needs work

In limited testing, this fixes the view for new installs, but does not update it for existing installs.

  • gaurav.kapoor authored 197ec7d on 8.x-1.x
    Issue #3155177 by gaurav.kapoor: Role Watchdog History view is checking...
gaurav.kapoor’s picture

Status: Needs work » Fixed

Pushing this for now, so that at least correct permissions are set for new installs. All those with installation based on the previous configuration will have to modify it manually from the view.

Please open a new issue in case you get any.

Status: Fixed » Closed (fixed)

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