Revert the QM_RUN_IN_CLI constant (#216)

adamziel · web-flow · commit c5fd3974617f · 2025-07-24T18:25:41.000+02:00
#215 added a `QM_RUN_IN_CLI` check to enable running Query Monitor with the CLI sapi (it's always `cli` in Playground). Turns out, QM_TESTS is enough and shouldn't have any side-effects (at least in the current release). Let's roll back `QM_RUN_IN_CLI`.
diff --git a/integrations/query-monitor/boot.php b/integrations/query-monitor/boot.php
@@ -39,9 +39,7 @@
 }
 
 if ( 'cli' === php_sapi_name() && ! defined( 'QM_TESTS' ) ) {
-	if ( ! defined( 'QM_RUN_IN_CLI' ) || ! QM_RUN_IN_CLI ) {
-		return;
-	}
+	return;
 }
 
 if ( defined( 'DOING_CRON' ) && DOING_CRON ) {

Original file line number	Diff line number	Diff line change
`@@ -39,9 +39,7 @@`
`39`	`39`	`}`
`40`	`40`
`41`	`41`	`if ( 'cli' === php_sapi_name() && ! defined( 'QM_TESTS' ) ) {`
`42`		`- if ( ! defined( 'QM_RUN_IN_CLI' ) \|\| ! QM_RUN_IN_CLI ) {`
`43`		`- return;`
`44`		`- }`
	`42`	`+ return;`
`45`	`43`	`}`
`46`	`44`
`47`	`45`	`if ( defined( 'DOING_CRON' ) && DOING_CRON ) {`