← All changes
|
includes/modules/BrowserCache/BrowserCacheModule.php
+23
-15
1.1.6
→
1.3.2
View file →
| @@ -28,11 +28,11 @@ | ||
| 28 | 28 | public const VERSION = '1.0.0'; |
| 29 | 29 | |
| 30 | 30 | public function ui_metadata(): array { |
| 31 | 31 | return array( |
| 32 | - 'label' => 'Browser Cache', | |
| 32 | + 'label' => __( 'Browser Cache', 'xspeed' ), | |
| 33 | 33 | 'icon' => 'Clock', |
| 34 | - 'description' => 'Tell browsers (and intermediate CDNs) how long to cache static assets and HTML.', | |
| 34 | + 'description' => __( 'Tell browsers (and intermediate CDNs) how long to cache static assets and HTML.', 'xspeed' ), | |
| 35 | 35 | ); |
| 36 | 36 | } |
| 37 | 37 | |
| 38 | 38 | public function settings_schema(): array { |
| @@ -39,10 +39,10 @@ | ||
| 39 | 39 | return array( |
| 40 | 40 | 'enabled' => array( |
| 41 | 41 | 'type' => 'bool', |
| 42 | 42 | 'default' => false, |
| 43 | - 'label' => 'Enable browser cache headers', | |
| 44 | - 'description' => 'On Apache/LiteSpeed this writes Cache-Control + Expires rules into .htaccess. On nginx it just stores the settings — you paste the snippet into your server block manually.', | |
| 43 | + 'label' => __( 'Enable browser cache headers', 'xspeed' ), | |
| 44 | + 'description' => __( 'On Apache/LiteSpeed this writes Cache-Control + Expires rules into .htaccess. On nginx it just stores the settings — you paste the snippet into your server block manually.', 'xspeed' ), | |
| 45 | 45 | ), |
| 46 | 46 | 'asset_ttl' => array( |
| 47 | 47 | 'type' => 'int', |
| 48 | 48 | 'default' => Browser_Cache::DEFAULT_ASSET_TTL, |
| @@ -47,11 +47,11 @@ | ||
| 47 | 47 | 'type' => 'int', |
| 48 | 48 | 'default' => Browser_Cache::DEFAULT_ASSET_TTL, |
| 49 | 49 | 'min' => 0, |
| 50 | 50 | 'max' => 31536000, |
| 51 | - 'label' => 'Static asset TTL (seconds)', | |
| 51 | + 'label' => __( 'Static asset TTL (seconds)', 'xspeed' ), | |
| 52 | 52 | 'unit' => 'seconds', |
| 53 | - 'description' => 'Cache lifetime for CSS, JS, fonts, images. Defaults to 1 year + immutable (the industry-standard "fingerprinted assets never change" pattern).', | |
| 53 | + 'description' => __( 'Cache lifetime for CSS, JS, fonts, images. Defaults to 1 year + immutable (the industry-standard "fingerprinted assets never change" pattern).', 'xspeed' ), | |
| 54 | 54 | 'dependsOn' => array( 'field' => 'enabled' ), |
| 55 | 55 | ), |
| 56 | 56 | 'html_ttl' => array( |
| 57 | 57 | 'type' => 'int', |
| @@ -57,11 +57,11 @@ | ||
| 57 | 57 | 'type' => 'int', |
| 58 | 58 | 'default' => Browser_Cache::DEFAULT_HTML_TTL, |
| 59 | 59 | 'min' => 0, |
| 60 | 60 | 'max' => 31536000, |
| 61 | - 'label' => 'HTML TTL (seconds)', | |
| 61 | + 'label' => __( 'HTML TTL (seconds)', 'xspeed' ), | |
| 62 | 62 | 'unit' => 'seconds', |
| 63 | - 'description' => 'Cache lifetime for the HTML document itself. Keep short (default 1h) so post edits roll out same-day.', | |
| 63 | + 'description' => __( 'Cache lifetime for the HTML document itself. Keep short (default 1h) so post edits roll out same-day.', 'xspeed' ), | |
| 64 | 64 | 'dependsOn' => array( 'field' => 'enabled' ), |
| 65 | 65 | ), |
| 66 | 66 | ); |
| 67 | 67 | } |
| @@ -98,13 +98,19 @@ | ||
| 98 | 98 | if ( empty( $opts['enabled'] ) ) { |
| 99 | 99 | return array(); |
| 100 | 100 | } |
| 101 | 101 | |
| 102 | - // Probe whether the snippet has actually been pasted + reloaded. | |
| 103 | - // If the live HEAD shows Cache-Control: immutable on a known | |
| 104 | - // static asset, the user is done — suppress the notice. Avoids | |
| 105 | - // the false-alarm "do something" prompt we used to show forever. | |
| 106 | - if ( Browser_Cache::probe_headers_present() ) { | |
| 102 | + // Probe the live response for caching headers on a known static | |
| 103 | + // asset. Suppress the notice unless the probe PROVES nothing is | |
| 104 | + // being sent: | |
| 105 | + // | |
| 106 | + // true → headers present, from our snippet or from the host's own | |
| 107 | + // vhost — either way the feature's job is done (issue #329) | |
| 108 | + // null → the loopback never completed, so we know nothing; a | |
| 109 | + // broken probe is not evidence of a broken server and must | |
| 110 | + // not raise a warning the operator cannot act on (issue #18) | |
| 111 | + // false → proven absent, fall through and show the notice | |
| 112 | + if ( false !== Browser_Cache::probe_headers_present() ) { | |
| 107 | 113 | return array(); |
| 108 | 114 | } |
| 109 | 115 | |
| 110 | 116 | // nginx hosts can't auto-write Cache-Control / Expires headers, and the |
| @@ -140,8 +146,9 @@ | ||
| 140 | 146 | array( |
| 141 | 147 | 'name' => 'xspeed browser-cache', |
| 142 | 148 | 'callback' => array( $this, 'cli_handler' ), |
| 143 | 149 | 'shortdesc' => 'Print the Apache or nginx browser-cache snippet.', |
| 150 | + 'ai_hint' => 'Get the server config snippet that sets browser cache-control headers for static assets. Use when PageSpeed reports "serve static assets with an efficient cache policy", or when the user needs the rules to paste into Apache/nginx.', | |
| 144 | 151 | 'synopsis' => array( |
| 145 | 152 | array( |
| 146 | 153 | 'type' => 'positional', |
| 147 | 154 | 'name' => 'flavor', |
| @@ -165,10 +172,11 @@ | ||
| 165 | 172 | } |
| 166 | 173 | } |
| 167 | 174 | |
| 168 | 175 | /** |
| 169 | - * Cache-Control / Expires directives for the unified nginx | |
| 170 | - * server-block snippet. Null when the module is disabled — no | |
| 176 | + * Cache-Control directives for the unified nginx server-block | |
| 177 | + * snippet. nginx sends no Expires — `expires off;` pins the block so | |
| 178 | + * only the explicit `add_header` speaks (#259). Null when the module is disabled — no | |
| 171 | 179 | * directives to install. |
| 172 | 180 | */ |
| 173 | 181 | public function nginx_directives(): ?string { |
| 174 | 182 | $opts = $this->get_settings(); |