Added options to reverse the logs form oldest at the top to newest.

evotodi · Sep 16, 2021 · 7dedc46 · 7dedc46
1 parent 724dead
commit 7dedc46
Show file tree

Hide file tree

Showing 5 changed files with 18 additions and 3 deletions.
diff --git a/src/Controller/LogViewerController.php b/src/Controller/LogViewerController.php
@@ -63,7 +63,11 @@ public function logViewAction(Request $request): Response
 	    }
 
 	    if(!empty($lines)){
-	    	$context['log'] = $lines;
+            if($this->logList->getLogsReverse()) {
+                $context['log'] = array_reverse($lines);
+            }else {
+                $context['log'] = $lines;
+            }
 	    }else{
 	    	$context['noLog'] = true;
 	    }

diff --git a/src/DependencyInjection/Configuration.php b/src/DependencyInjection/Configuration.php
@@ -46,6 +46,7 @@ public function getConfigTreeBuilder(): TreeBuilder
 			->booleanNode('show_app_logs')->defaultTrue()->info('Show App logs in var/log')->end()
             ->scalarNode('app_pattern')->info('See ddtraceweb/monolog-parser for patterns.')->defaultNull()->end()
             ->scalarNode('app_date_format')->info('PHP style date format of app log files')->defaultNull()->end()
+            ->booleanNode('logs_reverse')->defaultFalse()->info('Show log file as newest at the top')->end()
 		->end();
 
 		return $treeBuilder;

diff --git a/src/DependencyInjection/EvotodiLogViewerExtension.php b/src/DependencyInjection/EvotodiLogViewerExtension.php
@@ -27,6 +27,7 @@ public function load(array $configs, ContainerBuilder $container): array
 		$definition->replaceArgument(2, $config['show_app_logs']);
         $definition->replaceArgument(3, $config['app_pattern']);
         $definition->replaceArgument(4, $config['app_date_format']);
+        $definition->replaceArgument(5, $config['logs_reverse']);
 
 		return $config;
 	}

diff --git a/src/Resources/config/evo_log_viewer.xml b/src/Resources/config/evo_log_viewer.xml
@@ -13,6 +13,7 @@
             <argument/>
             <argument/>
             <argument/>
+            <argument/>
         </service>
         <service id="Evotodi\LogViewerBundle\Service\LogList" alias="evotodi_log_list.log_list_service"/>
 

diff --git a/src/Service/LogList.php b/src/Service/LogList.php
@@ -5,7 +5,6 @@
 
 use Evotodi\LogViewerBundle\Models\LogFile;
 use Symfony\Component\DependencyInjection\ParameterBag\ParameterBagInterface;
-use Symfony\Component\Finder\Exception\DirectoryNotFoundException;
 use Symfony\Component\Finder\Finder;
 
 class LogList
@@ -15,6 +14,7 @@ class LogList
 	private bool $useAppLogs;
     private ?string $appPattern;
     private ?string $appDateFormat;
+    private ?bool $logsReverse;
 
 	protected array $levels = [
 		"debug" => "DEBUG",
@@ -26,13 +26,16 @@ class LogList
         "critical" => "CRITICAL",
         "emergency" => "EMERGENCY",
 	];
-	public function __construct(ParameterBagInterface $parameterBag, array $logFiles, bool $useAppLogs = false, ?string $appPattern = null, ?string $appDateFormat = null)
+
+	public function __construct(ParameterBagInterface $parameterBag, array $logFiles, bool $useAppLogs = false, ?string $appPattern = null, ?string $appDateFormat = null, ?bool $logsReverse = null)
 	{
 		$this->parameterBag = $parameterBag;
 		$this->logFiles = $logFiles;
 		$this->useAppLogs = $useAppLogs;
         $this->appPattern = $appPattern;
         $this->appDateFormat = $appDateFormat;
+        $this->logsReverse = $logsReverse;
+
         if(is_null($this->appDateFormat)){
             $this->appDateFormat = 'Y-m-d H:i:s';
         }
@@ -97,4 +100,9 @@ public function getLogList(): array
 	    }
         return $logs;
     }
+
+    public function getLogsReverse(): ?bool
+    {
+        return $this->logsReverse;
+    }
 }