Bug 8904

Summary: Plack variable scoping in tools/overduerules
Product: Koha Reporter: Paul Poulain <paul.poulain>
Component: ToolsAssignee: Paul Poulain <paul.poulain>
Status: CLOSED FIXED QA Contact: Mason James <mtj>
Severity: blocker    
Priority: P5 - low CC: chris, mtj
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on:    
Bug Blocks: 7172    
Attachments: Bug 5904 fix Plack scoping for overduerules.pl
Bug 8904: fix Plack scoping for overduerules.pl

Description Paul Poulain 2012-10-11 13:11:38 UTC
It's impossible to save overduerules when you're running plack. Plack logs say:

Variable "@rule_params" is not available at /home/paul/koha.dev/koha-community/tools/overduerules.pl line 44.
Variable "$input" is not available at /home/paul/koha.dev/koha-community/tools/overduerules.pl line 52.
Comment 1 Paul Poulain 2012-11-02 08:54:23 UTC Comment hidden (obsolete)
Comment 2 Paul Poulain 2012-11-02 08:55:02 UTC
Upgrading severity to BLO, because it's a blocker for Plack
Comment 3 Jared Camins-Esakov 2012-11-02 11:27:36 UTC
Created attachment 13198 [details] [review]
Bug 8904: fix Plack scoping for overduerules.pl

There are 2 Plack scoping errors in tools/overduerules.pl:
the local sub blank_row require that @rule_params and input are global

Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>
Comment 4 Mason James 2012-11-05 00:42:12 UTC
(In reply to comment #3)
> Created attachment 13198 [details] [review]
> Bug 8904: fix Plack scoping for overduerules.pl
> 
> There are 2 Plack scoping errors in tools/overduerules.pl:
> the local sub blank_row require that @rule_params and input are global
> 
> Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>

patch looks good and works as stated, passing QA...

$ koha-qa.pl -c 1

testing 1 commit(s) (applied to commit dcf4a9b)
 * fa2cf71 Bug 8904: fix Plack scoping for overduerules.pl
      tools/overduerules.pl

* tools/overduerules.pl                                                    OK
Comment 5 Jared Camins-Esakov 2012-11-05 00:59:36 UTC
This patch has been pushed to master.
Comment 6 Paul Poulain 2012-11-05 08:52:10 UTC
Patch pushed to branch 3.10.x
Comment 7 Chris Cormack 2013-04-21 11:09:45 UTC
Released in 3.10.0