Refixing #5338 - Boost compatibility

Make this compatible with all PHP releases.
Cacti · Jun 3, 2023 · c3041fe · c3041fe
1 parent 2194ba1
commit c3041fe
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/CHANGELOG b/CHANGELOG
@@ -23,7 +23,7 @@ Cacti CHANGELOG
 -issue#5319: Data Source Debug 'Run All' generates too many log messages
 -issue#5323: Cosmetic error in the system utilities - rebuild poller cache - description
 -issue#5329: Upon displaying a form with errors, the session error fields variable isn't cleared
--issue#5338:
+-issue#5338: Compatibility changes for Boost under PHP 8.1
 -feature: Upgrade billboard.js to version 3.7.4
 -feature: Upgrade d3.js to version 7.8.2
 

diff --git a/poller_boost.php b/poller_boost.php
@@ -119,15 +119,15 @@
 	/* find out if it's time to collect device information
 	 * support both old and new formats.
 	 */
-	$boost_last_run_time = read_config_option('boost_last_run_time') ?? '';
-	if (!is_numeric($boost_last_run_time)) {
+	$boost_last_run_time = read_config_option('boost_last_run_time');
+	if (!empty($boost_last_run_time) && !is_numeric($boost_last_run_time)) {
 		$last_run_time = strtotime($boost_last_run_time);
 	} else {
 		$last_run_time = $boost_last_run_time;
 	}
 
-	$boost_next_run_time = read_config_option('boost_next_run_time') ?? '';
-	if (!is_numeric($boost_next_run_time)) {
+	$boost_next_run_time = read_config_option('boost_next_run_time');
+	if (!empty($boost_next_run_time) && !is_numeric($boost_next_run_time)) {
 		$next_run_time = strtotime($boost_next_run_time);
 	} else {
 		$next_run_time = $boost_next_run_time;