Item alert and note creation silently fails in unified holdings/attribute editor

Bug #1959716 reported by Elaine Hardy
94
This bug affects 18 people
Affects Status Importance Assigned to Milestone
Evergreen
Fix Released
Medium
Unassigned
3.8
Fix Released
Medium
Unassigned

Bug Description

In 3.8

You cannot add item alert messages or item notes to an item in the process of creating it, as you could in previous version when using the unified holdings and item attributes display.

Attempting to add item alerts or notes in the attributes editor silently fails and locks the screen. Clicking on the item alerts or item notes button fails to open the interface and the screen locks so that you can't scroll. While you can make edits to those attributes in screen view, you can't scroll down to save the item(s) and have to close the editor tab to exit. Losing any edits you have already made.

The console message is:

main.83e1c293da137dcd2dd9.js:1 ERROR TypeError: Cannot read properties of undefined (reading 'barcode')
    at P (5.363c283a262b34e0b274.js:1:10331)
    at ii (main.83e1c293da137dcd2dd9.js:1:298711)
    at ni (main.83e1c293da137dcd2dd9.js:1:297286)
    at main.83e1c293da137dcd2dd9.js:1:297713
    at ni (main.83e1c293da137dcd2dd9.js:1:297738)
    at Ri (main.83e1c293da137dcd2dd9.js:1:305708)
    at hs.detectChanges (main.83e1c293da137dcd2dd9.js:1:311637)
    at tick (main.83e1c293da137dcd2dd9.js:1:388988)
    at main.83e1c293da137dcd2dd9.js:1:387435
    at l.invoke (polyfills.9b26e9b6c6d2049cc2b1.js:1:7302)

If you create the item first by clicking on Apply All & save, you can add alert messages and item notes.

Elaine Hardy (ehardy)
tags: added: silentfailure
Revision history for this message
Elaine Hardy (ehardy) wrote :
Revision history for this message
Jennifer Pringle (jpringle-u) wrote :

When I click "Apply All, Save & Exit" the alert or item note interface (whichever I tried to use) appears for a second while the holdings editor is closing. See attached screenshot.

Changed in evergreen:
status: New → Confirmed
Revision history for this message
Mary Llewellyn (mllewell) wrote :

The only reason it doesn't fail in the non-unified editor is that you have to click on Apply All & Save when you move from the Holdings tab to the Item Attributes tab. I tried going directly to the Item Attribute tab first to add an Item Note, but experienced the inability to add the note with the screen locking up.

Revision history for this message
Elaine Hardy (ehardy) wrote :

Yes, that is currently our work around for adding items to new items -- not use the unified editor to create the item first.

Revision history for this message
Christine Burns (christine-burns) wrote :

persists in 3.9

Revision history for this message
Bill Erickson (berick) wrote :
Changed in evergreen:
assignee: nobody → Bill Erickson (berick)
milestone: none → 3.9.1
Bill Erickson (berick)
tags: added: pullrequest
Changed in evergreen:
assignee: Bill Erickson (berick) → nobody
Galen Charlton (gmc)
Changed in evergreen:
importance: Undecided → Medium
assignee: nobody → Galen Charlton (gmc)
Revision history for this message
Galen Charlton (gmc) wrote :

Works for me. Pushed to master, rel_3_9, and rel_3_8 along with a follow-up to extend the fix to include item tags. Thanks, Bill!

Changed in evergreen:
status: Confirmed → Fix Committed
assignee: Galen Charlton (gmc) → nobody
Revision history for this message
Bill Erickson (berick) wrote :

Regression from this bug fixed in bug #1955065

Revision history for this message
Galen Charlton (gmc) wrote :

And a regression introduced by my follow-up fixed in bug 1981095

Changed in evergreen:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.