[Discovered during testing the patch for bug 10884] If circulation exports are enabled (by turning on ExportWithCsvProfile), the table on the checkout page includes three columns of checkboxes -- 'renew', 'checkin', and 'export'. For each loan, the renew and checkout links should behave like radio buttons, but the state of the export checkbox is meant to be independent of the renew and checkin checkboxes. However, if the 'select all' link in the export column is clicked, active renew checkboxes are toggled off. The desired behavior is that clicking the select all link in the export column should only affect checkboxes in that column.
Created attachment 23070 [details] [review] Bug 11075 - 'select all' export clears renew checkboxes If circulation exports are enabled (by turning on ExportWithCsvProfile), the table on the checkout page includes three columns of checkboxes -- 'renew', 'checkin', and 'export'. For each loan, the renew and checkout links should behave like radio buttons, but the state of the export checkbox is meant to be independent of the renew and checkin checkboxes. However, if the 'select all' link in the export column is clicked, active renew checkboxes are toggled off. The desired behavior is that clicking the select all link in the export column should only affect checkboxes in that column.
Galen, I took the liberty of copying your description in the commit message.
Created attachment 23850 [details] [review] Bug 11075 - 'select all' export clears renew checkboxes If circulation exports are enabled (by turning on ExportWithCsvProfile), the table on the checkout page includes three columns of checkboxes -- 'renew', 'checkin', and 'export'. For each loan, the renew and checkout links should behave like radio buttons, but the state of the export checkbox is meant to be independent of the renew and checkin checkboxes. However, if the 'select all' link in the export column is clicked, active renew checkboxes are toggled off. The desired behavior is that clicking the select all link in the export column should only affect checkboxes in that column. Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
Created attachment 23856 [details] [review] [PASSED QA] Bug 11075 - 'select all' export clears renew checkboxes If circulation exports are enabled (by turning on ExportWithCsvProfile), the table on the checkout page includes three columns of checkboxes -- 'renew', 'checkin', and 'export'. For each loan, the renew and checkout links should behave like radio buttons, but the state of the export checkbox is meant to be independent of the renew and checkin checkboxes. However, if the 'select all' link in the export column is clicked, active renew checkboxes are toggled off. The desired behavior is that clicking the select all link in the export column should only affect checkboxes in that column. Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz> Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> Passes all tests and QA script - one line JavaScript change. Works as described.
Pushed to master. Thanks, Jonathan!
Patch pushed to 3.14.x, will be in 3.14.3.