Changeset 6987

Timestamp:

05/01/2013 07:20:08 PM (13 years ago)

Author:

boonebgorges

Message:

Whitespace and coding standards in BP_Group_Extension::_register(). See #4955

File:

• trunk/bp-groups/bp-groups-classes.php (modified) (3 diffs)

trunk/bp-groups/bp-groups-classes.php

@@ -1443 +1443 @@
         // If admin/create names and slugs are not provided, they fall back on the main
         // name and slug for the extension
-        if ( !$this->admin_name ) {
+        if ( ! $this->admin_name ) {
             $this->admin_name = $this->name;
         }
 
-        if ( !$this->admin_slug ) {
+        if ( ! $this->admin_slug ) {
             $this->admin_slug = $this->slug;
         }
 
-        if ( !$this->create_name ) {
+        if ( ! $this->create_name ) {
             $this->create_name = $this->name;
         }
 
-        if ( !$this->create_slug ) {
+        if ( ! $this->create_slug ) {
             $this->create_slug = $this->slug;
         }
 
-        if ( !empty( $this->enable_create_step ) ) {
+        if ( ! empty( $this->enable_create_step ) ) {
             // Insert the group creation step for the new group extension
-            $bp->groups->group_creation_steps[$this->create_slug] = array( 'name' => $this->create_name, 'slug' => $this->create_slug, 'position' => $this->create_step_position );
+            $bp->groups->group_creation_steps[ $this->create_slug ] = array(
+                'name'     => $this->create_name,
+                'slug'     => $this->create_slug,
+                'position' => $this->create_step_position,
+            );
 
             // Attach the group creation step display content action
@@ -1474 +1478 @@
             if ( $this->visibility == 'public' || ( $this->visibility != 'public' && $bp->groups->current_group->user_has_access ) ) {
                 if ( $this->enable_nav_item ) {
-                    bp_core_new_subnav_item( array( 'name' => ( !$this->nav_item_name ) ? $this->name : $this->nav_item_name, 'slug' => $this->slug, 'parent_slug' => $bp->groups->current_group->slug, 'parent_url' => bp_get_group_permalink( $bp->groups->current_group ), 'position' => $this->nav_item_position, 'item_css_id' => 'nav-' . $this->slug, 'screen_function' => array( &$this, '_display_hook' ), 'user_has_access' => $this->enable_nav_item ) );
+                    bp_core_new_subnav_item( array(
+                        'name' => !$this->nav_item_name ? $this->name : $this->nav_item_name,
+                        'slug' => $this->slug,
+                        'parent_slug' => $bp->groups->current_group->slug,
+                        'parent_url' => bp_get_group_permalink( $bp->groups->current_group ),
+                        'position' => $this->nav_item_position,
+                        'item_css_id' => 'nav-' . $this->slug,
+                        'screen_function' => array( &$this, '_display_hook' ),
+                        'user_has_access' => $this->enable_nav_item
+                    ) );
 
                     // When we are viewing the extension display page, set the title and options title
@@ -1484 +1497 @@
 
                 // Hook the group home widget
-                if ( !bp_current_action() && bp_is_current_action( 'home' ) )
+                if ( ! bp_current_action() && bp_is_current_action( 'home' ) ) {
                     add_action( $this->display_hook, array( &$this, 'widget_display' ) );
+                }
             }
         }
 
         // Construct the admin edit tab for the new group extension
-        if ( !empty( $this->enable_edit_item ) && bp_is_item_admin() ) {
-            add_action( 'groups_admin_tabs', create_function( '$current, $group_slug', '$selected = ""; if ( "' . esc_attr( $this->admin_slug ) . '" == $current ) $selected = " class=\"current\""; echo "<li{$selected}><a href=\"' . trailingslashit( bp_get_root_domain() . '/' . bp_get_groups_root_slug() . '/{$group_slug}/admin/' . esc_attr( $this->admin_slug ) ) . '\">' . esc_attr( $this->admin_name ) . '</a></li>";' ), 10, 2 );
+        if ( ! empty( $this->enable_edit_item ) && bp_is_item_admin() ) {
+            add_action( 'groups_admin_tabs', create_function( '$current, $group_slug',
+                '$selected = "";
+                if ( "' . esc_attr( $this->admin_slug ) . '" == $current )
+                    $selected = " class=\"current\"";
+                echo "<li{$selected}><a href=\"' . trailingslashit( bp_get_root_domain() . '/' . bp_get_groups_root_slug() . '/{$group_slug}/admin/' . esc_attr( $this->admin_slug ) ) . '\">' . esc_attr( $this->admin_name ) . '</a></li>";'
+            ), 10, 2 );
 
             // Catch the edit screen and forward it to the plugin template