CMS MADE SIMPLE FORGE

Frontend Users

 

[#3351] Permissions fail to alter admin top menu item

avatar
Created By: James Kassemi (jkassemi)
Date Submitted: Thu Apr 23 15:37:22 -0400 2009

Assigned To:
Version: None
CMSMS Version: None
Severity: Critical
Resolution: Fixed
State: Closed
Summary:
Permissions fail to alter admin top menu item
Detailed Description:
Setting the Manage Registering Users permission for lower level accounts has
absolutely no effect on the available menu options for such accounts.

Easy fix:

diff --git a/SelfRegistration.module.php b/SelfRegistration.module.php
index b126d67..61b0a47 100644
--- a/SelfRegistration.module.php
+++ b/SelfRegistration.module.php
@@ -350,7 +350,7 @@ Greetings {$name} welcome to the site.  This email is being
sent because, althou
    ---------------------------------------------------------*/
   function VisibleToAdminUser()
   {
-    $status = $this->CheckPermission('Modify Registering Users')
+    $status = $this->CheckPermission('Manage Registering Users')
       || $this->CheckPermission('Modify Site Preferences')
       || $this->CheckPermission('Modify Templates');
     return $status;


History

Comments
avatar
Date: 2009-04-29 10:47
Posted By: Robert Campbell (calguy1000)

Fixed in SVN
      
Updates

Updated: 2011-11-03 00:24
cmsms_version_id: => -1
state: Open => Closed

Updated: 2009-04-29 10:47
resolution_id: => 7