Skip to content

Commit de60b47

Browse files
authored
Closes wp-media#6413 Remove tags from services (wp-media#6977)
1 parent a0f3dec commit de60b47

28 files changed

+100
-201
lines changed

inc/Addon/Cloudflare/ServiceProvider.php

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -58,8 +58,7 @@ public function register(): void {
5858
->addArgument( $this->getContainer()->get( 'cloudflare' ) )
5959
->addArgument( $options )
6060
->addArgument( $this->getContainer()->get( 'options_api' ) )
61-
->addArgument( $this->getContainer()->get( 'cloudflare_auth_factory' ) )
62-
->addTag( 'cloudflare_subscriber' );
61+
->addArgument( $this->getContainer()->get( 'cloudflare_auth_factory' ) );
6362
$this->getContainer()->addShared(
6463
'cloudflare_admin_subscriber',
6564
CloudflareAdminSubscriber::class

inc/Addon/ServiceProvider.php

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -40,19 +40,16 @@ public function register(): void {
4040

4141
// Sucuri Addon.
4242
$this->getContainer()->addShared( 'sucuri_subscriber', SucuriSubscriber::class )
43-
->addArgument( $options )
44-
->addTag( 'common_subscriber' );
43+
->addArgument( $options );
4544

4645
$this->getContainer()->addShared( 'webp_admin_subscriber', WebPAdminSubscriber::class )
4746
->addArgument( $options )
4847
->addArgument( $this->getContainer()->get( 'cdn_subscriber' ) )
49-
->addArgument( $this->getContainer()->get( 'beacon' ) )
50-
->addTag( 'common_subscriber' );
48+
->addArgument( $this->getContainer()->get( 'beacon' ) );
5149

5250
$this->getContainer()->addShared( 'webp_subscriber', WebPSubscriber::class )
5351
->addArgument( $options )
5452
->addArgument( $this->getContainer()->get( 'options_api' ) )
55-
->addArgument( $this->getContainer()->get( 'cdn_subscriber' ) )
56-
->addTag( 'common_subscriber' );
53+
->addArgument( $this->getContainer()->get( 'cdn_subscriber' ) );
5754
}
5855
}

inc/Addon/Varnish/ServiceProvider.php

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,6 @@ public function register(): void {
3737
$this->getContainer()->add( 'varnish', Varnish::class );
3838
$this->getContainer()->addShared( 'varnish_subscriber', Subscriber::class )
3939
->addArgument( $this->getContainer()->get( 'varnish' ) )
40-
->addArgument( $this->getContainer()->get( 'options' ) )
41-
->addTag( 'common_subscriber' );
40+
->addArgument( $this->getContainer()->get( 'options' ) );
4241
}
4342
}

inc/Engine/Admin/Beacon/ServiceProvider.php

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,6 @@ public function register(): void {
3636
$this->getContainer()->addShared( 'beacon', Beacon::class )
3737
->addArgument( $this->getContainer()->get( 'options' ) )
3838
->addArgument( $this->getContainer()->get( 'template_path' ) . '/settings' )
39-
->addArgument( $this->getContainer()->get( 'support_data' ) )
40-
->addTag( 'admin_subscriber' );
39+
->addArgument( $this->getContainer()->get( 'support_data' ) );
4140
}
4241
}

inc/Engine/Admin/Database/ServiceProvider.php

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,6 @@ public function register(): void {
4040
->addArgument( $this->getContainer()->get( 'db_optimization_process' ) );
4141
$this->getContainer()->addShared( 'db_optimization_subscriber', Subscriber::class )
4242
->addArgument( $this->getContainer()->get( 'db_optimization' ) )
43-
->addArgument( $this->getContainer()->get( 'options' ) )
44-
->addTag( 'common_subscriber' );
43+
->addArgument( $this->getContainer()->get( 'options' ) );
4544
}
4645
}

inc/Engine/Admin/ServiceProvider.php

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -47,15 +47,12 @@ public function register(): void {
4747
->addArgument( $this->getContainer()->get( 'options_api' ) )
4848
->addArgument( $options );
4949
$this->getContainer()->addShared( 'deactivation_intent_subscriber', Subscriber::class )
50-
->addArgument( $this->getContainer()->get( 'deactivation_intent' ) )
51-
->addTag( 'admin_subscriber' );
50+
->addArgument( $this->getContainer()->get( 'deactivation_intent' ) );
5251
$this->getContainer()->addShared( 'hummingbird_subscriber', Hummingbird::class )
53-
->addArgument( $options )
54-
->addTag( 'admin_subscriber' );
52+
->addArgument( $options );
5553
$this->getContainer()->addShared( 'actionscheduler_admin_subscriber', ActionSchedulerSubscriber::class );
5654
$this->getContainer()->addShared( 'post_edit_options_subscriber', PostEditOptionsSubscriber::class )
5755
->addArgument( $options )
58-
->addArgument( $this->getContainer()->get( 'template_path' ) . '/metaboxes' )
59-
->addTag( 'admin_subscriber' );
56+
->addArgument( $this->getContainer()->get( 'template_path' ) . '/metaboxes' );
6057
}
6158
}

inc/Engine/Admin/Settings/ServiceProvider.php

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,6 @@ public function register(): void {
5151
->addArgument( $this->getContainer()->get( 'template_path' ) )
5252
->addArgument( $this->getContainer()->get( 'options' ) );
5353
$this->getContainer()->addShared( 'settings_page_subscriber', Subscriber::class )
54-
->addArgument( $this->getContainer()->get( 'settings_page' ) )
55-
->addTag( 'admin_subscriber' );
54+
->addArgument( $this->getContainer()->get( 'settings_page' ) );
5655
}
5756
}

inc/Engine/CDN/RocketCDN/ServiceProvider.php

Lines changed: 4 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -48,25 +48,21 @@ public function register(): void {
4848
// RocketCDN Data manager subscriber.
4949
$this->getContainer()->addShared( 'rocketcdn_data_manager_subscriber', DataManagerSubscriber::class )
5050
->addArgument( $this->getContainer()->get( 'rocketcdn_api_client' ) )
51-
->addArgument( $this->getContainer()->get( 'rocketcdn_options_manager' ) )
52-
->addTag( 'admin_subscriber' );
51+
->addArgument( $this->getContainer()->get( 'rocketcdn_options_manager' ) );
5352
// RocketCDN REST API Subscriber.
5453
$this->getContainer()->addShared( 'rocketcdn_rest_subscriber', RESTSubscriber::class )
5554
->addArgument( $this->getContainer()->get( 'rocketcdn_options_manager' ) )
56-
->addArgument( $options )
57-
->addTag( 'common_subscriber' );
55+
->addArgument( $options );
5856
// RocketCDN Notices Subscriber.
5957
$this->getContainer()->addShared( 'rocketcdn_notices_subscriber', NoticesSubscriber::class )
6058
->addArgument( $this->getContainer()->get( 'rocketcdn_api_client' ) )
6159
->addArgument( $this->getContainer()->get( 'beacon' ) )
62-
->addArgument( __DIR__ . '/views' )
63-
->addTag( 'admin_subscriber' );
60+
->addArgument( __DIR__ . '/views' );
6461
// RocketCDN settings page subscriber.
6562
$this->getContainer()->addShared( 'rocketcdn_admin_subscriber', AdminPageSubscriber::class )
6663
->addArgument( $this->getContainer()->get( 'rocketcdn_api_client' ) )
6764
->addArgument( $options )
6865
->addArgument( $this->getContainer()->get( 'beacon' ) )
69-
->addArgument( __DIR__ . '/views' )
70-
->addTag( 'admin_subscriber' );
66+
->addArgument( __DIR__ . '/views' );
7167
}
7268
}

inc/Engine/CDN/ServiceProvider.php

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -42,9 +42,7 @@ public function register(): void {
4242
->addArgument( $options );
4343
$this->getContainer()->addShared( 'cdn_subscriber', Subscriber::class )
4444
->addArgument( $options )
45-
->addArgument( $this->getContainer()->get( 'cdn' ) )
46-
->addTag( 'common_subscriber' );
47-
$this->getContainer()->addShared( 'cdn_admin_subscriber', AdminSubscriber::class )
48-
->addTag( 'admin_subscriber' );
45+
->addArgument( $this->getContainer()->get( 'cdn' ) );
46+
$this->getContainer()->addShared( 'cdn_admin_subscriber', AdminSubscriber::class );
4947
}
5048
}

inc/Engine/Cache/ServiceProvider.php

Lines changed: 4 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -62,22 +62,18 @@ public function register(): void {
6262
->addArgument( $cache_query );
6363
$this->getContainer()->addShared( 'purge_actions_subscriber', PurgeActionsSubscriber::class )
6464
->addArgument( $this->getContainer()->get( 'options' ) )
65-
->addArgument( $this->getContainer()->get( 'purge' ) )
66-
->addTag( 'common_subscriber' );
65+
->addArgument( $this->getContainer()->get( 'purge' ) );
6766
$this->getContainer()->addShared( 'admin_cache_subscriber', AdminSubscriber::class )
6867
->addArgument( $this->getContainer()->get( 'advanced_cache' ) )
69-
->addArgument( $this->getContainer()->get( 'wp_cache' ) )
70-
->addTag( 'admin_subscriber' );
68+
->addArgument( $this->getContainer()->get( 'wp_cache' ) );
7169

7270
$this->getContainer()->add( 'expired_cache_purge', PurgeExpiredCache::class )
7371
->addArgument( rocket_get_constant( 'WP_ROCKET_CACHE_PATH' ) );
7472
$this->getContainer()->addShared( 'expired_cache_purge_subscriber', Subscriber::class )
7573
->addArgument( $this->getContainer()->get( 'options' ) )
76-
->addArgument( $this->getContainer()->get( 'expired_cache_purge' ) )
77-
->addTag( 'common_subscriber' );
74+
->addArgument( $this->getContainer()->get( 'expired_cache_purge' ) );
7875
$this->getContainer()->add( 'cache_config', ConfigSubscriber::class )
7976
->addArgument( $this->getContainer()->get( 'options' ) )
80-
->addArgument( $this->getContainer()->get( 'options_api' ) )
81-
->addTag( 'common_subscriber' );
77+
->addArgument( $this->getContainer()->get( 'options_api' ) );
8278
}
8379
}

0 commit comments

Comments
 (0)