Changeset 3227 for branches/plugin/bbp-admin/importers/bbpress.php

Timestamp:

05/26/2011 01:50:45 AM (15 years ago)

Author:

johnjamesjacoby

Message:

Sanity check on user roles to prevent error if user does not have a role assigned to them.

File:

• branches/plugin/bbp-admin/importers/bbpress.php (modified) (1 diff)

branches/plugin/bbp-admin/importers/bbpress.php

@@ -773 +773 @@
 
                     // Get the bbPress roles
-                    $bb_roles =& $user->{ $bbdb->prefix . 'capabilities' };
+                    $bb_roles        =& $user->{ $bbdb->prefix . 'capabilities' };
                     $converted_roles = $converted_level = array();
 
                     // Loop through each role the user has
-                    foreach ( $bb_roles as $bb_role => $bb_role_value ) {
-
-                        // If we have one of those in our roles map, add the WP counterpart in the new roles array
-                        if ( $roles_map[strtolower( $bb_role )] && !empty( $bb_role_value ) ) {
-                            $converted_roles[$roles_map[strtolower( $bb_role )]] = true;
-
-                            // Have support for deprecated levels too
-                            $converted_level[] = $wp_user_level_map[$roles_map[strtolower( $bb_role )]];
-
-                            // We need an admin for future use
-                            if ( empty( $admin_user ) && 'administrator' == $roles_map[strtolower( $bb_role )] )
-                                $admin_user = $user;
+                    if ( !empty( $bb_roles ) ) {
+                        foreach ( $bb_roles as $bb_role => $bb_role_value ) {
+
+                            // If we have one of those in our roles map, add the WP counterpart in the new roles array
+                            if ( $roles_map[strtolower( $bb_role )] && !empty( $bb_role_value ) ) {
+                                $converted_roles[$roles_map[strtolower( $bb_role )]] = true;
+
+                                // Have support for deprecated levels too
+                                $converted_level[] = $wp_user_level_map[$roles_map[strtolower( $bb_role )]];
+
+                                // We need an admin for future use
+                                if ( empty( $admin_user ) && 'administrator' == $roles_map[strtolower( $bb_role )] )
+                                    $admin_user = $user;
+                            }
+
                         }
-
                     }