← All changes | includes/modules/Database/DatabaseModule.php +97 -5 1.3.4 → 1.3.7 View file →

		@@ -28,10 +28,11 @@
28	28	public function ui_metadata(): array {
29	29	return array(
30	30	'label' => __( 'Database', 'xspeed' ),
31	31	'icon' => 'Trash2',
32		- 'description' => __( 'Scan + clean WordPress bloat (revisions, spam, transients, orphan meta) and optimize tables.', 'xspeed' ),
	32	+ 'description' => __( 'Remove old revisions, spam and other leftovers, and optimize database tables.', 'xspeed' ),
33	33	'custom_panel' => 'DatabaseCleanerPanel',
	34	+ 'group' => 'tools',
34	35	);
35	36	}
36	37
37	38	public function settings_schema(): array {
		@@ -45,21 +46,102 @@
45	46	'hourly' => 'Hourly',
46	47	'daily' => 'Daily',
47	48	'weekly' => 'Weekly',
48	49	),
49		- 'label' => __( 'Auto-Cleanup Schedule', 'xspeed' ),
50		- 'description' => __( 'How often to run cleanup automatically. Manual means cleanup only runs when you press the button.', 'xspeed' ),
	50	+ 'label' => __( 'Automatic cleanup', 'xspeed' ),
	51	+ 'description' => __( 'How often xSpeed cleans the database on its own. Manual means it only cleans when you press the button.', 'xspeed' ),
51	52	),
52	53	'included_types' => array(
53	54	'type' => 'list',
54	55	'default' => array(),
55	56	'item_type' => 'string',
56		- 'label' => __( 'Auto-Cleanup Types', 'xspeed' ),
57		- 'description' => __( 'Which cleanup categories run on the schedule above. Leave empty to keep auto-cleanup disabled even if a schedule is set.', 'xspeed' ),
	57	+ 'label' => __( 'What to clean automatically', 'xspeed' ),
	58	+ 'description' => __( 'The items the scheduled cleanup removes. If you leave this empty, nothing is cleaned, even with a schedule set.', 'xspeed' ),
	59	+ 'dependsOn' => array(
	60	+ 'any' => array(
	61	+ array( 'field' => 'schedule', 'value' => 'hourly' ),
	62	+ array( 'field' => 'schedule', 'value' => 'daily' ),
	63	+ array( 'field' => 'schedule', 'value' => 'weekly' ),
	64	+ ),
	65	+ ),
58	66	),
	67	+ 'limit_revisions' => array(
	68	+ 'type' => 'bool',
	69	+ 'default' => false,
	70	+ 'label' => __( 'Limit post revisions', 'xspeed' ),
	71	+ 'description' => __( 'Cap how many revisions WordPress keeps for each post. Cleanup removes revisions that already exist; this stops them piling up again.', 'xspeed' ),
	72	+ ),
	73	+ 'revisions_to_keep' => array(
	74	+ 'type' => 'int',
	75	+ 'default' => 5,
	76	+ 'min' => 0,
	77	+ 'max' => 500,
	78	+ 'label' => __( 'Revisions to keep', 'xspeed' ),
	79	+ 'description' => __( '0 turns revisions off completely. Older revisions are trimmed the next time each post is saved; to remove the backlog now, clean Post Revisions above.', 'xspeed' ),
	80	+ 'dependsOn' => array( 'field' => 'limit_revisions' ),
	81	+ ),
59	82	);
60	83	}
61	84
	85	+ /**
	86	+ * WP_POST_REVISIONS as wp-config.php (or an earlier plugin) set it, or
	87	+ * null when nobody did. Read in boot(), on plugins_loaded: after that,
	88	+ * core defines it as `true` itself, so defined() stops telling the two
	89	+ * cases apart.
	90	+ *
	91	+ * Not the schema's `constants` pinning (#398): that tests defined() at
	92	+ * request time, when core's own default makes it always true.
	93	+ *
	94	+ * @var bool|int|null
	95	+ */
	96	+ private static $configured_revisions = null;
	97	+
	98	+ /**
	99	+ * @param int $num Revisions WordPress would keep; -1 means all.
	100	+ * @param \WP_Post $post
	101	+ * @return int
	102	+ */
	103	+ public static function filter_revisions_to_keep( $num, $post ) {
	104	+ // A wp-config.php value is the site owner's decision; leave it alone.
	105	+ if ( null !== self::$configured_revisions ) {
	106	+ return $num;
	107	+ }
	108	+ // Core already answered 0 for post types without revision support.
	109	+ if ( ! $post instanceof \WP_Post || ! post_type_supports( $post->post_type, 'revisions' ) ) {
	110	+ return $num;
	111	+ }
	112	+ $opts = Settings_Manager::get( self::SLUG );
	113	+ if ( empty( $opts['limit_revisions'] ) ) {
	114	+ return $num;
	115	+ }
	116	+ return max( 0, (int) ( $opts['revisions_to_keep'] ?? 5 ) );
	117	+ }
	118	+
	119	+ public function ui_notices(): array {
	120	+ if ( null === self::$configured_revisions ) {
	121	+ return array();
	122	+ }
	123	+ $value = self::$configured_revisions;
	124	+ if ( true === $value ) {
	125	+ $shown = 'true';
	126	+ } elseif ( false === $value ) {
	127	+ $shown = 'false';
	128	+ } else {
	129	+ $shown = (string) (int) $value;
	130	+ }
	131	+ return array(
	132	+ array(
	133	+ 'tone' => 'info',
	134	+ 'title' => __( 'The revision limit is set outside xSpeed', 'xspeed' ),
	135	+ 'body' => sprintf(
	136	+ /* translators: %s: value of WP_POST_REVISIONS, e.g. 5 or false. */
	137	+ __( 'WP_POST_REVISIONS is set to %s, usually in wp-config.php, so Limit Post Revisions below has no effect. Remove that definition to manage the limit here.', 'xspeed' ),
	138	+ $shown
	139	+ ),
	140	+ ),
	141	+ );
	142	+ }
	143	+
62	144	public function rest_routes(): array {
63	145	// Schema POST/GET come from the base; we add three custom routes.
64	146	$default = parent::rest_routes();
65	147	return array_merge(
		@@ -108,8 +190,10 @@
108	190	);
109	191	}
110	192
111	193	public function boot(): void {
	194	+ self::$configured_revisions = defined( 'WP_POST_REVISIONS' ) ? constant( 'WP_POST_REVISIONS' ) : null;
	195	+ add_filter( 'wp_revisions_to_keep', array( __CLASS__, 'filter_revisions_to_keep' ), 10, 2 );
112	196	add_action( Database_Cleaner::CRON_HOOK, array( Database_Cleaner::class, 'cron_tick' ) );
113	197	add_action( 'update_option_xspeed_module_database', array( $this, 'on_settings_change' ), 10, 2 );
114	198	add_action( 'add_option_xspeed_module_database', array( $this, 'on_settings_added' ), 10, 2 );
115	199	}
		@@ -118,8 +202,16 @@
118	202	wp_clear_scheduled_hook( Database_Cleaner::CRON_HOOK );
119	203	}
120	204
121	205	public function on_settings_change( $old, $new ): void {
	206	+ // A revision-limit save must not reschedule cleanup: apply_schedule()
	207	+ // restarts the timer, which pulled a weekly run forward on every save.
	208	+ $old = is_array( $old ) ? $old : array();
	209	+ if ( is_array( $new )
	210	+ && ( $old['schedule'] ?? null ) === ( $new['schedule'] ?? null )
	211	+ && ( $old['included_types'] ?? null ) === ( $new['included_types'] ?? null ) ) {
	212	+ return;
	213	+ }
122	214	$schedule = is_array( $new ) ? (string) ( $new['schedule'] ?? 'manual' ) : 'manual';
123	215	$types = is_array( $new ) && isset( $new['included_types'] ) && is_array( $new['included_types'] ) ? $new['included_types'] : array();
124	216	Database_Cleaner::apply_schedule( $schedule, $types );
125	217	}