Code style + translation

Author: tatevikg1

resources/translations/messages.en.xlf

@@ -730,6 +730,10 @@ Thank you.</target>
         <source>Campaign not found or not in submitted status</source>
         <target>__Campaign not found or not in submitted status</target>
       </trans-unit>
+      <trans-unit id="TBYUW2m" resname="Conflict: email and foreign key refer to different subscribers.">
+        <source>Conflict: email and foreign key refer to different subscribers.</source>
+        <target>__Conflict: email and foreign key refer to different subscribers.</target>
+      </trans-unit>
     </body>
   </file>
 </xliff>

src/Domain/Subscription/Service/CsvRowToDtoMapper.php

@@ -16,10 +16,7 @@ class CsvRowToDtoMapper
     public function map(array $row): ImportSubscriberDto
     {
         // Normalize keys to lower-case for header matching safety (CSV library keeps original headers)
-        $normalizedRow = [];
-        foreach ($row as $key => $value) {
-            $normalizedRow[strtolower((string)$key)] = is_string($value) ? trim($value) : $value;
-        }
+        $normalizedRow = $this->normalizeData($row);
 
         $email = strtolower(trim((string)($normalizedRow['email'] ?? '')));
 
@@ -42,4 +39,14 @@ public function map(array $row): ImportSubscriberDto
             foreignKey: $foreignKey ?? null,
         );
     }
+
+    private function normalizeData(array $row): array
+    {
+        $normalizedRow = [];
+        foreach ($row as $key => $value) {
+            $normalizedRow[strtolower((string)$key)] = is_string($value) ? trim($value) : $value;
+        }
+
+        return $normalizedRow;
+    }
 }

src/Domain/Subscription/Service/SubscriberCsvImporter.php

@@ -204,20 +204,7 @@ private function processRow(
 
         $this->attributeManager->processAttributes($subscriber, $dto->extraAttributes);
 
-        $addedNewSubscriberToList = false;
-        $listLines = [];
-        if (!$subscriber->isBlacklisted() && count($options->listIds) > 0) {
-            foreach ($options->listIds as $listId) {
-                $created = $this->subscriptionManager->addSubscriberToAList($subscriber, $listId);
-                if ($created) {
-                    $addedNewSubscriberToList = true;
-                    $listLines[] = $this->translator->trans(
-                        'Subscribed to %list%',
-                        ['%list%' => $created->getSubscriberList()->getName()]
-                    );
-                }
-            }
-        }
+        [$listLines, $addedNewSubscriberToList] = $this->getHistoryListLines($subscriber, $options);
 
         if ($subscriber->isBlacklisted()) {
             $stats['blacklisted']++;
@@ -300,4 +287,27 @@ private function prepareConfirmationMessage(
 
         return null;
     }
+
+    private function getHistoryListLines(Subscriber $subscriber, SubscriberImportOptions $options): array
+    {
+        $addedNewSubscriberToList = false;
+        $listLines = [];
+        if (!$subscriber->isBlacklisted() && count($options->listIds) > 0) {
+            foreach ($options->listIds as $listId) {
+                $created = $this->subscriptionManager->addSubscriberToAList($subscriber, $listId);
+                if ($created) {
+                    $addedNewSubscriberToList = true;
+                    $listLines[] = $this->translator->trans(
+                        'Subscribed to %list%',
+                        ['%list%' => $created->getSubscriberList()->getName()]
+                    );
+                }
+            }
+        }
+
+        return [
+            $listLines,
+            $addedNewSubscriberToList,
+        ];
+    }
 }