Changeset 1057 in ExiteCMS for trunk/includes/user_functions.php

Timestamp:

11/04/07 13:41:50 (5 years ago)

Author:

hverton

Message:

Fixed user_groups parsing error when the user is member of no groups

File:

• trunk/includes/user_functions.php (modified) (1 diff)

trunk/includes/user_functions.php

@@ -136 +136 @@
 
 if (isset($userdata) && is_array($userdata)) {
-    // get the users own group memberships into an array
-    $groups = explode(".", substr($userdata['user_groups'], 1));
-    foreach ($groups as $group) {
-        // check if this groups has subgroups. If so, add them to the array
-        getsubgroups($group);
-    }
-    // create a new user_group field with all inherited groups, and
-    // get the inherited group rights and add them to the user own rights
-    $userdata['user_groups'] = "";
-    foreach ($groups as $group) {
-        $userdata['user_groups'] .= ".".$group;
-        $result = dbarray(dbquery("SELECT group_rights FROM ".$db_prefix."user_groups WHERE group_id = '".$group."'"));
-        if (isset($result['group_rights']) && $result['group_rights'] != "") {
-            $userdata['user_rights'] .= ($userdata['user_rights']==""?"":".").$result['group_rights'];
+    // if group memberships are defined, get the users own group memberships into an array
+    if (!empty($userdata['user_groups'])) {
+        $groups = explode(".", substr($userdata['user_groups'], 1));
+        foreach ($groups as $group) {
+            // check if this groups has subgroups. If so, add them to the array
+            getsubgroups($group);
+        }
+        // create a new user_group field with all inherited groups, and
+        // get the inherited group rights and add them to the user own rights
+        $userdata['user_groups'] = "";
+        foreach ($groups as $group) {
+            $userdata['user_groups'] .= ".".$group;
+            $result = dbarray(dbquery("SELECT group_rights FROM ".$db_prefix."user_groups WHERE group_id = '".$group."'"));
+            if (isset($result['group_rights']) && $result['group_rights'] != "") {
+                $userdata['user_rights'] .= ($userdata['user_rights']==""?"":".").$result['group_rights'];
+            }
         }
     }