feat: Added context check fns

Author: seebeen

phpcs.xml

@@ -14,5 +14,7 @@
     <severity>0</severity>
   </rule>
 
+  <file>config/</file>
   <file>src/</file>
+  <file>wc-sync-core.php</file>
 </ruleset>

src/Hook_Context.php src/Core/Context.phpsrc/Hook_Context.php renamed to src/Core/Context.php

@@ -6,16 +6,14 @@
  * @subpackage Dependency Injection
  */
 
-namespace XWP\DI;
-
 use Automattic\Jetpack\Constants;
 
 /**
  * Determines execution context.
  *
  * @since 1.0.0
  */
-final class Hook_Context {
+final class XWP_Context {
     /**
      * Frontend context.
      */
@@ -120,6 +118,17 @@ public static function admin(): bool {
         return \is_admin() && ! self::ajax();
     }
 
+    /**
+     * Check if the request is an admin request for a specific page.
+     *
+     * @param  string      $page The page to check.
+     * @param  string|null $type The post type to check.
+     * @return bool
+     */
+    public static function admin_page( string $page, ?string $type = null ): bool {
+        return self::admin() && ( $GLOBALS['pagenow'] ?? '' ) === $page && ( ! $type || ( $GLOBALS['typenow'] ?? '' ) === $type );
+    }
+
     /**
      * Check if the request is an AJAX request.
      *
@@ -129,6 +138,16 @@ public static function ajax(): bool {
         return Constants::is_true( 'DOING_AJAX' );
     }
 
+    /**
+     * Check if the request is an AJAX request for a specific action.
+     *
+     * @param  string $action The action to check.
+     * @return bool
+     */
+    public static function ajax_action( string $action ): bool {
+        return self::ajax() && \xwp_fetch_req_var( 'action', '' ) === $action;
+    }
+
     /**
      * Check if the request is a cron request.
      *

src/Decorators/Handler.php

@@ -284,6 +284,10 @@ public function is_lazy(): bool {
     }
 
     public function is_hookable(): bool {
-        return null === $this->hookable && $this->check_context() || $this->hookable;
+        if ( ! $this->check_context() ) {
+            return false;
+        }
+
+        return $this->hookable ?? true;
     }
 }

src/Decorators/Hook.php

@@ -12,8 +12,8 @@
 use DI\Container;
 use ReflectionClass;
 use ReflectionMethod;
-use XWP\DI\Hook_Context;
 use XWP\DI\Interfaces\Can_Hook;
+use XWP_Context;
 
 /**
  * Base hook from which the action and filter decorators inherit.
@@ -111,7 +111,7 @@ public function can_load(): bool {
      * @return bool
      */
     public function check_context(): bool {
-        return Hook_Context::validate( $this->context );
+        return XWP_Context::validate( $this->context );
     }
 
     /**