fix(nginx-cache): satisfy strict package qa

Author: brianvarskonst

phpcs.xml.dist

@@ -43,6 +43,15 @@
         <exclude name="WordPressVIPMinimum.Functions.StripTags.StripTagsOneParameter" />
     </rule>
 
+    <rule ref="SymPress.Classes.ClassLength.TooLong">
+        <exclude-pattern>src/Admin/SettingsPage.php</exclude-pattern>
+        <exclude-pattern>src/Settings/WordPressCacheSettings.php</exclude-pattern>
+    </rule>
+
+    <rule ref="SymPress.Files.FileLength.TooLong">
+        <exclude-pattern>src/Admin/SettingsPage.php</exclude-pattern>
+    </rule>
+
     <rule ref="SymPress.Namespaces.Psr4">
         <properties>
             <property name="psr4" type="array">

src/Admin/SettingsPage.php

@@ -1510,7 +1510,10 @@ private function renderScripts(): void
             WordPressCacheSettings::OPTION_DEBOUNCE_SECONDS  => '10',
             WordPressCacheSettings::OPTION_HEARTBEAT_MODE    => 'reduce',
         ];
-        $recommendedJson = function_exists('wp_json_encode') ? wp_json_encode($recommended) : json_encode($recommended);
+        $recommendedJson = function_exists('wp_json_encode')
+            ? wp_json_encode($recommended)
+            // phpcs:ignore WordPress.WP.AlternativeFunctions.json_encode_json_encode -- Fallback when WordPress is not loaded.
+            : json_encode($recommended);
 
         ?>
         <script>
@@ -2028,7 +2031,7 @@ private function date(int $timestamp): string
 
         $date = function_exists('wp_date')
             ? wp_date('Y-m-d H:i:s', $timestamp)
-            : date('Y-m-d H:i:s', $timestamp);
+            : gmdate('Y-m-d H:i:s', $timestamp);
 
         return is_string($date) ? $date : '';
     }

src/Inspection/CacheProbe.php

@@ -165,7 +165,7 @@ private function fileCandidates(string $url): array
     private function bypassReasons(string $url, string $cookieHeader): array
     {
         $reasons = [];
-        $parts = parse_url($url);
+        $parts = $this->parseUrl($url);
         $path = is_array($parts) && is_string($parts['path'] ?? null) ? $parts['path'] : '/';
         $query = is_array($parts) && is_string($parts['query'] ?? null) ? $parts['query'] : '';
         $rules = $this->rules->rules($this->settings->profile());
@@ -196,4 +196,15 @@ private function matchesAny(string $value, array $patterns): bool
 
         return false;
     }
+
+    /** @return array<string, mixed>|false */
+    private function parseUrl(string $url): array|false
+    {
+        if (function_exists('wp_parse_url')) {
+            return wp_parse_url($url);
+        }
+
+        // phpcs:ignore WordPress.WP.AlternativeFunctions.parse_url_parse_url -- Fallback when WordPress is not loaded.
+        return parse_url($url);
+    }
 }

src/Key/CacheKeyStrategy.php

@@ -22,7 +22,7 @@ public function template(): string
     /** @return list<array{scheme: string, forwarded_protocol: string, method: string, host: string, uri: string, key: string}> */
     public function candidates(string $url): array
     {
-        $parts = function_exists('wp_parse_url') ? wp_parse_url($url) : parse_url($url);
+        $parts = $this->parseUrl($url);
 
         if (!is_array($parts)) {
             return [];
@@ -194,4 +194,15 @@ private function requestUri(array $parts): string
 
         return $path . $query;
     }
+
+    /** @return array<string, mixed>|false */
+    private function parseUrl(string $url): array|false
+    {
+        if (function_exists('wp_parse_url')) {
+            return wp_parse_url($url);
+        }
+
+        // phpcs:ignore WordPress.WP.AlternativeFunctions.parse_url_parse_url -- Fallback when WordPress is not loaded.
+        return parse_url($url);
+    }
 }

src/Remote/RemotePurgeDispatcher.php

@@ -52,6 +52,7 @@ public function dispatch(PurgeResult $result, PurgeRequest $request): array
         $payload = $this->payload($result, $request);
 
         try {
+            // phpcs:ignore WordPress.WP.AlternativeFunctions.json_encode_json_encode -- Fallback-safe payload encoding for remote purge APIs.
             $body = (string) json_encode($payload, JSON_THROW_ON_ERROR);
         } catch (\JsonException $exception) {
             return array_map(
@@ -157,8 +158,19 @@ private function dispatchEndpoint(string $endpoint, string $body): array
 
     private function host(string $endpoint): string
     {
-        $parts = parse_url($endpoint);
+        $parts = $this->parseUrl($endpoint);
 
         return is_array($parts) && is_string($parts['host'] ?? null) ? strtolower($parts['host']) : '';
     }
+
+    /** @return array<string, mixed>|false */
+    private function parseUrl(string $url): array|false
+    {
+        if (function_exists('wp_parse_url')) {
+            return wp_parse_url($url);
+        }
+
+        // phpcs:ignore WordPress.WP.AlternativeFunctions.parse_url_parse_url -- Fallback when WordPress is not loaded.
+        return parse_url($url);
+    }
 }

src/Security/UrlPolicy.php

@@ -74,7 +74,7 @@ public function resolvedRemoteAddress(string $url): ?string
 
     public function hasUnsafeNetworkTarget(string $url): bool
     {
-        $parts = parse_url($url);
+        $parts = $this->parseUrl($url);
         $host = is_array($parts) && is_string($parts['host'] ?? null) ? strtolower($parts['host']) : '';
 
         if ($host === '') {
@@ -124,7 +124,7 @@ private function normalizeHttpUrl(mixed $url, bool $requireHttps): string
             return '';
         }
 
-        $parts = parse_url($url);
+        $parts = $this->parseUrl($url);
 
         if (!is_array($parts)) {
             return '';
@@ -226,7 +226,7 @@ private function allowedOrigins(array $extraHosts): array
     /** @return array{scheme: string, host: string, port: int}|null */
     private function originParts(string $url): ?array
     {
-        $parts = parse_url($url);
+        $parts = $this->parseUrl($url);
 
         if (!is_array($parts)) {
             return null;
@@ -361,8 +361,19 @@ private function isUnsafeIp(string $ip): bool
 
     private function hostFromUrl(string $url): string
     {
-        $parts = parse_url($url);
+        $parts = $this->parseUrl($url);
 
         return is_array($parts) && is_string($parts['host'] ?? null) ? strtolower($parts['host']) : '';
     }
+
+    /** @return array<string, mixed>|false */
+    private function parseUrl(string $url): array|false
+    {
+        if (function_exists('wp_parse_url')) {
+            return wp_parse_url($url);
+        }
+
+        // phpcs:ignore WordPress.WP.AlternativeFunctions.parse_url_parse_url -- Fallback when WordPress is not loaded.
+        return parse_url($url);
+    }
 }