Bug 13293 - Regression: Override renewal limit option broken with AJAX circ
commit24a749d7ed85dc97d899448df24d7469b029ce3f
authorKyle M Hall <kyle@bywatersolutions.com>
Fri, 21 Nov 2014 13:13:46 +0000 (21 08:13 -0500)
committerTomas Cohen Arazi <tomascohen@gmail.com>
Sat, 22 Nov 2014 14:15:33 +0000 (22 11:15 -0300)
treeaa3aaa2e53eddfe4964b8c8723701bd2505bdfb9
parentaeb86b69ebc40e03ee9b56c0c9d99287db140559
Bug 13293 - Regression: Override renewal limit option broken with AJAX circ

The "Override renewal limit" checkbox no longer works to un-hide the
renewal checkboxes in the table of checkouts. I assume this is because
the script works in such a way that it is looking for the checkboxes
before they are part of the document.

Test Plan:
1) Apply this patch
2) Check the "Override renewal limit" checkbox
3) Note the checkbox now appears

Note, this change allows the "X of Y renewals remaining" to be displayed
for non-renewable items. I left this as it is for two reasons: 1) it
gives the librarian more information that may be useful, and 2) it looks
nicer and more uniform.

Signed-off-by: Frederic Demians <f.demians@tamil.fr>
I confirm the bug and the solution.

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Passes tests and QA script, works as described.
Exception: on-site checkouts can't be renewed.

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
koha-tmpl/intranet-tmpl/prog/en/js/checkouts.js