https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16219

--- Comment #5 from Nick Clemens <n...@bywatersolutions.com> ---
(In reply to Katrin Fischer from comment #3)
> This works, but feels a little hacky:
> - When the report expects params, the SQL message is output. We could
> possibly print a nicer error message instead with the number of params to
> expected.

Ok, done

> - When the report is built with reusable variables, you need to enter each
> of those on command line. If not easy to fix, hinting at it in the help
> might be good.

Noted in the POD, really reports needs an update and move to objects, this is a
helpful patch mostly for sysadmins (and to prevent libraries having to have
hardcoded cron version reports and staff side version with params)

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to