http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=6403 Paul Poulain <paul.poulain@biblibre.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED CC| |paul.poulain@biblibre.com Patch Status|Signed Off |Passed QA --- Comment #6 from Paul Poulain <paul.poulain@biblibre.com> 2011-10-07 14:52:37 UTC --- QA comment * good test plan * works as expected * nothing to say about the code itself My only comment could be that some libraries consider saving such returns as a bug, so we could set the default value of the new syspref to 1 directly. But we've a rule to say a new syspref sould not change the behaviour for libraries that don't do anything with it. SO : * passed QA * it's worth being very clear in the release note that most libraries will probably want to set this value to 1 after upgrading -- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA Contact for the bug.