Bug 10126: Remove my variables at module level
authorJonathan Druart <jonathan.druart@biblibre.com>
Thu, 9 Oct 2014 15:16:34 +0000 (17:16 +0200)
committerTomas Cohen Arazi <tomascohen@gmail.com>
Wed, 15 Oct 2014 21:21:14 +0000 (18:21 -0300)
commita7f23c0afca81de0b22252650a9a23c8d8b731d9
tree96d6ddff2af255fbef179515d2c7079fb883f5d1
parentcb4b7f7c810adbb5212725e1f68e01cbed98d9e4
Bug 10126: Remove my variables at module level

In C4::Reports::Guided, a lot of variables was defined at module level
and reused in subroutine.

I didn't find any problem with Plack, so I'm not sure this patch is
useful.

Test plan:
Verify there is no more ^my in the module and you don't find any
regression with the guided reports tools (with and without Plack)

Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
I'm unsure if this is needed, but I have verified it causes no
regressions

Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
No regressions spotted. I'd prefer this code to be fully refactored of course :-P
C4/Reports/Guided.pm