https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=26517
Katrin Fischer <katrin.fisc...@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Failed QA --- Comment #7 from Katrin Fischer <katrin.fisc...@bsz-bw.de> --- Hm, a few small things: 1) if a user has only subpermissions, flags are 0. Should we extend the check to take care of these? I could imagine someone removing catalog permission for someone temporarily not working at the library, but expected to return. 2) I think all the tests for GetBororwersToExpunge should be grouped into a subtest (but this was an issue before your patch) 3) All patrons I see in the tests have flags = undef... am I missing the one blocked from deleting for flags? -- You are receiving this mail because: You are watching all bug changes. _______________________________________________ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/