diff --git a/projects/packages/jitm/changelog/update-jitm-approved-screens-to-const b/projects/packages/jitm/changelog/update-jitm-approved-screens-to-const new file mode 100644 index 0000000000000..f1bc3350f3d8c --- /dev/null +++ b/projects/packages/jitm/changelog/update-jitm-approved-screens-to-const @@ -0,0 +1,4 @@ +Significance: patch +Type: changed + +Refactor approved screens in preparation for https://github.com/Automattic/jetpack/pull/41252 diff --git a/projects/packages/jitm/src/class-jitm.php b/projects/packages/jitm/src/class-jitm.php index 0682b268c8a28..f8f41cd5f8eb3 100644 --- a/projects/packages/jitm/src/class-jitm.php +++ b/projects/packages/jitm/src/class-jitm.php @@ -22,6 +22,18 @@ class JITM { const PACKAGE_VERSION = '4.0.6'; + /** + * List of screen IDs where JITMs are allowed to display. + * + * @var string[] + */ + const APPROVED_SCREEN_IDS = array( + 'jetpack', + 'woo', + 'shop', + 'product', + ); + /** * The configuration method that is called from the jetpack-config package. */ @@ -156,7 +168,10 @@ public function is_a8c_admin_page() { return ( $current_screen && $current_screen->id - && (bool) preg_match( '/jetpack|woo|shop|product/', $current_screen->id ) + && (bool) preg_match( + '/' . implode( '|', self::APPROVED_SCREEN_IDS ) . '/', + $current_screen->id + ) ); } @@ -226,7 +241,7 @@ public function ajax_message() { return; } - // Only add this to Jetpack or Woo admin pages. + // Only add this to specifically allowed pages. if ( ! $this->is_a8c_admin_page() ) { return; }