← All changes | includes/modules/Preloader/PreloaderModule.php +83 -16 1.0.7 → 1.3.2 View file →

		@@ -28,15 +28,16 @@
28	28	public const VERSION = '1.0.0';
29	29
30	30	public function ui_metadata(): array {
31	31	return array(
32		- 'label' => 'Preloader',
	32	+ 'label' => __( 'Preloader', 'xspeed' ),
	33	+ 'tab_label' => __( 'Crawl Now', 'xspeed' ), // its own tab on the Preloader page
33	34	'icon' => 'Wand2',
34		- 'description' => 'Crawl the sitemap to warm cache so visitors never hit a cold MISS.',
	35	+ 'description' => __( 'Crawl the sitemap to warm cache so visitors never hit a cold MISS.', 'xspeed' ),
35	36	// Custom panel wraps the schema-driven settings with a
36	37	// Start/Stop control surface + a live status readout
37	38	// (queue depth, last URL, recent errors).
38		- 'custom_panel' => 'PreloaderPanel',
	39	+ 'custom_panel' => 'PreloaderHost',
39	40	);
40	41	}
41	42
42	43	public function settings_schema(): array {
		@@ -43,10 +44,10 @@
43	44	return array(
44	45	'enabled' => array(
45	46	'type' => 'bool',
46	47	'default' => false,
47		- 'label' => 'Enable Preloader',
48		- 'description' => 'When on, xSpeed crawls the sitemap on the schedule below and warms the page cache.',
	48	+ 'label' => __( 'Enable Preloader', 'xspeed' ),
	49	+ 'description' => __( 'When on, xSpeed crawls the sitemap on the schedule below and warms the page cache.', 'xspeed' ),
49	50	),
50	51	'schedule' => array(
51	52	'type' => 'enum',
52	53	'default' => 'manual',
		@@ -56,10 +57,11 @@
56	57	'hourly' => 'Hourly',
57	58	'daily' => 'Daily',
58	59	'weekly' => 'Weekly',
59	60	),
60		- 'label' => 'Schedule',
61		- 'description' => 'How often to start a fresh crawl. Manual means you trigger it from the dashboard.',
	61	+ 'label' => __( 'Schedule', 'xspeed' ),
	62	+ 'description' => __( 'How often to start a fresh crawl. Manual means you trigger it from the dashboard.', 'xspeed' ),
	63	+ 'dependsOn' => array( 'field' => 'enabled' ),
62	64	),
63	65	'batch_size' => array(
64	66	'type' => 'int',
65	67	'default' => 5,
		@@ -64,28 +66,32 @@
64	66	'type' => 'int',
65	67	'default' => 5,
66	68	'min' => 1,
67	69	'max' => 50,
68		- 'label' => 'Batch Size',
69		- 'description' => 'URLs warmed per cron tick. Higher = faster crawl, more load on the origin.',
	70	+ 'label' => __( 'Batch Size', 'xspeed' ),
	71	+ 'description' => __( 'URLs warmed per cron tick. Higher = faster crawl, more load on the origin.', 'xspeed' ),
	72	+ 'dependsOn' => array( 'field' => 'enabled' ),
70	73	),
71	74	'sitemap_url' => array(
72	75	'type' => 'string',
73	76	'default' => '',
74		- 'label' => 'Sitemap URL (optional)',
75		- 'description' => 'Override the auto-detected WordPress core sitemap (/wp-sitemap.xml). Leave blank for default.',
	77	+ 'label' => __( 'Sitemap URL (optional)', 'xspeed' ),
	78	+ 'description' => __( 'Override the auto-detected WordPress core sitemap (/wp-sitemap.xml). Leave blank for default.', 'xspeed' ),
	79	+ 'dependsOn' => array( 'field' => 'enabled' ),
76	80	),
77	81	'warm_on_publish' => array(
78	82	'type' => 'bool',
79	83	'default' => true,
80		- 'label' => 'Warm new content immediately',
81		- 'description' => 'When a post or page is published, fetch it once so the first visitor sees a cache HIT, not a cold MISS.',
	84	+ 'label' => __( 'Warm new content immediately', 'xspeed' ),
	85	+ 'description' => __( 'When a post or page is published, fetch it once so the first visitor sees a cache HIT, not a cold MISS.', 'xspeed' ),
	86	+ 'dependsOn' => array( 'field' => 'enabled' ),
82	87	),
83	88	'warm_on_comment' => array(
84	89	'type' => 'bool',
85	90	'default' => false,
86		- 'label' => 'Re-warm after comments',
87		- 'description' => 'Re-warm a page after a comment is posted (Cache purges the page on comment; this fetches it back into cache).',
	91	+ 'label' => __( 'Re-warm after comments', 'xspeed' ),
	92	+ 'description' => __( 'Re-warm a page after a comment is posted (Cache purges the page on comment; this fetches it back into cache).', 'xspeed' ),
	93	+ 'dependsOn' => array( 'field' => 'enabled' ),
88	94	),
89	95	);
90	96	}
91	97
		@@ -120,8 +126,9 @@
120	126	array(
121	127	'name' => 'xspeed preloader',
122	128	'callback' => array( $this, 'cli_handler' ),
123	129	'shortdesc' => 'Drive the cache preloader (start | stop | status).',
	130	+ 'ai_hint' => 'Cache warming: crawl the site so visitors hit a warm cache instead of paying for the first render. Use after a full purge, or when the first visitor to each page reports a slow load.',
124	131	'synopsis' => array(
125	132	array(
126	133	'type' => 'positional',
127	134	'name' => 'action',
		@@ -133,8 +140,27 @@
133	140	);
134	141	}
135	142
136	143	public function boot(): void {
	144	+ /*
	145	+ * Deferred to `init`. This module reads its own settings to decide
	146	+ * what to hook, and reading settings builds settings_schema(), whose
	147	+ * labels are declared through __(). boot() runs on `plugins_loaded`,
	148	+ * before `after_setup_theme` — the point WordPress 6.7+ treats as the
	149	+ * earliest safe moment to translate — so doing that here fires
	150	+ * _load_textdomain_just_in_time on every request AND resolves the
	151	+ * labels against a domain that is not loaded yet.
	152	+ *
	153	+ * Everything below hooks actions that fire after `init`, so running
	154	+ * one hook later is equivalent.
	155	+ */
	156	+ add_action( 'init', array( $this, 'boot_on_init' ) );
	157	+ }
	158	+
	159	+ /**
	160	+ * The real boot body — see boot() for why it runs on `init`.
	161	+ */
	162	+ public function boot_on_init(): void {
137	163	add_action( Preloader::CRON_HOOK, array( Preloader::class, 'tick' ) );
138	164	add_action( 'xspeed_preloader_recurring', array( Preloader::class, 'recurring_kickoff' ) );
139	165
140	166	// Apply schedule changes immediately whenever this module's
		@@ -264,8 +290,30 @@
264	290	\WP_CLI::error( 'Preloader is disabled. Enable it via wp xspeed preloader set --enabled=1 first.' );
265	291	return;
266	292	}
267	293	$state = Preloader::start();
	294	+ // Don't print a green Success over a crawl that queued
	295	+ // nothing — that exit-0 was the whole complaint in #142.
	296	+ $sitemap_error = (string) ( $state['sitemap_error'] ?? '' );
	297	+ if ( 0 === (int) $state['total'] ) {
	298	+ \WP_CLI::error(
	299	+ '' !== $sitemap_error
	300	+ ? sprintf( 'Queued 0 URLs. %s', $sitemap_error )
	301	+ : 'Queued 0 URLs — nothing to warm. Check the sitemap URL and the cache exclusion rules.'
	302	+ );
	303	+ return;
	304	+ }
	305	+ if ( 'fallback' === ( $state['source'] ?? '' ) ) {
	306	+ \WP_CLI::warning( $sitemap_error );
	307	+ \WP_CLI::success(
	308	+ sprintf(
	309	+ 'Queued %d URL%s from the site content instead of the sitemap.',
	310	+ $state['total'],
	311	+ 1 === $state['total'] ? '' : 's'
	312	+ )
	313	+ );
	314	+ return;
	315	+ }
268	316	\WP_CLI::success( sprintf( 'Queued %d URL%s.', $state['total'], 1 === $state['total'] ? '' : 's' ) );
269	317	return;
270	318
271	319	case 'stop':
		@@ -282,9 +330,28 @@
282	330	}
283	331	if ( ! empty( $state['errors'] ) ) {
284	332	\WP_CLI::log( 'Errors : ' . count( $state['errors'] ) );
285	333	foreach ( array_slice( $state['errors'], -5 ) as $e ) {
286		- \WP_CLI::log( ' ' . $e['url'] . ' — ' . $e['error'] );
	334	+ // Tolerate a bare string as well as the {url, error}
	335	+ // shape. A string entry fataled this command outright
	336	+ // ("Cannot access offset of type string on string"),
	337	+ // which also took MCP's get_preloader_status down with
	338	+ // it — an agent asking why a preload failed got a type
	339	+ // error instead of the reason. The writer is fixed, but
	340	+ // `status` is a diagnostic: it should survive whatever
	341	+ // it is handed rather than die reporting on it. (QA F1)
	342	+ if ( is_array( $e ) ) {
	343	+ $url = isset( $e['url'] ) ? (string) $e['url'] : '';
	344	+ $msg = isset( $e['error'] ) ? (string) $e['error'] : '';
	345	+ // The sitemap message already names the URL, so
	346	+ // prefixing it would print the URL twice on one line.
	347	+ $line = ( '' !== $url && false === strpos( $msg, $url ) )
	348	+ ? $url . ' — ' . $msg
	349	+ : $msg;
	350	+ } else {
	351	+ $line = (string) $e;
	352	+ }
	353	+ \WP_CLI::log( ' ' . $line );
287	354	}
288	355	}
289	356	return;
290	357