Instance tags should not be deleted when the instance is updated by an imported MARC record

Description

Overview: The ability to assign tags to instance records (as well as holdings and items) was added in Iris. Tags are not controlled by any underlying MARC field, and are not part of the default MARC-to-Instance map. However, when a user updates an Instance via Data Import, if the instance has tags assigned to it, the tags are removed. That should not happen. Tag data should be treated like other instance data not controlled by the underlying source record (Catalog date, status, statistical code, nature of content). And since it it not (yet) mappable in the Data Import field mapping profiles, any existing Tags data on an Instance (or Holdings or Item) should not be changed or removed when the Inventory record is updated via Data Import.

Current workaround: None

Steps to Reproduce:

  1. Log into FOLIO-snapshot-load as diku_admin

  2. Import the attached MARC record using the default profile that creates an SRS MARC and Instance

  3. Go to the Instance and click on the tags icon a the top right

  4. Add a couple tags to the record. Once the tags are added, the count next to the tags icon will go from 0 to 2

  5. Save the UUID of the instance, and then export that instance using the standard export map

  6. Create an update job profile

    • Match profile: MARC 001 to Instance HRID

    • Action profile: Update instance

    • Field mapping profile: Add a catalog date and a status

  7. Import the exported MARC record using the Update job profile

  8. Go to Inventory and check to see if any tags are on the instance

Expected Results: The tags should still be on the instance

Add testrails for testing tags on Holdings and Items as well

Actual Results: The tags are removed

Note: If this is happening because of the MARC-to-Instance map, then I think it would only affect instances. If this is happening because tags are not accounted for in the field mapping profiles, then it may be affecting holdings and items as well. Ant inventory record (instance, holdings, and/or item) should retain its tags when updated via an import. At some point, we will add mapping for tags into the field mapping profiles, but not yet. ()

Additional Information: See attached video and MARC file

CSP Request Details

None

CSP Rejection Details

None

Potential Workaround

None

Attachments

3

Checklist

hide

TestRail: Results

Activity

Show:

Ann-Marie BreauxJune 28, 2021 at 3:47 AM

Tested on Iris Bugfest - all worked fine after the permissions update. Closing this ticket. Will write TestRails to check tags on all 3 inventory record types

Ann-Marie BreauxJune 24, 2021 at 3:28 PM

Hi Based on our conversation with , it sounds like the UI permission called Data Import: all permissions needs to have the following backend permissions added to it:

  • inventory storage - create individual preceding-succeeding-title

  • inventory storage - delete individual preceding-succeeding-title

  • inventory storage - get individual preceding-succeeding-title

  • inventory storage - get preceding-succeeding-title collection

  • inventory storage - modify preceding-succeeding-title

If possible, it would be awesome to code the new Jira ticket as Iris Hotfix 2, and get it released today or tomorrow, since that will allow me to (hopefully) test and close 3 of these Iris Hotfix 2 issues that are not testing properly on Iris Bugfest now.

cc:

Volodymyr RohachJune 24, 2021 at 12:16 PM
Edited

Hi ! Yes,  it seems like there are some problems with the environment. Tags are not related to the preceding/succeeding at all. I think the problem is in the permission, which Ruslan mentioned in these tickets (/). 

So, it seems like this fix is fine, but there are some problems with permissions for the preceding/succeeding. 

 

CC:

Ann-Marie BreauxJune 24, 2021 at 9:56 AM

Thank you! And I think there may be a related one that I alerted Ruslan to, on and . Since these all tested fine on snapshot, I'm about 60% sure that it's a config problem, or an issue with the MARC-to-Instance default map in Iris Bugfest.

Volodymyr RohachJune 24, 2021 at 9:52 AM

Hi ! I'm investigating this one now. Thanks!

Done

Details

Assignee

Reporter

Priority

Story Points

Sprint

Development Team

Folijet

Fix versions

Release

R1 2021 Hot FIx #2

CSP Approved

Yes

TestRail: Cases

Open TestRail: Cases

TestRail: Runs

Open TestRail: Runs

Created May 13, 2021 at 6:07 AM
Updated June 28, 2021 at 4:17 AM
Resolved May 28, 2021 at 11:26 AM
TestRail: Cases
TestRail: Runs