https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16694 --- Comment #29 from Magnus Enger <magnus@libriotech.no> --- Hm, I think I agree that a plugin might make sense here. But can we agree on how to signal that the patron should be denied access? The vendors of "more open" that I have worked with expect a SIP2 response that has us using this to set the proper status: $patron->{'charge_ok'} Could we pass $patron to the plugin and let the plugin do what it needs to it, before returning it? So instead of this (as suggested by Mark): $patron = undef unless ( check_patron ( $patron, $server ) ); we could do this? $patron = check_patron ( $patron, $server ); And the plugin could return undef or just a changed $patron. I would also suggest a more detailed name for the config-parameter: <accounts> <login id="kanopy" password="**password**" delimiter="|" error-detect="enabled" institution="RPL" plugin_class="Koha::Plugin::Com::Bloovis::Sip2patron" /> </accounts> Maybe something like patron_validation_plugin_class? There could be other plugins that do other things in the future. -- You are receiving this mail because: You are watching all bug changes.