cleaning

COmanage · Nov 7, 2025 · e202357 · e202357
1 parent ebdf7c5
commit e202357
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 19 deletions.
diff --git a/app/plugins/Transmogrify/src/Command/TransmogrifyCommand.php b/app/plugins/Transmogrify/src/Command/TransmogrifyCommand.php
@@ -210,9 +210,6 @@ public function execute(Arguments $args, ConsoleIo $io): int
     $this->args = $args;
     $this->io = $io;
 
-    // Info is forced to be white
-    $io->setStyle('info', ['text' => '0;39']);
-
     // Now that BaseCommand set verbosity, construct the printer so it can detect it correctly
     $this->cmdPrinter = new CommandLinePrinter($io, 'green', 50, true);
 
@@ -409,15 +406,15 @@ public function execute(Arguments $args, ConsoleIo $io): int
           // not linked to a CO Person that was not migrated.
           $warns++;
           $this->cache['rejected'][$outboundQualifiedTableName][$row['id']] = $row;
-          $this->cmdPrinter->warn("Skipping $t record " . $row['id'] . " due to invalid foreign key: " . $e->getMessage());
+          $this->cmdPrinter->warning("Skipping $t record " . $row['id'] . " due to invalid foreign key: " . $e->getMessage());
           $this->cmdPrinter->pause();
         }
         catch(\InvalidArgumentException $e) {
           // If we can't find a value for mapping we skip the record
           // (ie: mapLegacyFieldNames basically requires a successful mapping)
           $warns++;
           $this->cache['rejected'][$outboundQualifiedTableName][$row['id']] = $row;
-          $this->cmdPrinter->warn("Skipping $t record " . $row['id'] . ": " . $e->getMessage());
+          $this->cmdPrinter->warning("Skipping $t record " . $row['id'] . ": " . $e->getMessage());
           $this->cmdPrinter->pause();
         }
         catch(\Exception $e) {
@@ -678,7 +675,7 @@ private function skipIfRejectedParent(string $currentTable, array $row): bool
       if ($childId !== null) {
         $this->cache['rejected'][$qualifiedCurrent][$childId] = $row;
       }
-      $this->cmdPrinter->warn(sprintf(
+      $this->cmdPrinter->warning(sprintf(
         'Skipping record %d in table %s - parent %s(%d) was rejected (self-reference)',
         (int)($childId ?? 0),
         $currentTable,
@@ -711,7 +708,7 @@ private function skipIfRejectedParent(string $currentTable, array $row): bool
         if ($childId !== null) {
           $this->cache['rejected'][$qualifiedCurrent][$childId] = $row;
         }
-        $this->cmdPrinter->warn(sprintf(
+        $this->cmdPrinter->warning(sprintf(
           'Skipping record %d in table %s - parent %s(%d) was rejected',
           (int)($childId ?? 0),
           $currentTable,

diff --git a/app/plugins/Transmogrify/src/Lib/Util/CommandLinePrinter.php b/app/plugins/Transmogrify/src/Lib/Util/CommandLinePrinter.php
@@ -91,6 +91,10 @@ public function __construct(?ConsoleIo $io = null, string $barColor = 'blue', in
     $this->barColor = in_array($barColor, ['blue', 'green'], true) ? $barColor : 'blue';
     $this->barWidth = max(10, $barWidth);
     $this->useColors = $useColors;
+
+    // Info is forced to be white
+    $this->io->setStyle('info', ['text' => '0;39']);
+    $this->io->setStyle('question', ['text' => '0;39']);
   }
 
   /**
@@ -208,18 +212,6 @@ public function info(string $message): void
     $this->message($message, 'info');
   }
 
-  /**
-   * Display a warning level message
-   *
-   * @param string $message Message to display
-   * @return void
-   * @since  COmanage Registry v5.2.0
-   */
-  public function warn(string $message): void
-  {
-    $this->message($message, 'warn');
-  }
-
   /**
    * Display a warning level message (alias of warn())
    *