Bug 16452

Summary:	PatronLists.t raises a warning
Product:	Koha	Reporter:	Jonathan Druart <jonathan.druart>
Component:	Architecture, internals, and plumbing	Assignee:	Jonathan Druart <jonathan.druart>
Status:	CLOSED FIXED	QA Contact:	Testopia <testopia>
Severity:	minor
Priority:	P5 - low	CC:	brendan, julian.maurice, veron
Version:	Main
Hardware:	All
OS:	All
Change sponsored?:	---	Patch complexity:	Trivial patch
Documentation contact:		Documentation submission:
Text to go in the release notes:		Version(s) released in:
Circulation function:
Attachments:	Bug 16452: Remove the warnings raised by PatronLists.t Bug 16452: Remove the warnings raised by PatronLists.t [PASSED QA] Bug 16452: Remove the warnings raised by PatronLists.t

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)

Created attachment 51240 [details] [review]
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

Comment 2 Marc Véron 2016-05-05 14:59:02 UTC Comment hidden (obsolete)

Created attachment 51245 [details] [review]
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>

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