[Koha-bugs] [Bug 8826] New: Initialization of hash triggers error log entries.

[bugzilla-daemon at bugs.koha-community.org]

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

          Priority: P5 - low
 Change sponsored?: ---
            Bug ID: 8826
                CC: gmcharlt at gmail.com, kyle.m.hall at gmail.com
          Assignee: koha-bugs at lists.koha-community.org
           Summary: Initialization of hash triggers error log entries.
          Severity: enhancement
    Classification: Unclassified
                OS: All
          Reporter: mtompset at hotmail.com
          Hardware: All
            Status: NEW
           Version: unspecified
         Component: Circulation
           Product: Koha

Bug 7851 initialized a hash incorrectly.
our %renew_failed = {};
This attempts to create a hash of a single anonymous hash reference which
generates the error:
circulation.pl: Reference found where even-sized list expected at
/usr/share/koha/intranet/cgi-bin/circ/circulation.pl line 97.

As I do not know how my proposed change of:
our %renew_failed = ();
will work in a plack context, I would need someone to assist in testing that.

I was thinking of reverting to no initialization, but based on my reading of
bug 7851, that isn't an option.

In the worst case scenario:
our %renew_failed = ( "dummyplaceholderkey" => "dummyplaceholdervalue" );
should work.

This only applies to 3.8.x and master, as 3.6.x still has an uninitialized
value (which should not be problematic as far as I can tell).

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