]> git.koha-community.org Git - koha.git/commit
Bug 3130 and general cleanup
authorJoe Atzberger <joe.atzberger@liblime.com>
Tue, 28 Apr 2009 23:23:36 +0000 (18:23 -0500)
committerGalen Charlton <galen.charlton@liblime.com>
Wed, 29 Apr 2009 12:29:10 +0000 (07:29 -0500)
commitde38cf6df199d6bdc52179d9d55e7a0e22e4e7d8
treea896ef74f604693d2cc4f5cc6c03bc176712a4bd
parent0501737f1c9b20ca1cd5c9388a0b058cc3cb3008
Bug 3130 and general cleanup

Javascript error in FF and IE when loading page and no data matches criteria.
This was because it was passing more holds than necessary to the template and
letting a template conditional inside the loop control whether to display or not.
That doesn't make sense, and it forces the table to be displayed even when all
rows FAIL the conditional.  jquery.tablesorter was confused trying to be added
on top of an empty table with nothing to sort.

There were several other errors addressed by this patch:
 ~ 13 unused variables deleted.
 ~ regexps run on user-supplied $var before checking $var defined
 ~ decimal ratio prohibited and silently replaced with "3"

I also added the hold ratio to a column display, with a jquery tweak to put it in
the ratio input box on click.  Hidden .sql div now contains the actual query run,
like the other wizard reports.

Signed-off-by: Galen Charlton <galen.charlton@liblime.com>
circ/reserveratios.pl
koha-tmpl/intranet-tmpl/prog/en/modules/circ/reserveratios.tmpl