Inventory cannot process Holdings with virtual fields

RCA Group

None

Description

Deserialization fails in mod-inventory for the Holdings received by querying GET /holdings-storage/holdings endpoint after the changes introduced in the scope of https://folio-org.atlassian.net/browse/MODINVSTOR-1094#icft=MODINVSTOR-1094 for the holdings record schema and API. HoldingsRecord schema no longer has "holdingsItems" and "bareHoldingsItems" fields, however GET response returns entity with those fields

Testing:

  1. Create a match profile 901$h to Holdings HRID

  2. Create a field mapping and action profile that adds an admin note to the holdings if matched

  3. Create job profile

    • Holdings HRID match

      • Match: Update holdings

      • No match: Take no action

  4. Import a MARC Bib file with holdings HRID in 901$h (either added manually or exported from FOLIO)

  5. Import should complete without errors, and holdings is updated with the new admin note

Environment

None

Potential Workaround

None

Attachments

3
  • 04 Jan 2024, 09:23 AM
  • 04 Jan 2024, 09:23 AM
  • 03 Jan 2024, 08:00 AM

Checklist

hide

TestRail: Results

Activity

Show:

Ryan Taylor January 4, 2024 at 7:58 PM

Thanks ! Followed along and this makes sense to me. Moving ticket to Done.

Tetiana Paranich January 4, 2024 at 9:26 AM
Edited

Hi ! I've added files. For checking this bug I used a test case C356791

Ryan Taylor January 3, 2024 at 8:57 PM

Thanks ! Do you have an example file to use when follow the steps described? Thank you!

Tetiana Paranich January 2, 2024 at 8:31 PM
Edited

Hi and ! Works as expected on https://folio-snapshot-2.dev.folio.org

See the attached

Done

Details

Assignee

Reporter

Priority

Story Points

Sprint

Development Team

Folijet

Fix versions

Release

Quesnelia (R1 2024)

TestRail: Cases

Open TestRail: Cases

TestRail: Runs

Open TestRail: Runs

Created December 13, 2023 at 1:28 PM
Updated March 20, 2024 at 9:16 PM
Resolved January 4, 2024 at 7:58 PM
TestRail: Cases
TestRail: Runs

Flag notifications