https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=32968 --- Comment #7 from Jonathan Druart <jonathan.druart+koha@gmail.com> --- 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.