https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32968
--- Comment #7 from Jonathan Druart <[email protected]> --- Yes, this looks good to me. One suggestion is to move the "OR" logic in the template Pass permission.agreement = permissions.admin || permissions.agreement ? 1 : 0 And simply test permission.agreement later you could also have + permissions.admin || + permissions.agreement || + permissions.show in show_agreement? Or... ... generate a hash like (pseudo-code): { agreement => { read => { permissions.admin || permissions.agreement || permissions.show } write => { permissions.admin || permissions.agreement } } } Then simply test for agreement.read and agreement.write? -- You are receiving this mail because: You are watching all bug changes. _______________________________________________ Koha-bugs mailing list [email protected] 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/
