UIQM-711: Update �alidateFixedFieldPositions to display all 008 field…

… errors instead of one in Bibliographic records. (#747)

CHANGELOG.md

@@ -38,6 +38,7 @@
 * [UIQM-714](https://issues.folio.org/browse/UIQM-714) *BREAKING* Upgrade `inventory` to `14.0`.
 * [UIQM-709](https://issues.folio.org/browse/UIQM-709) LCCN duplication: Update the bib request query to include only the non-suppressed record in the search results.
 * [UIQM-701](https://issues.folio.org/browse/UIQM-701) Use new GA workflows.
+* [UIQM-711](https://issues.folio.org/browse/UIQM-711) Update `validateFixedFieldPositions` to display all 008 field errors instead of one in Bibliographic records.
 
 ## [8.0.1] (https://github.com/folio-org/ui-quick-marc/tree/v8.0.1) (2024-04-18)
 

src/hooks/useValidation/validators.js

@@ -399,7 +399,7 @@ export const validateFixedFieldPositions = ({ marcRecords, fixedFieldSpec, marcT
       const subFieldContentArray = Array.isArray(contents) ? contents : [contents];
 
       if (!subFieldContentArray.every(content => subField.allowedValues.find(value => value.code === content))) {
-        return { ...acc, [field.id]: [rule.message(subField.code)] };
+        acc[field.id] = [...(acc[field.id] || []), rule.message(subField.code)];
       }
     }
 

src/hooks/useValidation/validators.test.js

@@ -713,6 +713,25 @@ describe('validators', () => {
 
       expect(rule.message).toHaveBeenCalledWith('Ills');
     });
+
+    it('should return all errors instead of one', () => {
+      const marcRecords = [{
+        tag: LEADER_TAG,
+        content: bibLeader,
+      }, {
+        id: 'id-008',
+        tag: FIXED_FIELD_TAG,
+        content: {
+          DtSt: '_',
+          Ills: '_',
+        },
+      }];
+      const marcType = MARC_TYPES.BIB;
+
+      const errors = validators.validateFixedFieldPositions({ marcRecords, fixedFieldSpec, marcType }, rule);
+
+      expect(errors['id-008'].length).toBe(2);
+    });
   });
 
   describe('validateLccnDuplication', () => {