Bug 6503

Summary: Variable $sth is declared 3 times (with keyword "my") in "makepayment" sub of "Accounts.pm"
Product: Koha Reporter: Ricardo Dias Marques <ricmarques>
Component: PatronsAssignee: Kyle M Hall <kyle.m.hall>
Status: CLOSED FIXED QA Contact: Bugs List <koha-bugs>
Severity: minor    
Priority: PATCH-Sent (DO NOT USE) CC: chris, colin.campbell, gmcharlt, koha.sekjal, ricmarques
Version: Main   
Hardware: All   
OS: All   
GIT URL: Change sponsored?: ---
Patch complexity: --- Documentation contact:
Documentation submission: Text to go in the release notes:
Version(s) released in:
Circulation function:
Attachments: Patch to fix Bug 6503
Bug 6503 - Variable $sth is declared 3 times (with keyword "my") in "makepayment" sub of "Accounts.pm"

Description Ricardo Dias Marques 2011-06-14 15:59:22 UTC
In the code available today (14-Jun-2011), in Koha git's "master" branch, the file "Accounts.pm" (in the "C4" directory) has the variable "$sth" declared 3 times in the "makepayment" sub. 

This causes the following warnings to appear when running "make test" in Koha installation, for several tests:

t/Heading.t ......................... "my" variable $sth masks earlier declaration in same scope at kohaclone/blib/PERL_MODULE_DIR/C4/Accounts.pm line 172.
"my" variable $sth masks earlier declaration in same scope at kohaclone/blib/PERL_MODULE_DIR/C4/Accounts.pm line 188.
[...]
t/ILSDI_Services.t .................. "my" variable $sth masks earlier declaration in same scope at kohaclone/blib/PERL_MODULE_DIR/C4/Accounts.pm line 172.
"my" variable $sth masks earlier declaration in same scope at kohaclone/blib/PERL_MODULE_DIR/C4/Accounts.pm line 188.

[...etc...]


Therefore, I'm removing the "my" keyword from the declarations of "$sth" in lines 172 and 188 of "Accounts.pm".

Patch will follow shortly.
Comment 1 Ricardo Dias Marques 2011-06-14 16:32:29 UTC
Created attachment 4453 [details]
Patch to fix Bug 6503

Proposed patch in the above link to fix this bug 6503.
Comment 2 Chris Cormack 2011-06-14 22:44:28 UTC
Created attachment 4467 [details] [review]
Bug 6503 - Variable $sth is declared 3 times (with keyword "my") in "makepayment" sub of "Accounts.pm"

In the code available today (14-Jun-2011), in Koha git's "master" branch,
the file "Accounts.pm" (in the "C4" directory) has the variable "$sth"
declared 3 times in the "makepayment" sub. This causes the following
warnings to appear when running "make test" in Koha installation, for
several tests:
t/Heading.t ......................... "my" variable $sth masks earlier declaration
in same scope at kohaclone/blib/PERL_MODULE_DIR/C4/Accounts.pm line 172.
"my" variable $sth masks earlier declaration in same scope at
kohaclone/blib/PERL_MODULE_DIR/C4/Accounts.pm line 188.
[...]
t/ILSDI_Services.t .................. "my" variable $sth masks earlier declaration
in same scope at kohaclone/blib/PERL_MODULE_DIR/C4/Accounts.pm line 172.
"my" variable $sth masks earlier declaration in same scope
at kohaclone/blib/PERL_MODULE_DIR/C4/Accounts.pm line 188.
[...etc...]

Therefore, I'm removing the "my" keyword from the declarations of "$sth"
in lines 172 and 188 of "Accounts.pm".

Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
Comment 3 Ian Walls 2011-07-15 02:54:19 UTC
Simple change, easy to read, marking as Passed QA
Comment 4 Chris Cormack 2011-07-17 08:42:30 UTC
Pushed, please test
Comment 5 Magnus Enger 2011-07-28 09:09:25 UTC
*** Bug 6469 has been marked as a duplicate of this bug. ***
Comment 6 Jared Camins-Esakov 2012-12-31 00:02:57 UTC
There have been no further reports of problems so I am marking this bug resolved.