Bug 14484

Summary: Warns when changing patron password
Product: Koha Reporter: Aleisha Amohia <aleisha>
Component: PatronsAssignee: Aleisha Amohia <aleisha>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: minor    
Priority: P5 - low CC: gmcharlt, jonathan.druart, kyle.m.hall, mirko, tomascohen
Version: master   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 14484: Silences warns triggered when changing patron password
Bug 14484: Jonathan Druart's fix
Bug 14484: warns when changing patron pw
Bug 14484: warns when changing patron pw
Bug 14484: warns when changing patron pw
Bug 14484: warns when changing patron pw

Description Aleisha Amohia 2015-07-01 05:40:41 UTC
To recreate, go to any patron's page and click Change Password

member-password.pl: Use of uninitialized value $roadtype in concatenation (.) or string at /home/vagrant/kohaclone/members/member-password.pl line 105
member-password.pl: Use of uninitialized value $roadtype in concatenation (.) or string at /home/vagrant/kohaclone/members/member-password.pl line 105
Comment 1 Aleisha Amohia 2015-07-01 05:42:44 UTC Comment hidden (obsolete)
Comment 2 Jonathan Druart 2015-07-02 14:36:47 UTC
I'd suggest
  my $roadtype = C4::Koha::GetAuthorisedValueByCode( 'ROADTYPE', $bor->{streettype} ) // '';
Comment 3 Aleisha Amohia 2015-07-20 04:26:54 UTC Comment hidden (obsolete)
Comment 4 Mirko Tietgen 2015-08-11 12:56:24 UTC
I get an error when I try to apply the patch:

> Applying: Bug 14484: Jonathan Druart's fix
> fatal: sha1 information is lacking or useless (members/member-password.pl).
> Repository lacks necessary blobs to fall back on 3-way merge.
> Cannot fall back to three-way merge.
> Patch failed at 0001 Bug 14484: Jonathan Druart's fix

It looks like the second patch modifies the first one, so

1) the first patch should not be obsoleted OR
2) the second one should be done against master without the first in between

the second being the better option.
Comment 5 Aleisha Amohia 2015-08-12 02:54:30 UTC Comment hidden (obsolete)
Comment 6 Mirko Tietgen 2015-08-12 11:50:10 UTC
Sorry, but the new attachment looks the same as the one before (and I get the same error)
Comment 7 Aleisha Amohia 2015-08-14 02:21:01 UTC Comment hidden (obsolete)
Comment 8 Joonas Kylmälä 2015-08-14 12:09:26 UTC Comment hidden (obsolete)
Comment 9 Jonathan Druart 2015-08-18 14:41:40 UTC
Created attachment 41603 [details] [review]
Bug 14484: warns when changing patron pw

To test:
1) Go to any patron's page, click Change password
2) Notice warns in error log
3) Apply patch, reload page
4) Click Change password
5) Notice warns are gone and page still works correctly

Signed-off-by: Joonas Kylmälä <j.kylmala@gmail.com>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 10 Tomás Cohen Arazi 2015-08-20 16:48:54 UTC
Patch pushed to master.

Thanks Aleisha!