Summary: | PatronSelfRegistrationLibraryList results in empty branch list on opac-memberentry.pl | ||
---|---|---|---|
Product: | Koha | Reporter: | Andrew Fuerste-Henry <andrew> |
Component: | OPAC | Assignee: | Jonathan Druart <jonathan.druart> |
Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
Severity: | major | ||
Priority: | P5 - low | CC: | 1joynelson, bgkriegel, jonathan.druart, lucas, martin.renvoize |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
GIT URL: | Change sponsored?: | --- | |
Patch complexity: | Trivial patch | Documentation contact: | |
Documentation submission: | Text to go in the release notes: | ||
Version(s) released in: |
20.05.00
|
Circulation function: | |
Bug Depends on: | 23084 | ||
Bug Blocks: | 25136 | ||
Attachments: |
Bug 25137: (bug 23084 follow-up) Fix incorrect grep ternary condition
Bug 25137: (bug 23084 follow-up) Fix incorrect grep ternary condition Bug 25137: (bug 23084 follow-up) Fix incorrect grep ternary condition |
Description
Andrew Fuerste-Henry
2020-04-13 19:48:46 UTC
Caused by commit 82716a01727f143ba2c167cd86233a531f330390 Bug 23084: Replace grep {^$var$} with grep {$_ eq $var} Created attachment 102917 [details] [review] Bug 25137: (bug 23084 follow-up) Fix incorrect grep ternary condition The lack of parenthesis makes the condition evaluated wrong. Test plan: - confirm you have a branch with code X - enter X in PatronSelfRegistrationLibraryList - go to cgi-bin/koha/opac-memberentry.pl (either as a logged in patron or as a self-reg patron) - confirm branch dropdown is not empty and contain X - empty out PatronSelfRegistrationLibraryList - reload cgi-bin/koha/opac-memberentry.pl and confirm branches show @QA I did check the other occurrences modified by bug 23084 and they appear correct. Another look would be good anyway! Was the lack of () the reason? If that is the case perhaps C4/Utils/DataTables/Members.pm lines 59/60 Why write a logical test not surrounded by parenthesis? Tested, fix the error. Created attachment 102921 [details] [review] Bug 25137: (bug 23084 follow-up) Fix incorrect grep ternary condition The lack of parenthesis makes the condition evaluated wrong. Test plan: - confirm you have a branch with code X - enter X in PatronSelfRegistrationLibraryList - go to cgi-bin/koha/opac-memberentry.pl (either as a logged in patron or as a self-reg patron) - confirm branch dropdown is not empty and contain X - empty out PatronSelfRegistrationLibraryList - reload cgi-bin/koha/opac-memberentry.pl and confirm branches show Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com> Created attachment 102951 [details] [review] Bug 25137: (bug 23084 follow-up) Fix incorrect grep ternary condition The lack of parenthesis makes the condition evaluated wrong. Test plan: - confirm you have a branch with code X - enter X in PatronSelfRegistrationLibraryList - go to cgi-bin/koha/opac-memberentry.pl (either as a logged in patron or as a self-reg patron) - confirm branch dropdown is not empty and contain X - empty out PatronSelfRegistrationLibraryList - reload cgi-bin/koha/opac-memberentry.pl and confirm branches show Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Nice work everyone! Pushed to master for 20.05 missing dependencies - not backported to 19.11.x |