Angular circ modifier admin fm-editor needs some required fields and a dropdown for SIP2 Media Type

Bug #1860466 reported by Jane Sandberg
40
This bug affects 7 people
Affects Status Importance Assigned to Milestone
Evergreen
Confirmed
Medium
Unassigned

Bug Description

It is pretty easy to fail at creating a new Circulation Modifier in the new Angular interface. Code, Name, Description, and SIP2 Media Type are all required, but they are not indicated as such. These four fields should be marked as required in the IDL. Bill Erickson gives some great instructions for doing this here: https://bugs.launchpad.net/evergreen/+bug/1807461/comments/2

Additionally, the SIP2 Media Type field should have a dropdown of possible values (like the Dojo interface did), so that users don't have to intuit these values when creating new circ modifiers.

Revision history for this message
Jane Sandberg (sandbergja) wrote :

FWIW, here's the dropdown from the dojo version: https://github.com/evergreen-library-system/Evergreen/blob/master/Open-ILS/src/templates/conify/global/config/circ_modifier.tt2#L6

It looks like the SIP code (e.g. '001', '005') is all that gets stored in the database. Then the dojo has some code to make sure that the grid and editor both show a human-readable name for that SIP2 Media Type.

One solution would be to add the SIP codes and their human-readable descriptions to their own table in the db in an i18n-friendly way, and set up a link in the IDL, so that this grid can still be cleanly auto-generated, and display all the right stuff to the user.

Revision history for this message
Andrea Neiman (aneiman) wrote :

Confirmed in 3.3 and 3.4, with the addendum that do we want to require SIP2 media type? IIRC that was not required in the past.

Changed in evergreen:
status: New → Confirmed
importance: Undecided → Medium
tags: added: usability
Revision history for this message
Jennifer Pringle (jpringle-u) wrote :

This continues to be an issue in 3.11-pre-release

Revision history for this message
Jennifer Pringle (jpringle-u) wrote :

Also, Magnetic Media must be Yes or No so the Unset option for that field should be removed.

If you don't select Yes or No Evergreen will set it to Yes when you save the circulation modifier.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

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