Bug 9769

Summary: Replace YUI buttons on acquisitions basket page with Bootstrap
Product: Koha Reporter: Owen Leonard <oleonard>
Component: TemplatesAssignee: Owen Leonard <oleonard>
Status: CLOSED FIXED QA Contact: Katrin Fischer <katrin.fischer>
Severity: enhancement    
Priority: P5 - low CC: brendan, katrin.fischer, mathsabypro
Version: Main   
Hardware: All   
OS: All   
URL: /cgi-bin/koha/acqui/basket.pl?basketno=
Change sponsored?: --- Patch complexity: Small patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on:    
Bug Blocks: 9779    
Attachments: Bug 9769 - Replace YUI buttons on acquisitions basket page with Bootstrap
Bug 9769 [Revised] Replace YUI buttons on acquisitions basket page with Bootstrap
Bug 9769 [Revised] Replace YUI buttons on acquisitions basket page with Bootstrap
[PASSED QA] Bug 9769 [Revised] Replace YUI buttons on acquisitions basket page with Bootstrap
[SIGNED-OFF] Bug 9679 - can't place hold from cart in CCSR

Description Owen Leonard 2013-03-08 14:45:43 UTC
The toolbar buttons on the acquisitions basket page are constructed using YUI. They should use Bootstrap instead.
Comment 1 Owen Leonard 2013-03-08 14:58:04 UTC Comment hidden (obsolete)
Comment 2 Mathieu Saby 2013-03-15 17:57:35 UTC
I cannot reopen a closed basket. All the rest is correct.

M. Saby
Comment 3 Owen Leonard 2013-03-18 13:49:40 UTC Comment hidden (obsolete)
Comment 4 Brendan Gallagher 2013-03-18 15:40:27 UTC Comment hidden (obsolete)
Comment 5 Brendan Gallagher 2013-03-18 15:41:02 UTC
Followed test plan and this worked.
Comment 6 Katrin Fischer 2013-03-21 08:46:48 UTC
Created attachment 16573 [details] [review]
[PASSED QA] Bug 9769 [Revised] Replace YUI buttons on acquisitions basket page with Bootstrap

Toolbar buttons on the acquisitions basket page are constructed by YUI.
They should use Bootstrap instead.

Other minor changes: Fixes for invalid markup.

To test, view baskets in various states: Empty, open closed. View
baskets in groups and not in groups. Buttons should look correct and
work correctly:

- Edit
- Delete
- Close
- Reopen
- Export

Revision corrects check for 'skip_confirm_reopen' variable when
reopening baskets. Please test with BasketConfirmations both on and off.

Signed-off-by: Brendan <brendan@bywatersolutions.com>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Tested all options, some notes about behaviour:
- closing a basket is only possible, when there are no uncertain prices
  (no button close basket)
- reopening a basket requires, that you first remove the basket from
  any basket group (no button to reopen basket)
Comment 7 Jared Camins-Esakov 2013-03-21 14:11:33 UTC
This patch has been pushed to master.
Comment 8 Nicole C. Engard 2013-05-10 14:17:43 UTC Comment hidden (obsolete)
Comment 9 Nicole C. Engard 2013-05-10 14:19:35 UTC
Comment on attachment 18068 [details] [review]
[SIGNED-OFF] Bug 9679 - can't place hold from cart in CCSR

wrong bug