fix: consider all pools as active namespaces

Author: shyim

src/Command/RedisNamespaceCleanupCommand.php

@@ -31,20 +31,21 @@ protected function configure(): void
     protected function execute(InputInterface $input, OutputInterface $output): int
     {
         $io = new SymfonyStyle($input, $output);
-        $cacheAdapter = $this->cacheRegistry->get('cache.object');
+        $activeNamespaces = $this->cacheRegistry->getActiveNamespaces();
+
         $dryRun = $input->getOption('dry-run');
 
-        try {
-            $redis = $cacheAdapter->getRedisOrFail();
-        } catch (\RuntimeException $e) {
-            $io->error($e->getMessage());
+        if (empty($activeNamespaces)) {
+            $io->error('No active Redis namespaces found. Please check your configuration.');
 
             return Command::FAILURE;
         }
 
-        $activeNamespace = $cacheAdapter->getNamespace();
+        $cacheApp = $this->cacheRegistry->get('cache.app');
+        $redis = $cacheApp->getRedisOrFail();
+
         $io->title('Redis Namespace Cleanup');
-        $io->writeln(\sprintf('Active namespace: <info>%s</info>', $activeNamespace));
+        $io->writeln(\sprintf('Active namespaces: <info>%s</info>', implode(', ', $activeNamespaces)));
 
         if ($dryRun) {
             $io->note('Running in dry-run mode - no keys will be deleted');
@@ -73,7 +74,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
                 ++$namespaces[$namespace];
 
                 // Track keys that are not in the active namespace
-                if ($namespace !== $activeNamespace) {
+                if (!\in_array($namespace, $activeNamespaces, true)) {
                     $keysToDelete[] = $key;
                 }
             }
@@ -82,10 +83,14 @@ protected function execute(InputInterface $input, OutputInterface $output): int
         // Display namespace summary
         $tableData = [];
         foreach ($namespaces as $namespace => $count) {
-            $status = $namespace === $activeNamespace ? 'KEEP' : 'DELETE';
+            $status = \in_array($namespace, $activeNamespaces, true) ? 'KEEP' : 'DELETE';
             $tableData[] = [$namespace, $count, $status];
         }
 
+        usort($tableData, function ($a, $b) {
+            return $b[0] <=> $a[0];
+        });
+
         $io->section('Namespace Summary');
         $io->table(
             ['Namespace', 'Key Count', 'Action'],

src/Command/RedisNamespaceListCommand.php

@@ -35,7 +35,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
             return Command::FAILURE;
         }
 
-        $namespace = $cacheAdapter->getNamespace();
+        $activeNamespaces = $this->cacheRegistry->getActiveNamespaces();
         $io->title('Redis Key Groupping by Namespace');
 
         // Group keys by first 10 characters
@@ -66,9 +66,13 @@ protected function execute(InputInterface $input, OutputInterface $output): int
         // Display results in a table
         $tableData = [];
         foreach ($keyGroups as $prefix => $count) {
-            $tableData[] = [$prefix, $count, \sprintf('%.1f%%', ($count / $totalKeys) * 100), $namespace === $prefix ? 'Yes' : 'No'];
+            $tableData[] = [$prefix, $count, \sprintf('%.1f%%', ($count / $totalKeys) * 100), \in_array($prefix, $activeNamespaces, true) ? 'Yes' : 'No'];
         }
 
+        usort($tableData, function ($a, $b) {
+            return $b[0] <=> $a[0];
+        });
+
         $io->table(
             ['Prefix', 'Count', 'Percentage', 'Active'],
             $tableData

src/Components/CacheRegistry.php

@@ -37,4 +37,21 @@ public function has(string $name): bool
     {
         return isset($this->adapters[$name]);
     }
+
+    /**
+     * @return array<string>
+     */
+    public function getActiveNamespaces(): array
+    {
+        $list = [];
+
+        foreach ($this->adapters as $adapter) {
+            try {
+                $list[] = $adapter->getNamespace();
+            } catch (\Exception) {
+            }
+        }
+
+        return array_unique($list);
+    }
 }

src/Resources/app/administration/src/api/frosh-tools.js

@@ -187,7 +187,7 @@ class FroshTools extends ApiService {
                 headers: this.getBasicHeaders(),
             })
             .then((response) => {
-                return response
+                return response;
             });
     }
 

src/Resources/app/administration/src/module/frosh-tools/component/frosh-tools-tab-files/index.js

@@ -58,7 +58,9 @@ Component.register('frosh-tools-tab-files', {
 
         async createdComponent() {
             this.items = (await this.froshToolsService.getShopwareFiles()).data;
-            this.extensionItems = (await this.froshToolsService.getExtensionFiles()).data;
+            this.extensionItems = (
+                await this.froshToolsService.getExtensionFiles()
+            ).data;
             this.isLoading = false;
         },
 

src/Resources/app/administration/src/module/frosh-tools/component/frosh-tools-tab-files/template.twig

@@ -153,27 +153,19 @@
                 <dd v-for="file in extensionResult.newFiles">
                     {{ file }}
                 </dd>
-                <dt
-                    v-if="extensionResult.changedFiles.length > 0"
-                >
+                <dt v-if="extensionResult.changedFiles.length > 0">
                     {{ $t('frosh-tools.tabs.extensionFiles.changedFiles') }}
                 </dt>
-                <dd
-                    v-for="file in extensionResult.changedFiles"
-                >
+                <dd v-for="file in extensionResult.changedFiles">
                     {{ file }}
                 </dd>
-                <dt
-                    v-if="extensionResult.missingFiles.length > 0"
-                >
+                <dt v-if="extensionResult.missingFiles.length > 0">
                     {{ $t('frosh-tools.tabs.extensionFiles.missingFiles') }}
                 </dt>
-                <dd
-                    v-for="file in extensionResult.missingFiles"
-                >
+                <dd v-for="file in extensionResult.missingFiles">
                     {{ file }}
                 </dd>
             </sw-description-list>
         </div>
     </sw-card>
-</sw-card-view>
+</sw-card-view>