]> git.koha-community.org Git - koha.git/commit
Bug 24083: (follow-up) Squashed follow ups
authorAndrew Isherwood <andrew.isherwood@ptfs-europe.com>
Mon, 5 Oct 2020 11:39:48 +0000 (12:39 +0100)
committerJonathan Druart <jonathan.druart@bugs.koha-community.org>
Wed, 11 Nov 2020 15:09:58 +0000 (16:09 +0100)
commit6e311a1be124cc93756183aefea7768be74849ad
tree63ee574a40aa2ec6ca2c98b2fcc1d1c4e3981253
parente429d90fb3065a7c7c4bd6951ffb9d47135aa5ce
Bug 24083: (follow-up) Squashed follow ups

This squashed commit fixes:
-  a small error in the checkouts related JS

- GetRenewCount now returns 6 values when a call to it succeeds, a
failed call should also return the same number of values. This commit
adds these additional values.

- Some changes in issue.t had broken the tests for unseen renewals (the
unseen tests were using variables that had been moved out of the tests'
scope).

- Also now using Koha::CirculationRules::set_rules to set circ rules
rather than using SQL queries.

- Fixed expected number of return values from GetRenewCount

- Moved unseen tests in issue.t to the bottom of the file to remove the
risk of interference with other test circ rules.

- There was a real mess in C4/Circulation.pm due to a bad rebase back in
February. Frankly it's a wonder anything worked at all. This commit
fixes that problem and reinstates the correct patch for
C4/Circulation.pm

- Somehow I'd never noticed this before but the columns in smart-rules.tt
were misaligned when UnseenRenewals was turned off. This was due to the
display of a <td> not being conditional when it should have been. This
is now fixed.

- This commit also fixes items 1 & 2 descibed by Katrin in comment #74 ->
comment #76.

- Fixed missing check for too_unseen in opac-user.tt, this test did used
to exist but got lost during sizeable rebase a few weeks ago :-(
- Added test for too_unseen to all AUTO_RENEWAL notice templates apart
from de-DE (as previously requested by Katrin)

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
14 files changed:
C4/Circulation.pm
installer/data/mysql/en/mandatory/sample_notices.yml
installer/data/mysql/fr-CA/obligatoire/sample_notices.sql
installer/data/mysql/fr-FR/1-Obligatoire/sample_notices.sql
installer/data/mysql/it-IT/necessari/notices.sql
installer/data/mysql/nb-NO/1-Obligatorisk/sample_notices.sql
installer/data/mysql/pl-PL/mandatory/sample_notices.sql
installer/data/mysql/ru-RU/mandatory/sample_notices.sql
installer/data/mysql/uk-UA/mandatory/sample_notices.sql
koha-tmpl/intranet-tmpl/prog/en/modules/admin/smart-rules.tt
koha-tmpl/intranet-tmpl/prog/js/checkouts.js
koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-user.tt
t/db_dependent/Circulation.t
t/db_dependent/Circulation/issue.t