Error messages in the OPAC reserves tool need to include unique IDs so the text can be easily customized with JQuery.
Work To Be Done:
* Add id attributes to all id-less div tags containing holds error messages, including:
How to test:
* Create a test patron
* For each message, supply the necessary criteria to trigger it. Attempt to place a hold...
-- too_much_oweing: ...when the test patron owes too many fines, as per system preferences
-- too_many_reserves: ...whenthe test patron already has more reserves than allowed by circulation rules
-- bad_biblionumber: ...on a biblio that doesn’t exist
-- no_items_selected: ...when no items are selected
-- no_branch_selected: ...without selecting a pickup branch
-- no_biblionumber: ...without supplying any biblionumber
-- bad_data: ...on multiple titles, without supplying either a biblionumber, itemnumber or branchcode for each title
-- none_available: ...on a title with no available items
* If the HTML source of the page includes an id for each div tag that wraps the error message, then the test is passed.
Created attachment 4740 [details] [review]
Simple html changes, pushing
Patch is pushed and working, closing bug.