| Summary: | Remove unconditional warn from Koha::NorwegianPatronDB | ||
|---|---|---|---|
| Product: | Koha | Reporter: | Magnus Enger <magnus> | 
| Component: | Patrons | Assignee: | Magnus Enger <magnus> | 
| Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> | 
| Severity: | trivial | ||
| Priority: | P5 - low | CC: | chris, gmcharlt, indradg, jonathan.druart, kyle, liz, mtj, tomascohen | 
| Version: | Main | ||
| Hardware: | All | ||
| OS: | All | ||
| GIT URL: | Initiative type: | --- | |
| Sponsorship status: | --- | Crowdfunding goal: | 0 | 
| Patch complexity: | Trivial patch | Documentation contact: | |
| Documentation submission: | Text to go in the release notes: | ||
| Version(s) released in: | Circulation function: | ||
| 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
        
       Created attachment 40233 [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. Created attachment 40236 [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> 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> Patch pushed to master. Good catch Magnus! Pushed to 3.20.x will be in 3.20.2 Pushed to 3.18.x will be in 3.18.09 Skipping for 3.16.x series, due to patch conflicts |