Added is_automattician check for Atomic sites (#34798)

* Added is_automattician alternative for Atomic sites * changelog * Tests added * Ver bump * Lock maintenance --------- Co-authored-by: Brandon Kraft <[email protected]>
Automattic · Apr 24, 2024 · 42b91bd · 42b91bd
1 parent 8723df5
commit 42b91bd
Show file tree

Hide file tree

Showing 35 changed files with 499 additions and 1,112 deletions.
diff --git a/projects/packages/status/changelog/add-is_automattician-check-for-Atomic-sites b/projects/packages/status/changelog/add-is_automattician-check-for-Atomic-sites
@@ -0,0 +1,4 @@
+Significance: minor
+Type: added
+
+Added is_automattician_feature_flags_only on Jetpack sites as an alternative to is_automattician present on simple sites.
diff --git a/projects/packages/status/src/class-visitor.php b/projects/packages/status/src/class-visitor.php
@@ -40,4 +40,17 @@ public function get_ip( $check_all_headers = false ) {
 
 		return ! empty( $_SERVER['REMOTE_ADDR'] ) ? filter_var( wp_unslash( $_SERVER['REMOTE_ADDR'] ) ) : '';
 	}
+
+	/**
+	 * Simple gate check for a11n feature testing purposes using AT_PROXIED_REQUEST constant.
+	 * IMPORTANT: Only use it for internal feature test purposes, not authorization.
+	 *
+	 * The goal of this function is to help us gate features by using a similar function name
+	 * we find on simple sites: is_automattician().
+	 *
+	 * @return bool True if the current request is PROXIED, false otherwise.
+	 */
+	public function is_automattician_feature_flags_only() {
+		return ( defined( 'AT_PROXIED_REQUEST' ) && AT_PROXIED_REQUEST );
+	}
 }
diff --git a/projects/packages/status/tests/php/test-visitor.php b/projects/packages/status/tests/php/test-visitor.php
@@ -208,4 +208,17 @@ public function get_ip_data_provider() {
 			),
 		);
 	}
+
+	/**
+	 * Tests is_automattician_feature_flags_only method.
+	 */
+	public function test_is_automattician_feature_flags_only() {
+		$is_a11n = $this->visitor_obj->is_automattician_feature_flags_only();
+		$this->assertFalse( $is_a11n );
+
+		define( 'AT_PROXIED_REQUEST', true );
+
+		$is_a11n = $this->visitor_obj->is_automattician_feature_flags_only();
+		$this->assertTrue( $is_a11n );
+	}
 }
diff --git a/...gins/automattic-for-agencies-client/changelog/add-is_automattician-check-for-Atomic-sites b/...gins/automattic-for-agencies-client/changelog/add-is_automattician-check-for-Atomic-sites
@@ -0,0 +1,5 @@
+Significance: patch
+Type: changed
+Comment: Updated composer.lock.
+
+
diff --git a/projects/plugins/automattic-for-agencies-client/composer.lock b/projects/plugins/automattic-for-agencies-client/composer.lock
diff --git a/projects/plugins/backup/changelog/add-is_automattician-check-for-Atomic-sites b/projects/plugins/backup/changelog/add-is_automattician-check-for-Atomic-sites
@@ -0,0 +1,5 @@
+Significance: patch
+Type: changed
+Comment: Updated composer.lock.
+
+
diff --git a/projects/plugins/backup/changelog/add-is_automattician-check-for-Atomic-sites#2 b/projects/plugins/backup/changelog/add-is_automattician-check-for-Atomic-sites#2
@@ -0,0 +1,5 @@
+Significance: patch
+Type: changed
+Comment: Updated composer.lock.
+
+