Fine level and loan duration aren't displayed when editing items
Bug #1741072 reported by
Jason Boyer
This bug affects 3 people
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Evergreen |
Fix Released
|
Medium
|
Unassigned |
Bug Description
Eg 3.X
In the web client item editor the loan duration and fine level fields are hard coded as select tags so that we can manually map the numbers to translatable strings. While this appears to work when initially creating an item or applying a template (provided these numbers are stored as strings in the template...) these fields don't display their current values when editing an existing item. This obviously leads to staff confusion and questioning whether the values were set correctly initially.
Changed in evergreen: | |
status: | New → Confirmed |
assignee: | nobody → Bill Erickson (berick) |
Changed in evergreen: | |
status: | Incomplete → Confirmed |
Changed in evergreen: | |
assignee: | nobody → Bill Erickson (berick) |
Changed in evergreen: | |
assignee: | nobody → Galen Charlton (gmc) |
importance: | Undecided → Medium |
tags: | added: signedoff |
Changed in evergreen: | |
status: | Fix Committed → Fix Released |
To post a comment you must log in.
The simple fix for this is to add a new directive to the ui service to convert the model from a number to a string. Since this kind of thing has come up in previous bugs (bug 1713064 and bug 1714390, which conflicted with bug 1712646 because a similar but not identical fix is required) I plan to address it by adding 2 pairs of symmetric directives to the shared egUiMod module; str-to-int and int-to-str, and str-to-float and float-to-str. Additionally the current uses of string-to-number in staff/cat/ volcopy/ app.js will be converted to the appropriate shared directive and the old string-to-number will be removed.