Bug 16452 - PatronLists.t raises a warning
Summary: PatronLists.t raises a warning
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Architecture, internals, and plumbing (show other bugs)
Version: Main
Hardware: All All
: P5 - low minor (vote)
Assignee: Jonathan Druart
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-05-05 14:36 UTC by Jonathan Druart
Modified: 2016-12-05 21:25 UTC (History)
3 users (show)

See Also:
Change sponsored?: ---
Patch complexity: Trivial patch
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments
Bug 16452: Remove the warnings raised by PatronLists.t (963 bytes, patch)
2016-05-05 14:37 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 16452: Remove the warnings raised by PatronLists.t (1.08 KB, patch)
2016-05-05 14:59 UTC, Marc Véron
Details | Diff | Splinter Review
[PASSED QA] Bug 16452: Remove the warnings raised by PatronLists.t (1.14 KB, patch)
2016-05-05 19:53 UTC, Katrin Fischer
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Jonathan Druart 2016-05-05 14:36:47 UTC
% prove t/db_dependent/PatronLists.t
t/db_dependent/PatronLists.t .. 1/9 C4::Context->userenv not defined! at Koha/List/Patron.pm line 66.
t/db_dependent/PatronLists.t .. 3/9 C4::Context->userenv not defined! at Koha/List/Patron.pm line 66.
t/db_dependent/PatronLists.t .. 5/9 C4::Context->userenv not defined! at Koha/List/Patron.pm line 66.
C4::Context->userenv not defined! at Koha/List/Patron.pm line 94.
C4::Context->userenv not defined! at Koha/List/Patron.pm line 94.
C4::Context->userenv not defined! at Koha/List/Patron.pm line 66.
t/db_dependent/PatronLists.t .. ok   
All tests successful.
Files=1, Tests=9,  2 wallclock secs ( 0.02 usr  0.00 sys +  1.00 cusr  0.02 csys =  1.04 CPU)
Result: PASS
Comment 1 Jonathan Druart 2016-05-05 14:37:57 UTC Comment hidden (obsolete)
Comment 2 Marc Véron 2016-05-05 14:59:02 UTC Comment hidden (obsolete)
Comment 3 Katrin Fischer 2016-05-05 19:53:02 UTC
Created attachment 51257 [details] [review]
[PASSED QA] Bug 16452: Remove the warnings raised by PatronLists.t

We need to define a userenv to get rid of the warnings

Test plan:
  prove t/db_dependent/PatronLists.t
should not return any warnings

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

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 4 Brendan Gallagher 2016-05-05 20:45:42 UTC
Pushed to Master - Should be in the May 2016 Release.  

Thanks!
Comment 5 Julian Maurice 2016-05-16 11:47:37 UTC
Patch pushed to 3.22.x, will be in 3.22.7