UIIN-3558: Add additional call number to version history for item

CHANGELOG.md

@@ -86,6 +86,7 @@ UIIN-3437.
 * Wrong message is displayed after updating ownership of item. Fixes UIIN-3560.
 * When moving item within one holding manually, recalculate other item orders based on their position in the list. Fixes UIIN-3539.
 * Handle audit-marc dependency: hide audit button. Refs UIIN-3576.
+* Include additional call numbers in Version History for Inventory Item. Refs UIIN-3558.
 
 ## [13.0.10](https://github.com/folio-org/ui-inventory/tree/v13.0.10) (2025-09-01)
 [Full Changelog](https://github.com/folio-org/ui-inventory/compare/v13.0.9...v13.0.10)

src/Item/ViewItem/components/ItemVersionHistory/ItemVersionHistory.js

@@ -58,6 +58,32 @@ export const createFieldFormatter = (referenceData, circulationHistory) => ({
   },
 });
 
+export const createItemFormatter = (fieldLabelsMap, fieldFormatter) => (element, i) => {
+  if (!element) return null;
+
+  const { name: fieldName, value, collectionName } = element;
+  const compositeKey = collectionName && fieldName
+    ? `${collectionName}.${fieldName}`
+    : null;
+
+  const label = (compositeKey && fieldLabelsMap?.[compositeKey])
+    || fieldLabelsMap?.[fieldName]
+    || fieldLabelsMap?.[collectionName];
+
+  const formattedValue = (compositeKey && fieldFormatter?.[compositeKey]?.(value))
+    || fieldFormatter?.[fieldName]?.(value)
+    || fieldFormatter?.[collectionName]?.(value)
+    || value;
+
+  return (
+    <li key={i}>
+      {fieldName && <strong>{label}: </strong>}
+      {formattedValue}
+    </li>
+  );
+};
+
+
 const ItemVersionHistory = ({
   item,
   onClose,
@@ -137,9 +163,21 @@ const ItemVersionHistory = ({
     typeId: formatMessage({ id: 'ui-inventory.effectiveCallNumberType' }),
     volume: formatMessage({ id: 'ui-inventory.volume' }),
     yearCaption: formatMessage({ id: 'ui-inventory.yearCaption' }),
+    additionalCallNumbers: formatMessage({ id: 'ui-inventory.additionalCallNumbers' }),
+    'additionalCallNumbers.prefix': formatMessage({ id: 'ui-inventory.additionalCallNumberPrefix' }),
+    'additionalCallNumbers.suffix': formatMessage({ id: 'ui-inventory.additionalCallNumberSuffix' }),
+    'additionalCallNumbers.typeId': formatMessage({ id: 'ui-inventory.additionalCallNumberType' }),
+    'additionalCallNumbers.callNumber': formatMessage({ id: 'ui-inventory.additionalCallNumber' }),
+    'circulationNotes.noteType': formatMessage({ id: 'ui-inventory.noteType' }),
+    'circulationNotes.note': formatMessage({ id: 'ui-inventory.note' }),
+    'circulationNotes.id': formatMessage({ id: 'ui-inventory.identifier' }),
+    'circulationNotes.date': formatMessage({ id: 'ui-inventory.date' }),
+    'circulationNotes.staffOnly': formatMessage({ id: 'ui-inventory.staffOnly' }),
+    'circulationNotes.source': formatMessage({ id: 'ui-inventory.source' }),
   };
 
   const fieldFormatter = createFieldFormatter(referenceData, circulationHistory);
+  const itemFormatter = createItemFormatter(fieldLabelsMap, fieldFormatter);
 
   return (
     <AuditLogPane
@@ -151,6 +189,7 @@ const ItemVersionHistory = ({
       isInitialLoading={isLoading}
       fieldLabelsMap={fieldLabelsMap}
       fieldFormatter={fieldFormatter}
+      itemFormatter={itemFormatter}
       actionsMap={actionsMap}
       totalVersions={totalVersions}
     />

src/Item/ViewItem/components/ItemVersionHistory/ItemVersionHistory.test.js

@@ -11,7 +11,7 @@ import {
   translationsProperties,
 } from '../../../../../test/jest/helpers';
 
-import ItemVersionHistory, { createFieldFormatter } from './ItemVersionHistory';
+import ItemVersionHistory, { createFieldFormatter, createItemFormatter } from './ItemVersionHistory';
 import { DataContext } from '../../../../contexts';
 
 import {
@@ -278,3 +278,104 @@ describe('createFieldFormatter', () => {
     expect(fieldFormatter.source({ personal: { lastName: 'Doe' } })).toBe('Doe');
   });
 });
+
+describe('createItemFormatter', () => {
+  const fieldLabelsMap = {
+    barcode: 'Item Barcode',
+    discoverySuppress: 'Discovery Suppress',
+    circulationNotes: 'Circulation History',
+    'additionalCallNumbers.prefix': 'Additional call number prefix',
+    'additionalCallNumbers.suffix': 'Additional call number suffix',
+    'additionalCallNumbers.typeId': 'Additional call number type',
+    'additionalCallNumbers.callNumber': 'Additional call number',
+    'circulationNotes.staffOnly': 'Staff Only',
+    'circulationNotes.note': 'Note',
+    'circulationNotes.noteType': 'Note Type',
+    materialTypeId: 'Material Type',
+  };
+
+  const fieldFormatter = createFieldFormatter(mockReferenceData, {
+    servicePointName: 'Main Desk',
+    source: 'Librarian User',
+  });
+
+  const itemFormatter = createItemFormatter(fieldLabelsMap, fieldFormatter);
+
+  it('should return null for null element', () => {
+    expect(itemFormatter(null, 0)).toBeNull();
+  });
+
+  it('should return null for undefined element', () => {
+    expect(itemFormatter(undefined, 0)).toBeNull();
+  });
+
+  it('should format field with collectionName using composite key', () => {
+    const element = {
+      name: 'staffOnly',
+      value: false,
+      collectionName: 'circulationNotes',
+    };
+
+    const result = itemFormatter(element, 0);
+    const { container } = renderWithIntl(result, translationsProperties);
+
+    expect(container.querySelector('strong')).toHaveTextContent('Staff Only:');
+    expect(container.querySelector('li')).toHaveTextContent('Staff Only: false');
+  });
+
+  it('should fallback to fieldName label when composite key not found', () => {
+    const element = {
+      name: 'discoverySuppress',
+      value: true,
+      collectionName: 'unknownCollection',
+    };
+
+    const result = itemFormatter(element, 0);
+    const { container } = renderWithIntl(result, translationsProperties);
+
+    expect(container.querySelector('strong')).toHaveTextContent('Discovery Suppress:');
+    expect(container.querySelector('li')).toHaveTextContent('Discovery Suppress: true');
+  });
+
+  it('should fallback to collectionName label when fieldName not found', () => {
+    const element = {
+      name: 'unknownField',
+      value: 'test value',
+      collectionName: 'circulationNotes',
+    };
+
+    const result = itemFormatter(element, 0);
+    const { container } = renderWithIntl(result, translationsProperties);
+
+    expect(container.querySelector('strong')).toHaveTextContent('Circulation History:');
+    expect(container.querySelector('li')).toHaveTextContent('Circulation History: test value');
+  });
+
+  it('should render additionalCallNumbers.prefix with label and value', () => {
+    const element = {
+      name: 'prefix',
+      value: 'ABC',
+      collectionName: 'additionalCallNumbers',
+    };
+
+    const result = itemFormatter(element, 0);
+    const { container } = renderWithIntl(result, translationsProperties);
+
+    expect(container.querySelector('li'))
+      .toHaveTextContent('Additional call number prefix: ABC');
+  });
+
+  it('should render circulationNotes.note with label and value', () => {
+    const element = {
+      name: 'note',
+      value: 'Damaged cover',
+      collectionName: 'circulationNotes',
+    };
+
+    const result = itemFormatter(element, 0);
+    const { container } = renderWithIntl(result, translationsProperties);
+
+    expect(container.querySelector('li'))
+      .toHaveTextContent('Note: Damaged cover');
+  });
+});

src/edit/items/ItemForm.test.js

@@ -258,7 +258,7 @@ describe('ItemForm', () => {
         ...mockReferenceTables,
         callNumberTypes: [{ id: '1', name: 'Library of Congress classification' }],
       };
-      const { getByText, getAllByText, queryByText } = renderItemForm({
+      const { getByText, getAllByText } = renderItemForm({
         initialValues,
         referenceTables,
       });
@@ -291,9 +291,9 @@ describe('ItemForm', () => {
           'itemLevelCallNumberTypeId': { value: '2' },
           'additionalCallNumbers': {
             value: [{
-              callNumber: 'cn1',
-              prefix: 'prefix1',
-              suffix: 'suffix1',
+              additionalCallNumber: 'cn1',
+              additionalCallNumberPrefix: 'prefix1',
+              additionalCallNumberSuffix: 'suffix1',
               typeId: '1'
             }]
           }

translations/ui-inventory/en.json

@@ -216,6 +216,11 @@
   "callNumberPrefix": "Call number prefix",
   "callNumberSuffix": "Call number suffix",
   "primaryItemCallNumber": "Primary item call number",
+  "additionalCallNumbers": "Additional call numbers",
+  "additionalCallNumber": "Additional call number",
+  "additionalCallNumberPrefix": "Additional call number prefix",
+  "additionalCallNumberSuffix": "Additional call number suffix",
+  "additionalCallNumberType": "Additional call number type",
   "additionalItemCallNumbers": "Additional item call numbers",
   "primaryHoldingsCallNumber": "Primary holdings call number",
   "additionalHoldingsCallNumbers": "Additional holdings call numbers",