]> git.koha-community.org Git - koha.git/commit
Bug 25142: Fix wrong grep logical test
authorJonathan Druart <jonathan.druart@bugs.koha-community.org>
Tue, 14 Apr 2020 11:22:44 +0000 (13:22 +0200)
committerJoy Nelson <joy@bywatersolutions.com>
Mon, 20 Apr 2020 22:40:48 +0000 (22:40 +0000)
commitc33f598c5cdea829390660419498e15f968f9abe
treefc511666962a50882f10e3d0428c6564152de649
parent3401e94d942a8d8a4e216ea44bd295f96b8f3e24
Bug 25142: Fix wrong grep logical test

To test and understand what's going on, you can try that bit of code:
my @a = qw( a b c a);
my @b = qw( b c d );
my @c;
@c = grep { 'a' eq $_ } @a ? 'ok' : ();
say @c;
@c = ( grep { 'a' eq $_ } @a ) ? 'ok' : ();
say @c;
@c = grep { 'a' eq $_ } @a ? ('ok') : (undef);
say @c;

The problem here:
Have patrons in 3 branches CPL, MPL, SPL
Have a non superlibrarian with edit_borrowers permission but
without view_borrower_infos_from_any_libraries, from CPL
Create a library group with CPL, MPL
Use that non superlibrarian to search for patrons
You can search for patrons fro CPL and MPL
BUT, edit the value for CPL, use SPL (edit the DOM)
Search and... oops

Apply this patch, try again

Also use a superlibrarian patron (and/or with view_borrower_infos_from_any_libraries)
and confirm that they can see all patrons

Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com>
Tested with edit_borrowers permission but NOT view_borrower_infos_from_any_libraries
Editing the DOM I can search (not edit) from any branch
With patch no entries found :)

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Amended test plan slightly (+ without)

Signed-off-by: Joy Nelson <joy@bywatersolutions.com>
C4/Utils/DataTables/Members.pm