Bug 10884: circulation table: fix odd behavior on checkboxes

Test plan:
Play with renew, check in and export checkboxes.
The expected behavior is: the renew and check in cb should work as radio
button, without affecting the export cb.

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Template changes only.
Amended test plan:
* Create a CSV profile under Tools > CSV profiles
* Add CSV profile to system preference ExportWithCsvProfile

1) Check export checkbox
2) Check check in for same item
3) Switch to Renew

The export checkbox should not get unchecked when switching
between renew and check in.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
This commit is contained in:
Jonathan Druart 2013-09-13 16:28:39 +02:00 committed by Galen Charlton
parent 0f5dc609e0
commit fa3903fe3d
1 changed files with 5 additions and 12 deletions

View File

@ -80,14 +80,9 @@ $(document).ready(function() {
// Clicking the table cell checks the checkbox inside it
$("td").on("click",function(e){
if(e.target.tagName.toLowerCase() == 'td'){
$(this).find("input:checkbox:visible").each( function() {
if($(this).attr("checked")){
$(this).removeAttr("checked");
} else {
$(this).attr("checked","checked");
radioCheckBox($(this));
}
});
$(this).find("input:checkbox:visible").each( function() {
$(this).click();
});
}
});
});
@ -133,9 +128,7 @@ function validate1(date) {
// prevent adjacent checkboxes from being checked simultaneously
function radioCheckBox(box){
box.parents("td").siblings().find("input:checkbox:visible").each(function(){
if($(this).attr("checked")){
$(this).removeAttr("checked");
}
box.parents("td").siblings().find("input:checkbox.radio").each(function(){
$(this).removeAttr("checked");
});
}