chore: use correct log level

ColdTrick · Nov 25, 2024 · f7c7fd4 · f7c7fd4
1 parent 78e3ea1
commit f7c7fd4
Show file tree

Hide file tree

Showing 5 changed files with 11 additions and 6 deletions.
diff --git a/classes/ColdTrick/OpenSearch/Cron.php b/classes/ColdTrick/OpenSearch/Cron.php
@@ -6,6 +6,7 @@
 use ColdTrick\OpenSearch\Di\SearchService;
 use Elgg\Database\QueryBuilder;
 use OpenSearch\Common\Exceptions\OpenSearchException;
+use Psr\Log\LogLevel;
 
 /**
  * Cron handler
@@ -189,7 +190,7 @@ function (QueryBuilder $qb, $main_alias) {
 			});
 		} catch (OpenSearchException $e) {
 			// probably reached the end of the scroll
-			// elgg_log('OpenSearch cleanup: ' . $e->getMessage(), 'ERROR');
+//			 elgg_log('OpenSearch cleanup: ' . $e->getMessage(), LogLevel::ERROR);
 		}
 
 		// clear scroll

diff --git a/classes/ColdTrick/OpenSearch/Di/IndexingService.php b/classes/ColdTrick/OpenSearch/Di/IndexingService.php
@@ -5,6 +5,7 @@
 use Elgg\Database\QueryBuilder;
 use Elgg\Cli\Progress;
 use OpenSearch\Common\Exceptions\OpenSearchException;
+use Psr\Log\LogLevel;
 use Symfony\Component\Console\Helper\ProgressBar;
 
 /**
@@ -301,7 +302,7 @@ protected function processBulkIndexEntities(array $entities): void {
 				$this->skip_guids[] = $guid;
 
 				$error = elgg_extract('error', elgg_extract('index', $item));
-				elgg_log("OpenSearch failed to index {$guid} with error [{$status}][{$error['type']}]: {$error['reason']}", 'WARNING');
+				elgg_log("OpenSearch failed to index {$guid} with error [{$status}][{$error['type']}]: {$error['reason']}", LogLevel::WARNING);
 				continue;
 			}
 

diff --git a/classes/ColdTrick/OpenSearch/SearchEvents.php b/classes/ColdTrick/OpenSearch/SearchEvents.php
@@ -5,6 +5,7 @@
 use ColdTrick\OpenSearch\Di\SearchService;
 use Elgg\Exceptions\ExceptionInterface;
 use Elgg\Exceptions\UnexpectedValueException;
+use Psr\Log\LogLevel;
 
 /**
  * Listen to different search events
@@ -693,7 +694,7 @@ public static function sourceToEntity(\Elgg\Event $event): ?\ElggEntity {
 		try {
 			return _elgg_services()->entityTable->rowToElggStar($row);
 		} catch (ExceptionInterface $e) {
-			elgg_log($e->getMessage(), 'NOTICE');
+			elgg_log($e->getMessage(), LogLevel::NOTICE);
 		}
 
 		return null;

diff --git a/lib/functions.php b/lib/functions.php
@@ -9,6 +9,7 @@
 use Elgg\Database\Select;
 use Elgg\Exceptions\ExceptionInterface;
 use Elgg\Values;
+use Psr\Log\LogLevel;
 
 /**
  * Get the type/subtypes to index in OpenSearch
@@ -172,7 +173,7 @@ function opensearch_add_document_for_deletion(int $guid, array $info, mixed $tim
 	try {
 		$queue = DeleteQueue::instance();
 	} catch (\Exception $e) {
-		elgg_log($e, 'WARNING');
+		elgg_log($e, LogLevel::WARNING);
 		return;
 	}
 
@@ -224,7 +225,7 @@ function opensearch_get_documents_for_deletion(): array {
 	try {
 		$queue = DeleteQueue::instance();
 	} catch (ExceptionInterface $e) {
-		elgg_log($e, 'WARNING');
+		elgg_log($e, LogLevel::WARNING);
 		return [];
 	}
 

diff --git a/views/default/forms/opensearch/admin_search.php b/views/default/forms/opensearch/admin_search.php
@@ -2,6 +2,7 @@
 
 use ColdTrick\OpenSearch\Di\IndexManagementService;
 use Elgg\Exceptions\ExceptionInterface;
+use Psr\Log\LogLevel;
 
 $index_client = IndexManagementService::instance();
 if (!$index_client->isClientReady()) {
@@ -20,7 +21,7 @@
 try {
 	$status = $index_client->getIndexStatus();
 } catch (ExceptionInterface $e) {
-	elgg_log($e, 'ERROR');
+	elgg_log($e, LogLevel::ERROR);
 
 	echo elgg_echo('opensearch:error:no_index');
 	return;