Ticket #39065: 39065.diff

src/wp-includes/link-template.php

@@ -3472 +3472 @@
         $user_id = $user_id ? (int) $user_id : get_current_user_id();
 
         $blogs = get_blogs_of_user( $user_id );
-        if ( ! is_super_admin() && empty($blogs) ) {
+        if ( is_multisite() && ! user_can( $user_id, 'manage_network' ) && empty($blogs) ) {
                 $url = user_admin_url( $path, $scheme );
         } elseif ( ! is_multisite() ) {
                 $url = admin_url( $path, $scheme );
         } else {
                 $current_blog = get_current_blog_id();
-                if ( $current_blog  && ( is_super_admin( $user_id ) || in_array( $current_blog, array_keys( $blogs ) ) ) ) {
+                if ( $current_blog  && ( user_can( $user_id, 'manage_network' ) || in_array( $current_blog, array_keys( $blogs ) ) ) ) {
                         $url = admin_url( $path, $scheme );
                 } else {
                         $active = get_active_blog_for_user( $user_id );

tests/phpunit/tests/link/getDashboardUrl.php

@@ +1 @@
+<?php
+
+/**
+ * @group link
+ */
+class Tests_Link_GetDashboardUrl extends WP_UnitTestCase {
+
+        /**
+         * @ticket 39065
+         */
+        public function test_get_dashboard_url_for_current_site_user() {
+                $user_id = self::factory()->user->create();
+
+                $this->assertEquals( admin_url(), get_dashboard_url( $user_id ) );
+        }
+
+        /**
+         * @ticket 39065
+         */
+        public function test_get_dashboard_url_for_user_with_no_sites() {
+                $user_id = self::factory()->user->create();
+
+                add_filter( 'get_blogs_of_user', '__return_empty_array' );
+
+                $expected = is_multisite() ? user_admin_url() : admin_url();
+
+                $this->assertEquals( $expected, get_dashboard_url( $user_id ) );
+        }
+
+        /**
+         * @ticket 39065
+         */
+        public function test_get_dashboard_url_for_network_administrator_with_no_sites() {
+                if ( ! is_multisite() ) {
+                        $this->markTestSkipped( 'Test only runs in multisite.' );
+                }
+
+                $user_id = self::factory()->user->create( array( 'role' => 'administrator' ) );
+                grant_super_admin( $user_id );
+
+                add_filter( 'get_blogs_of_user', '__return_empty_array' );
+
+                $this->assertEquals( admin_url(), get_dashboard_url( $user_id ) );
+        }
+
+        /**
+         * @ticket 39065
+         */
+        public function test_get_dashboard_url_for_administrator_of_different_site() {
+                if ( ! is_multisite() ) {
+                        $this->markTestSkipped( 'Test only runs in multisite.' );
+                }
+
+                $user_id = self::factory()->user->create( array( 'role' => 'administrator' ) );
+                remove_user_from_blog( $user_id, get_current_blog_id() );
+
+                $site_id = self::factory()->blog->create( array( 'user_id' => $user_id ) );
+
+                $this->assertEquals( get_admin_url( $site_id ), get_dashboard_url( $user_id ) );
+        }
+}