Changeset 9555

Timestamp:

02/24/2015 01:48:42 PM (12 years ago)

Author:

johnjamesjacoby

Message:

Core: Allow custom components to continue to have directory pages.

Fixes regression introduced in 2.2. Props imath. See #6244. (trunk)

Location:

trunk

Files:

• src/bp-core/admin/bp-core-admin-components.php (modified) (3 diffs)
• tests/phpunit/testcases/admin/functions.php (modified) (2 diffs)

trunk/src/bp-core/admin/bp-core-admin-components.php

@@ -308 +308 @@
         }
 
-        $current_components = buddypress()->active_components;
+        $current_components       = buddypress()->active_components;
+        $packaged_components      = array_flip( bp_core_get_packaged_component_ids() );
+        $custom_active_components = array_diff_key( $current_components, $packaged_components );
 
         switch ( $current_action ) {
@@ -316 +318 @@
                                 if ( ! isset( $submitted[ $retired_component ] ) ) {
                                         unset( $current_components[ $retired_component ] );
+
+                                        // Make sure custom components does not contain a retired component
+                                        if ( isset( $custom_active_components[ $retired_component ] ) ) {
+                                                unset( $custom_active_components[ $retired_component ] );
+                                        }
                                 }
                         }
@@ -331 +338 @@
         }
 
-        return $components;
+        // Active components is the list of packaged and custom components
+        return array_merge( $components, $custom_active_components );
 }
 

trunk/tests/phpunit/testcases/admin/functions.php

@@ -39 +39 @@
         }
 
+        /**
+         * @group bp_core_admin_get_active_components_from_submitted_settings
+         */
         public function test_bp_core_admin_get_active_components_from_submitted_settings() {
                 $get_action = isset( $_GET['action'] ) ? $_GET['action'] : null;
@@ -123 +126 @@
 
         /**
+         * @group BP6244
+         * @group bp_core_admin_get_active_components_from_submitted_settings
+         */
+        public function test_bp_core_admin_get_active_components_from_submitted_settings_should_keep_custom_component_directory_page() {
+                $bp = buddypress();
+                $reset_active_components = $bp->active_components;
+
+                // Create and activate the foo component
+                $bp->foo = new BP_Component;
+                $bp->foo->id   = 'foo';
+                $bp->foo->slug = 'foo';
+                $bp->foo->name = 'Foo';
+                $bp->active_components[ $bp->foo->id ] = 1;
+                $new_page_ids = array( $bp->foo->id => $this->factory->post->create( array(
+                        'post_type'  => 'page',
+                        'post_title' => $bp->foo->name,
+                        'post_name'  => $bp->foo->slug,
+                ) ) );
+
+                $page_ids = array_merge( $new_page_ids, (array) bp_core_get_directory_page_ids() );
+                bp_core_update_directory_page_ids( $page_ids );
+
+                $bp->active_components = bp_core_admin_get_active_components_from_submitted_settings( $reset_active_components );
+                bp_core_add_page_mappings( $bp->active_components );
+
+                $this->assertContains( $bp->foo->id, array_keys( bp_core_get_directory_page_ids() ) );
+
+                // Reset buddypress() vars
+                $bp->active_components = $reset_active_components;
+        }
+
+        /**
          * @group bp_core_activation_notice
          */