Browse Source

use loose comparison instead of strict comparison (#7064)

Update comparison checks when testing the database status and
add a unified log message with a more detailed message from the
database when there is a failure

Authored-by: Tim Fry <[email protected]>
frytimo 1 year ago
parent
commit
0c61a8cc97
1 changed files with 1 additions and 1 deletions
  1. 1 1
      core/users/resources/classes/users.php

+ 1 - 1
core/users/resources/classes/users.php

@@ -304,7 +304,7 @@ if (!class_exists('users')) {
 					$database->execute($sql);
 					$code = $database->message['code'] ?? 0;
 					if ($code == 200) {
-						maintenance_service::log_write(self::class, "Removed database entries older than $retention_days", $domain_uuid);
+						maintenance_service::log_write(self::class, "Successfully removed entries older than $retention_days", $domain_uuid);
 					} else {
 						$message = $database->message['message'] ?? "An unknown error has occurred";
 						maintenance_service::log_write(self::class, "Unable to remove old database records. Error message: $message ($code)", $domain_uuid, maintenance_service::LOG_ERROR);