Skip to content

Commit f35f825

Browse files
mmtrmatticbot
authored andcommitted
Untangling: Show "Jetpack -> Subscribers" menu on all sites (#44302)
- Move the `/subscribers/:site` path to the Jetpack > Subscribers menu for all sites (that menu was previously only visible with the classic interface). - The existing Users > Subscribers menu (only visible with the default interface) will now point to `/subscribers/jetpack-subscribers/:site` so we can inform users with a callout that the menu has been moved (see DOTCOM-13883). This menu will be removed after a month, and it's only visible to existing users. Committed via a GitHub action: https://github.com/Automattic/jetpack/actions/runs/16344258243 Upstream-Ref: Automattic/jetpack@f71d3bc
1 parent 6309fba commit f35f825

File tree

7 files changed

+184
-190
lines changed

7 files changed

+184
-190
lines changed

jetpack_vendor/automattic/jetpack-masterbar/src/admin-menu/class-atomic-admin-menu.php

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@
1111
use Automattic\Jetpack\Current_Plan as Jetpack_Plan;
1212
use Automattic\Jetpack\JITMS\JITM;
1313
use Automattic\Jetpack\Modules;
14-
use Automattic\Jetpack\Subscribers_Dashboard\Dashboard as Subscribers_Dashboard;
1514

1615
require_once __DIR__ . '/class-admin-menu.php';
1716

@@ -138,13 +137,10 @@ public function add_users_menu() {
138137
$this->update_submenus( $slug, $submenus_to_update );
139138
}
140139

141-
if ( ! $this->use_wp_admin_interface() && ! apply_filters( 'jetpack_wp_admin_subscriber_management_enabled', false ) ) {
142-
// The 'Subscribers' menu exists in the Jetpack menu for Classic wp-admin interface, so only add it for non-wp-admin interfaces.
140+
// Temporary "Users > Subscribers" menu for existing users that shows a callout informing that the screen has moved to "Jetpack > Subscribers".
141+
if ( ! $this->use_wp_admin_interface() && ! apply_filters( 'jetpack_wp_admin_subscriber_management_enabled', false ) && get_current_user_id() < 268854000 ) {
143142
// // @phan-suppress-next-line PhanTypeMismatchArgumentProbablyReal -- Core should ideally document null for no-callback arg. https://core.trac.wordpress.org/ticket/52539.
144-
add_submenu_page( 'users.php', esc_attr__( 'Subscribers', 'jetpack-masterbar' ), __( 'Subscribers', 'jetpack-masterbar' ), 'list_users', 'https://wordpress.com/subscribers/' . $this->domain, null );
145-
} elseif ( apply_filters( 'jetpack_wp_admin_subscriber_management_enabled', false ) ) {
146-
$subscribers_dashboard = new Subscribers_Dashboard();
147-
$subscribers_dashboard->add_wp_admin_submenu();
143+
add_submenu_page( 'users.php', esc_attr__( 'Subscribers', 'jetpack-masterbar' ), __( 'Subscribers', 'jetpack-masterbar' ), 'list_users', 'https://wordpress.com/subscribers/jetpack-subscribers/' . $this->domain, null );
148144
}
149145

150146
// Users who can't 'list_users' will see "Profile" menu & "Profile > Account Settings" as submenu.

jetpack_vendor/automattic/jetpack-masterbar/src/admin-menu/class-wpcom-admin-menu.php

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,6 @@
77

88
namespace Automattic\Jetpack\Masterbar;
99

10-
use Automattic\Jetpack\Subscribers_Dashboard\Dashboard as Subscribers_Dashboard;
1110
use Jetpack_Custom_CSS;
1211
use JITM;
1312

@@ -271,12 +270,11 @@ public function add_users_menu() {
271270
$this->update_submenus( $slug, $submenus_to_update );
272271
// @phan-suppress-next-line PhanTypeMismatchArgumentProbablyReal -- Core should ideally document null for no-callback arg. https://core.trac.wordpress.org/ticket/52539.
273272
add_submenu_page( 'users.php', esc_attr__( 'Add New User', 'jetpack-masterbar' ), __( 'Add New User', 'jetpack-masterbar' ), 'promote_users', 'https://wordpress.com/people/new/' . $this->domain, null, 1 );
274-
if ( ! apply_filters( 'jetpack_wp_admin_subscriber_management_enabled', false ) ) {
273+
274+
// Temporary "Users > Subscribers" menu for existing users that shows a callout informing that the screen has moved to "Jetpack > Subscribers".
275+
if ( ! apply_filters( 'jetpack_wp_admin_subscriber_management_enabled', false ) && get_current_user_id() < 268854000 ) {
275276
// @phan-suppress-next-line PhanTypeMismatchArgumentProbablyReal -- Core should ideally document null for no-callback arg. https://core.trac.wordpress.org/ticket/52539.
276-
add_submenu_page( 'users.php', esc_attr__( 'Subscribers', 'jetpack-masterbar' ), __( 'Subscribers', 'jetpack-masterbar' ), 'list_users', 'https://wordpress.com/subscribers/' . $this->domain, null, 3 );
277-
} else {
278-
$subscribers_dashboard = new Subscribers_Dashboard();
279-
$subscribers_dashboard->add_wp_admin_submenu();
277+
add_submenu_page( 'users.php', esc_attr__( 'Subscribers', 'jetpack-masterbar' ), __( 'Subscribers', 'jetpack-masterbar' ), 'list_users', 'https://wordpress.com/subscribers/jetpack-subscribers/' . $this->domain, null, 3 );
280278
}
281279
}
282280

jetpack_vendor/automattic/jetpack-mu-wpcom/src/features/wpcom-admin-menu/wpcom-admin-menu.php

Lines changed: 16 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -245,6 +245,22 @@ function wpcom_add_jetpack_submenu() {
245245
null // @phan-suppress-current-line PhanTypeMismatchArgumentProbablyReal -- Core should ideally document null for no-callback arg. https://core.trac.wordpress.org/ticket/52539.
246246
);
247247

248+
// Jetpack > Subscribers.
249+
if ( ! apply_filters( 'jetpack_wp_admin_subscriber_management_enabled', false ) ) {
250+
wpcom_hide_submenu_page( 'jetpack', esc_url( Redirect::get_url( 'jetpack-menu-jetpack-manage-subscribers', array( 'site' => $blog_id ) ) ) );
251+
add_submenu_page(
252+
'jetpack',
253+
__( 'Subscribers', 'jetpack-mu-wpcom' ),
254+
__( 'Subscribers', 'jetpack-mu-wpcom' ),
255+
'manage_options',
256+
'https://wordpress.com/subscribers/' . $domain,
257+
null // @phan-suppress-current-line PhanTypeMismatchArgumentProbablyReal -- Core should ideally document null for no-callback arg. https://core.trac.wordpress.org/ticket/52539.
258+
);
259+
} else {
260+
$subscribers_dashboard = new Subscribers_Dashboard();
261+
$subscribers_dashboard->add_wp_admin_submenu();
262+
}
263+
248264
if ( $uses_wp_admin_interface ) {
249265
// Jetpack > Activity Log.
250266
wpcom_hide_submenu_page( 'jetpack', esc_url( Redirect::get_url( 'cloud-activity-log-wp-menu', array( 'site' => $blog_id ) ) ) );
@@ -257,22 +273,6 @@ function wpcom_add_jetpack_submenu() {
257273
null // @phan-suppress-current-line PhanTypeMismatchArgumentProbablyReal -- Core should ideally document null for no-callback arg. https://core.trac.wordpress.org/ticket/52539.
258274
);
259275

260-
// Jetpack > Subscribers.
261-
if ( ! apply_filters( 'jetpack_wp_admin_subscriber_management_enabled', false ) ) {
262-
wpcom_hide_submenu_page( 'jetpack', esc_url( Redirect::get_url( 'jetpack-menu-jetpack-manage-subscribers', array( 'site' => $blog_id ) ) ) );
263-
add_submenu_page(
264-
'jetpack',
265-
__( 'Subscribers', 'jetpack-mu-wpcom' ),
266-
__( 'Subscribers', 'jetpack-mu-wpcom' ),
267-
'manage_options',
268-
'https://wordpress.com/subscribers/' . $domain,
269-
null // @phan-suppress-current-line PhanTypeMismatchArgumentProbablyReal -- Core should ideally document null for no-callback arg. https://core.trac.wordpress.org/ticket/52539.
270-
);
271-
} else {
272-
$subscribers_dashboard = new Subscribers_Dashboard();
273-
$subscribers_dashboard->add_wp_admin_submenu();
274-
}
275-
276276
// Jetpack > Newsletter.
277277
if ( $is_simple_site ) {
278278
add_submenu_page(

jetpack_vendor/i18n-map.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -50,11 +50,11 @@
5050
),
5151
'jetpack-masterbar' => array(
5252
'path' => 'jetpack_vendor/automattic/jetpack-masterbar',
53-
'ver' => '0.19.0-alpha1752749930',
53+
'ver' => '0.19.0-alpha1752752834',
5454
),
5555
'jetpack-mu-wpcom' => array(
5656
'path' => 'jetpack_vendor/automattic/jetpack-mu-wpcom',
57-
'ver' => '6.7.0-alpha1752749930',
57+
'ver' => '6.7.0-alpha1752752834',
5858
),
5959
'jetpack-password-checker' => array(
6060
'path' => 'jetpack_vendor/automattic/jetpack-password-checker',

0 commit comments

Comments
 (0)