Course Materials Forms Should Not Allow Carriage Return to Activate the “Add Material” or “Add User” Button
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Evergreen |
Fix Released
|
Medium
|
Unassigned | ||
3.8 |
Fix Released
|
Medium
|
Unassigned |
Bug Description
When associating items with a course, the item barcode is the first element of the form. If the user scans the barcode (or types it in and presses the “enter” key), the item is added to the course immediately. This means that the user does not have the opportunity to add or edit values for the remaining fields on the form. At libraries in our consortium users will need to edit item data (e.g. shelving location, circulation modifier) for items placed on reserve.
The same behavior occurs if the user chooses to add materials using the “Associate Brief Record” option on the course materials tab. If the user presses “enter” in the Title, Uniform Resource Identifier or Link Text entry boxes, the item is added to the course and the user does not have the option to add data for the following fields.
The same behavior also occurs on the “Course Users” tab. If the user scans a barcode (or types it in and presses the “enter” key) the course user is attached to the course before the user “Role” can be selected.
This behavior is especially problematic because there is no way to edit the item / user data within the course module. See related bug: https:/
These forms should not allow a carriage return to activate the Add Material button.
Steps to test:
Open the Course Materials module
Select a course
Select the “Course Materials” tab
Select the “Associate Item” option
Scan an item barcode (or type the barcode and press the “enter” key)
Observe that the item is added to the course before the following fields can be edited: Relationship. Call Number, Circulation Modifier, Item Status, Shelving Location
EG 3-6-2
description: | updated |
Changed in evergreen: | |
assignee: | nobody → Kyle Huckins (khuckins) |
Changed in evergreen: | |
milestone: | none → 3.8.1 |
Changed in evergreen: | |
milestone: | 3.8.1 → none |
Changed in evergreen: | |
milestone: | none → 3.9.1 |
Changed in evergreen: | |
status: | Fix Committed → Fix Released |
no longer affects: | evergreen/3.7 |
I've pushed a branch resolving the issue here: https:/ /git.evergreen- ils.org/ ?p=working/ Evergreen. git;a=shortlog; h=refs/ heads/user/ khuckins/ lp1930896- course- reserves- carriage- return- shouldnt- add