Bug 14403

Summary: Remove unconditional warn from Koha::NorwegianPatronDB
Product: Koha Reporter: Magnus Enger <magnus>
Component: PatronsAssignee: Magnus Enger <magnus>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: trivial    
Priority: P5 - low CC: chris, gmcharlt, indradg, jonathan.druart, kyle.m.hall, liz, mtj, tomascohen
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:
Attachments: Bug 14403 - Remove warn in Koha::NorwegianPatronDB
Bug 14403 - Remove warn in Koha::NorwegianPatronDB
Bug 14403 - Remove warn in Koha::NorwegianPatronDB

Description Magnus Enger 2015-06-17 12:35:58 UTC
Line 99 og Koha/NorwegianPatronDB.pm looks like this:     

  warn "$combined_username => $combined_password";

This is clearly a leftover from development and should be deleted.
Comment 1 Magnus Enger 2015-06-17 12:39:35 UTC Comment hidden (obsolete)
Comment 2 Indranil Das Gupta 2015-06-17 12:49:25 UTC Comment hidden (obsolete)
Comment 3 Jonathan Druart 2015-06-18 08:56:36 UTC
Created attachment 40259 [details] [review]
Bug 14403 - Remove warn in Koha::NorwegianPatronDB

Line 99 has an unconditional warn, left over from development:

warn "$combined_username => $combined_password";

This patch deletes the line i question.

To test:
No testing needed, just have a look at the diff and see that
it makes sense to delete the warn.

Signed-off-by: Indranil Das Gupta (L2C2 Technologies) <indradg@gmail.com>

Signed-off-by: Jonathan Druart <jonathan.druart@koha-community.org>
Comment 4 Tomás Cohen Arazi 2015-06-19 14:48:40 UTC
Patch pushed to master.

Good catch Magnus!
Comment 5 Chris Cormack 2015-06-29 09:36:15 UTC
Pushed to 3.20.x will be in 3.20.2
Comment 6 Liz Rea 2015-07-20 22:02:33 UTC
Pushed to 3.18.x will be in 3.18.09
Comment 7 Mason James 2015-07-25 06:42:08 UTC
Skipping for 3.16.x series, due to patch conflicts