Checking for sysadmin permission isn't necessary
Description
Environment
None
Test Plan
None
Attachments
1
- 08 Sep 2009, 04:57 AM
Activity
Show:
Nuno Grilo September 9, 2009 at 2:20 AM
trunk: r63137
sitestats_2-0_2.6.x: r63138
sitestats_2-0_2.5.x: r63139
Fixed and verified.
Matthew Buckett September 8, 2009 at 4:57 AM
Attach patch which removes the check for sysadmin.
The security service always checks if the user is sysadmin first when a permission check is done. So the extra check inside sitestats is unneeded.