[Koha-bugs] [Bug 14066] Koha code does not really use Readonly

bugzilla-daemon at bugs.koha-community.org bugzilla-daemon at bugs.koha-community.org
Tue Apr 28 07:02:11 CEST 2015


http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14066

--- Comment #15 from Marc Véron <veron at veron.ch> ---
Created attachment 38593
  -->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=38593&action=edit
[Signed-off] Bug 14066: Correct and optimize

As far as I can tell $sunday is never used. And it seemed to be
scoped to only _init.

The remaining hashiness was cleaned up for the $return_by_hour.

TEST PLAN
---------
1) Apply both patches
2) (Skip this one - file does not exist / MV) prove -v
t/db_dependent/Calendar.t
3) prove -v t/db_dependent/Holidays.t
4) prove -v t/Calendar.t
5) perlcritic -3 Koha/Calendar.pm
   -- the message corresponds to a comment in the code already
      and not this change.
6) run koha qa test tools

Signed-off-by: Mark Tompsett <mtompset at hotmail.com>

Signed-off-by: Marc Véron <veron at veron.ch>

-- 
You are receiving this mail because:
You are watching all bug changes.


More information about the Koha-bugs mailing list