fix(registration): normalize list answer ordering + guard import row on answer persistence

Per CodeRabbit review: sort list-question value ids so the same selection
in a different order is not treated as a changed answer, and catch
ValidationException from the extra-question persistence path so one bad
payload cannot strand the remaining import rows.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

Author: JpMaxMan

app/Services/Model/Imp/SummitOrderService.php

@@ -4677,7 +4677,6 @@ public function processTicketData(int $summit_id, string $filename)
      * @param Summit $summit
      * @param SummitAttendee $attendee
      * @param array $row
-     * @throws ValidationException
      */
     private function upsertAttendeeExtraQuestionAnswers(Summit $summit, SummitAttendee $attendee, array $row): void
     {
@@ -4782,11 +4781,20 @@ private function upsertAttendeeExtraQuestionAnswers(Summit $summit, SummitAttend
                     );
                     continue;
                 }
+                sort($value_ids);
                 $value = implode(ExtraQuestionType::QuestionChoicesCharSeparator, $value_ids);
             }
 
             $former_answer = $attendee->getExtraQuestionAnswerByQuestion($question);
             $former_value = is_null($former_answer) ? '' : $former_answer->getValue();
+
+            // list answers compare as sets: the same selection in a different order is not a change
+            if ($question->allowsValues() && !empty($former_value)) {
+                $former_value_ids = array_map('intval', explode(ExtraQuestionType::QuestionChoicesCharSeparator, $former_value));
+                sort($former_value_ids);
+                if ($former_value_ids === $value_ids) $value = $former_value;
+            }
+
             if (!empty($former_value) && $former_value != $value && !$attendee->canChangeAnswerValue($question)) {
                 Log::warning
                 (
@@ -4838,7 +4846,21 @@ private function upsertAttendeeExtraQuestionAnswers(Summit $summit, SummitAttend
             )
         );
 
-        $attendee->hadCompletedExtraQuestions($extra_questions);
+        try {
+            $attendee->hadCompletedExtraQuestions($extra_questions);
+        } catch (ValidationException $ex) {
+            // never abort the import on a bad extra-question payload: the remaining
+            // rows must still be processed ( the queue job does not retry the file )
+            Log::warning
+            (
+                sprintf
+                (
+                    "SummitOrderService::upsertAttendeeExtraQuestionAnswers attendee %s could not persist answers: %s",
+                    $attendee->getEmail(),
+                    $ex->getMessage()
+                )
+            );
+        }
     }
 
     /**

tests/SummitOrderServiceTest.php

@@ -609,9 +609,10 @@ public function testImportTicketDataListQuestionStoresValueIds()
 
         $ticket = $this->getUnassignedTicket();
 
+        // reversed token order on purpose: stored value ids are normalized ( sorted )
         $csv_content = <<<CSV
 number,attendee_email,attendee_first_name,attendee_last_name,extra_question:T-Shirt Size
-{$ticket->getNumber()},new.attendee@nowhere.com,New,Attendee,Small|Large
+{$ticket->getNumber()},new.attendee@nowhere.com,New,Attendee,Large|Small
 CSV;
 
         $service = $this->buildTicketDataImportService($csv_content);
@@ -624,11 +625,12 @@ public function testImportTicketDataListQuestionStoresValueIds()
         $answer = $attendee->getExtraQuestionAnswerByQuestion($question);
         $this->assertNotNull($answer);
 
-        $expected_value = implode(',', [
+        $expected_value_ids = [
             $question->getValueByName('Small')->getId(),
             $question->getValueByName('Large')->getId(),
-        ]);
-        $this->assertEquals($expected_value, $answer->getValue());
+        ];
+        sort($expected_value_ids);
+        $this->assertEquals(implode(',', $expected_value_ids), $answer->getValue());
     }
 
     public function testImportTicketDataBadgeFeaturesStillClearedAndReSet()