On the main ILL Requests page that lists all requests ("illlist" action), the "List request" button is displayed, which makes no sense since we're already listing requests.
I find that button quite handy as a way to refresh the page. :-)
Created attachment 86839 [details] [review] Bug 22099: Fixed ILL toolbar to say Refresh when displaying requests On the ILL requests page that lists all requests, the List Request button is displayed as part of the ILL toolbar. This patch fixes this by instead changing the wording of the button to Refresh when on the main page. Test plan: 1) More -> ILL Requests, note that the List Requests button displays while all the requests are already displayed 2) Apply the patch and restart things 3) More -> ILL Requests, note that the button now says Refresh, and clicking it refreshes the page 4) Check that on all the other functions of the ILLs the button says List Requests 5) Sign off! Sponsored-by: Catalyst IT
Created attachment 90029 [details] [review] Bug 22099: Fixed ILL toolbar to say Refresh when displaying requests On the ILL requests page that lists all requests, the List Request button is displayed as part of the ILL toolbar. This patch fixes this by instead changing the wording of the button to Refresh when on the main page. Test plan: 1) More -> ILL Requests, note that the List Requests button displays while all the requests are already displayed 2) Apply the patch and restart things 3) More -> ILL Requests, note that the button now says Refresh, and clicking it refreshes the page 4) Check that on all the other functions of the ILLs the button says List Requests 5) Sign off! Sponsored-by: Catalyst IT Signed-off-by: Magnus Enger <magnus@libriotech.no> Small conflict resolved. Works as advertised.
Created attachment 90137 [details] [review] Bug 22099: Fixed ILL toolbar to say Refresh when displaying requests On the ILL requests page that lists all requests, the List Request button is displayed as part of the ILL toolbar. This patch fixes this by instead changing the wording of the button to Refresh when on the main page. Test plan: 1) More -> ILL Requests, note that the List Requests button displays while all the requests are already displayed 2) Apply the patch and restart things 3) More -> ILL Requests, note that the button now says Refresh, and clicking it refreshes the page 4) Check that on all the other functions of the ILLs the button says List Requests 5) Sign off! Sponsored-by: Catalyst IT Signed-off-by: Magnus Enger <magnus@libriotech.no> Small conflict resolved. Works as advertised. Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Not sure if this should be enhancement instead of trivial.
Nice work! Pushed to master for 19.11.00
Pushed to 19.05.x for 19.05.01
backported to 18.11.x for 18.11.07