User able to save record with created by user '001' field value when derive MARC Bib / create MARC Holdings

Description

Overview:  User can save the record with added by user "001" field when Deriving MARC Bibliographic / Creating MARC Holdings record (no error toast notification is shown).
There is no added by user 001 field in created record  :check_mark:
Steps to Reproduce:

  1. Log into Orchid Bugfest FOLIO environment as User with "view, edit, derive bibliographic/ create holdings records via quickmarc"

  2. Open the deriving window of Bibliographic / creating window of Holdings record.

  3. Click on "+" icon to add a new row.

  4. Fill in $a subfield with any value.

  5. Fill in "001" value in MARC tag text box.

  6. Click on the "Save & close" button.

Expected Results: When user click on the "Save & close" button the toast notification with the warning message "Record cannot be saved. Can only have one MARC 001." appears. The deriving / creating window stays opened.

Actual Results: No error shown. Record successfully created. No field added by user is displayed in the new record.

Additional Information:  
1) see attached screencast.
2) This issue is reproduced on Orchid bugfest environment and Snapshot.
3) See how it was implemented on Nolana bugfest environment.

Interested parties :

CSP Request Details

None

CSP Rejection Details

None

Potential Workaround

None

Attachments

4
  • 06 Apr 2023, 11:32 AM
  • 05 Apr 2023, 09:31 AM
  • 04 Apr 2023, 07:08 AM
  • 16 Mar 2023, 05:34 PM

Checklist

hide

TestRail: Results

Activity

Show:

Valery_Pilko April 11, 2023 at 5:57 AM

Moving this issue to PO review, according to our discussion on stand-up.
Hi  
Should I create a new issue to address the inconsistency in behavior when the user creates a new MARC Holdings record and adds a new 001 with some value?

Khalilah Gambrell April 7, 2023 at 12:28 PM

Thanks  

What is the easiest to implement for consistency 

  • Show error message? Or allow Save to proceed and remove the human entered 001? 

Valery_Pilko April 6, 2023 at 2:12 PM

Hey  
Yes

Khalilah Gambrell April 6, 2023 at 1:45 PM

So is the only question related to this issue whether we should have the same behavior for handling multiple 001s?

Valery_Pilko April 6, 2023 at 11:32 AM

Hey  
It's already working in the way you've described for "Create MARC Holdings" and working different for "Derive MARC Bib" (Error shown with message that you can't have multiple 001)
See screencast:

Done

Details

Assignee

Reporter

Priority

Story Points

Sprint

Development Team

Spitfire

Fix versions

Release

Poppy (R2 2023)

RCA Group

Legitimate regression

Affected releases

Orchid (R1 2023)

TestRail: Cases

Open TestRail: Cases

TestRail: Runs

Open TestRail: Runs
Created March 16, 2023 at 5:24 PM
Updated October 3, 2023 at 3:36 PM
Resolved April 15, 2023 at 1:10 PM
TestRail: Cases
TestRail: Runs

Flag notifications