Ticket #28436: 28436.3.diff

src/wp-includes/class-wp-theme.php

@@ -1107 +1107 @@
                  * @since MU
                  *
                  * @param array $allowed_themes An array of theme stylesheet names.
+                 * @param int $blog_id Id of the site or network. Defaults to current blog.
                  */
-                $network = (array) apply_filters( 'allowed_themes', self::get_allowed_on_network() );
+                $network = (array) apply_filters( 'allowed_themes_combined', self::get_allowed_on_network(), $blog_id );
                 return $network + self::get_allowed_on_site( $blog_id );
         }
 
@@ -1122 +1123 @@
          */
         public static function get_allowed_on_network() {
                 static $allowed_themes;
-                if ( ! isset( $allowed_themes ) )
-                        $allowed_themes = (array) get_site_option( 'allowedthemes' );
+                if ( ! isset( $allowed_themes ) ) {
+                        /**
+                         * Filter the array of themes allowed on the network.
+                         *
+                         * @since 4.2
+                         *
+                         * @param array $allowed_themes An array of theme stylesheet names.
+                         */
+                        $allowed_themes = (array) apply_filters( 'allowed_themes', get_site_option( 'allowedthemes' ) );
+                }
                 return $allowed_themes;
         }
 
@@ -1191 +1200 @@
                         }
                 }
 
-                return (array) $allowed_themes[ $blog_id ];
+                /**
+                 * Filter the array of themes allowed on the site.
+                 *
+                 * @since 4.2
+                 *
+                 * @param array $allowed_themes An array of theme stylesheet names.
+                 * @param int $blog_id Id of the site. Defaults to current blog.
+                 */
+                $allowed_themes = (array) apply_filters( 'allowed_themes_site', $allowed_themes[ $blog_id ], $blog_id );
+                return $allowed_themes;
         }
 
         /**

new file tests/phpunit/tests/multisite/theme.php

@@ +1 @@
+<?php
+
+if ( is_multisite() ) :
+
+        /**
+         * Tests specific to networks in multisite.
+         *
+         * @group ms-network
+         * @group multisite
+         */
+        class Tests_Multisite_Theme extends WP_UnitTestCase {
+
+                protected $filter_allowed_themes_combined_args;
+
+                function setUp() {
+                        parent::setUp();
+                }
+
+                function tearDown() {
+                        parent::tearDown();
+                }
+
+                function test_nav_menu_theme_filter_sends_blog_id() {
+                        add_filter( 'allowed_themes_combined', array( $this, 'filter_allowed_themes_combined' ), 10, 2 );
+
+                        $blog_id = 1;
+
+                        WP_Theme::get_allowed( $blog_id );
+
+                        $this->assertNotEmpty( $this->filter_allowed_themes_combined_args );
+                        $this->assertEquals( 2, count( $this->filter_allowed_themes_combined_args ) );
+                        $this->assertEquals( $blog_id, $this->filter_allowed_themes_combined_args[1] );
+                }
+
+                function filter_allowed_themes_combined() {
+                        $this->filter_allowed_themes_combined_args = func_get_args();
+
+                }
+        }
+
+endif;
+ No newline at end of file