Skip to content

Commit bdcf5bb

Browse files
committed
[APP-1107] add check for is_active
1 parent 96beed1 commit bdcf5bb

File tree

3 files changed

+9
-11
lines changed

3 files changed

+9
-11
lines changed

modules/analytics/module.php

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

55
use EA11y\Classes\Module_Base;
66
use EA11y\Modules\Analytics\Database\Analytics_Table;
7+
use EA11y\Modules\Settings\Classes\Settings;
78
use EA11y\Modules\Settings\Module as SettingsModule;
89

910
if ( ! defined( 'ABSPATH' ) ) {
@@ -39,11 +40,13 @@ public static function routes_list() : array {
3940
];
4041
}
4142

42-
public function __construct() {
43-
if ( ! SettingsModule::is_analytics_enabled() ) {
44-
return;
45-
}
43+
public static function is_active(): bool {
44+
$plan_data = Settings::get( Settings::PLAN_DATA );
45+
$is_analytics_enabled = Settings::get( Settings::ANALYTICS_SETTINGS );
46+
return 'Free Trial' !== $plan_data->plan->name && $is_analytics_enabled;
47+
}
4648

49+
public function __construct() {
4750
$this->register_components();
4851
$this->register_routes();
4952

modules/settings/module.php

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -442,12 +442,6 @@ public static function get_plan_data_refresh_transient(): bool {
442442
return get_transient( Settings::PLAN_DATA . '_refresh' );
443443
}
444444

445-
public static function is_analytics_enabled(): bool {
446-
$plan_data = Settings::get( Settings::PLAN_DATA );
447-
$is_analytics_enabled = Settings::get( Settings::ANALYTICS_SETTINGS );
448-
return 'Free Trial' !== $plan_data->plan->name && $is_analytics_enabled;
449-
}
450-
451445
/**
452446
* Module constructor.
453447
*/

modules/widget/module.php

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@
55
use EA11y\Classes\Module_Base;
66
use EA11y\Modules\Connect\Module as Connect;
77
use EA11y\Modules\Settings\Module as SettingsModule;
8+
use EA11y\Modules\Analytics\Module as AnalyticsModule;
89
use EA11y\Modules\Settings\Classes\Settings;
910
use Exception;
1011

@@ -43,7 +44,7 @@ public function enqueue_accessibility_widget() : void {
4344
true
4445
);
4546

46-
$is_analytics_enabled = SettingsModule::is_analytics_enabled();
47+
$is_analytics_enabled = AnalyticsModule::is_active();
4748

4849
wp_localize_script(
4950
'ea11y-widget',

0 commit comments

Comments
 (0)