[koha-commits] main Koha release repository branch master updated. v3.22.00-723-g0fdc102
Git repo owner
gitmaster at git.koha-community.org
Wed Mar 2 04:54:44 CET 2016
This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "main Koha release repository".
The branch, master has been updated
via 0fdc1021d4477d838c6b38a1172137a5eec08978 (commit)
from 10a9a1e147699bdb1fd2b58ab93fe87cb82074cf (commit)
Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.
- Log -----------------------------------------------------------------
commit 0fdc1021d4477d838c6b38a1172137a5eec08978
Author: Jonathan Druart <jonathan.druart at bugs.koha-community.org>
Date: Wed Feb 10 15:37:30 2016 +0000
Bug 15800: Koha::AuthorisedValues - Remove C4::Koha::IsAuthorisedValueCategory
C4::Koha::IsAuthorisedValueCategory contains only 2 useful calls, from
C4::Reports::Guided and reports/guided_reports.pl
It can be replaced with
Koha::AuthorisedValues->search({ category => $authorised_value})->count
Test plan:
1/ Create a sql report using an authorised value category, something
like:
SELECT COUNT(*) FROM items where itemlost=<<lost|LOST>>
2/ Execute the report and confirm that everything works fine.
3/ Create a sql report using a nonexistent authorised value categor,
something like:
SELECT COUNT(*) FROM items where itemlost=<<lost|NONEXIST>>
4/ When saving the report, you should get a warning message
"lost: The authorized value category (NONEXIST) you selected does not exist."
5/ Save anyway and execute the report, you should get the same warning
message.
QA:
git grep IsAuthorisedValueCategory
should not return any results
prove t/db_dependent/ReportsGuided.t
should return green
Signed-off-by: Hector Castro <hector.hecaxmmx at gmail.com>
Works as described
Signed-off-by: Kyle M Hall <kyle at bywatersolutions.com>
Signed-off-by: Brendan Gallagher brendan at bywatersolutions.com
-----------------------------------------------------------------------
Summary of changes:
C4/Koha.pm | 23 -----------------------
C4/Reports/Guided.pm | 4 +++-
reports/guided_reports.pl | 5 +++--
t/Koha.t | 22 +++++-----------------
t/db_dependent/ReportsGuided.t | 27 +++++----------------------
5 files changed, 16 insertions(+), 65 deletions(-)
hooks/post-receive
--
main Koha release repository
More information about the koha-commits
mailing list