From 8f6ed653bd3771aa0ad89fd83242171c60339fa1 Mon Sep 17 00:00:00 2001
From: sharmasahil <sahil.sharma@dgs.vic.gov.au>
Date: Mon, 9 Dec 2024 12:10:37 +1100
Subject: [PATCH] Test failed fixes

---
 src/Command/AuditLogCleanupCommand.php | 21 +++++++++------------
 1 file changed, 9 insertions(+), 12 deletions(-)

diff --git a/src/Command/AuditLogCleanupCommand.php b/src/Command/AuditLogCleanupCommand.php
index 15577b36..45ed71d2 100755
--- a/src/Command/AuditLogCleanupCommand.php
+++ b/src/Command/AuditLogCleanupCommand.php
@@ -3,10 +3,8 @@
 namespace Drupal\tide_core\Command;
 
 use Drush\Commands\DrushCommands;
-use Drupal\Core\Database\Database;
-use DateInterval;
-use DateTime;
 use Drupal\Core\Config\ConfigFactoryInterface;
+use Drupal\Core\Database\Database;
 use Symfony\Component\DependencyInjection\ContainerInterface;
 
 /**
@@ -14,7 +12,7 @@
  */
 class AuditLogCleanupCommand extends DrushCommands {
 
-/**
+  /**
    * The configuration service.
    *
    * @var \Drupal\Core\Config\ConfigFactoryInterface
@@ -50,7 +48,7 @@ public static function create(ContainerInterface $container) {
    * @option force-cleanup Skip confirmation and run the cleanup immediately.
    */
   public function cleanupLogs($options = ['force-cleanup' => FALSE]) {
-    // Check if the user passed the --force-cleanup option
+    // Check if the user passed the --force-cleanup option.
     if (!$options['force-cleanup']) {
       // If the force-cleanup flag isn't passed, ask for confirmation.
       $confirmation = $this->confirmCleanup();
@@ -59,24 +57,22 @@ public function cleanupLogs($options = ['force-cleanup' => FALSE]) {
         return;
       }
     }
-    
     $config = $this->configFactory->get('tide_core.settings');
     define('DEFAULT_LOG_RETENTION_DAYS', 30);
     $log_retention_days = $config->get('log_retention_days') ?: DEFAULT_LOG_RETENTION_DAYS;
-    // Get current date and time
+    // Get current date and time.
     $current_time = new DateTime();
     $current_time->sub(new DateInterval("P{$log_retention_days}D"));
     $threshold_timestamp = $current_time->getTimestamp();
-    // Connect to the database
+    // Connect to the database.
     $database = Database::getConnection();
     $deleted = $database->delete('admin_audit_trail')
       ->condition('created', $threshold_timestamp, '<')
       ->execute();
 
-    // Output the result
+    // Output the result.
     $this->output()->writeln("Deleted $deleted log entries older than $log_retention_days days.");
-    
-    // Run a database optimization command to recover space
+    // Run a database optimization command to recover space.
     $this->optimizeDatabase();
   }
 
@@ -90,7 +86,7 @@ private function confirmCleanup() {
     $question = 'Are you sure you want to delete log entries older than the configured retention period? (y/n): ';
     $confirmation = $this->io()->ask($question, 'n');
     $confirmation = strtolower($confirmation);
-    // Return TRUE if the user answers 'y' or 'yes'
+    // Return TRUE if the user answers 'y' or 'yes'.
     return in_array($confirmation, ['y', 'yes']);
   }
 
@@ -98,6 +94,7 @@ private function confirmCleanup() {
    * Run database optimization (optional).
    *
    * @return void
+   *  TRUE write the message.
    */
   private function optimizeDatabase() {
     $database = Database::getConnection();