Bug 13200

Summary: Followup of Bug 12246 - noisy C4/Auth.pm
Product: Koha Reporter: Mark Tompsett <mtompset>
Component: System AdministrationAssignee: Mark Tompsett <mtompset>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: normal    
Priority: P5 - low CC: gmcharlt, martin.renvoize, tomascohen
Version: Main   
Hardware: All   
OS: All   
See Also: http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12246
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on: 12246    
Bug Blocks:    
Attachments: Bug 13200 - Followup of Bug 12246 - noisy C4/Auth.pm
[SIGNED OFF] Bug 13200 - Followup of Bug 12246 - noisy C4/Auth.pm
Bug 13200 - Followup of Bug 12246 - noisy C4/Auth.pm

Description Mark Tompsett 2014-11-05 08:01:07 UTC
OpacMainUserBlockMobile is still pushed as a parameter in C4::Auth. This generates a noisy warning. This patch will remove the parameter in question.
Comment 1 Mark Tompsett 2014-11-05 08:14:19 UTC Comment hidden (obsolete)
Comment 2 Martin Renvoize 2014-11-05 08:31:51 UTC Comment hidden (obsolete)
Comment 3 Martin Renvoize 2014-11-05 08:32:56 UTC
Super Simple Patch.. works as expected :)
Comment 4 Jonathan Druart 2014-11-05 15:30:09 UTC
Created attachment 33249 [details] [review]
Bug 13200 - Followup of Bug 12246 - noisy C4/Auth.pm

While testing a bug, warnings in the opac error log were
building up due to a particular line in C4::Auth. After
reviewing the code, it was discovered that removal of the
OpacMainUserBlockMobile system preference created this.

Since the system preference no longer exists, and is not
used, the line was deleted from C4/Auth.pm to prevent this
warning from occuring.

TEST PLAN
----------
1) Go to any OPAC page.
2) Check your opac error log.
   -- there should be something about uninitialized values
      used in C4/Auth.pm around line 443.
3) Apply the patch
4) Refresh the page.
   -- that same error should not be triggered.
5) prove -v t/db_dependent/Auth.t
   -- this runs the get_template_and_user function
      which had the parameter removed.
6) run the koha qa test tools

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Comment 5 Tomás Cohen Arazi 2014-11-07 18:22:09 UTC
Patch pushed to master.

Thanks Mark!