Patron Editor can enter erroneous values for Claims-returned
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Evergreen |
Fix Released
|
Medium
|
Unassigned | ||
2.2 |
Won't Fix
|
Medium
|
Unassigned | ||
2.3 |
Fix Released
|
Medium
|
Unassigned | ||
2.4 |
Fix Released
|
Medium
|
Unassigned |
Bug Description
In the staff client, Patrons interface, Edit screen, when mouse-scrolling up and down the Edit form, it is possible to inadvertently change the value in Claims-returned Count or Claims Never Checked Out Count field. It will happen if the mouse hovers over either
data field while using the mouse wheel.
Both data fields use the dijit.form.
1. The data fields are of datatype int, and int is mapped into using dijit.form.
2. If the user hovers over one of the data fields and adjusts the mousewheel, it seems the input field is auto-focussed and up and down events are then handled.
All other data fields require an explicit click to focus on the input field and it should be no different for these two data fields.
A quick fix involves cancelling the mouse scroll event as it propagates to input fields in table rows that are using the dijit.form.
P.S. We found our production database contained about a hundred patron records with unusually high count values, probably due to the problem. We identified these patron records and 'corrected' the count values.
Evergreen version 2.2
Changed in evergreen: | |
status: | New → Confirmed |
importance: | Undecided → Medium |
milestone: | none → 2.5.0-m1 |
Changed in evergreen: | |
milestone: | 2.5.0-m1 → 2.5.0-m2 |
Changed in evergreen: | |
status: | Fix Committed → Fix Released |
Works for me, thanks Steven! Picked to master, rel_2_4, and rel_2_3. Marking rel_2_2 as "won't fix" since it's outside our maintenance window for bugs now.