Bug 22929: Allow SCI/SCO logins independent of GDPR_Policy
authorMarcel de Rooy <m.de.rooy@rijksmuseum.nl>
Fri, 17 May 2019 07:36:17 +0000 (07:36 +0000)
committerMartin Renvoize <martin.renvoize@ptfs-europe.com>
Mon, 9 Sep 2019 10:15:11 +0000 (11:15 +0100)
commit3640130d1fbc49ddc2cc4baef53a4c4d7fea76bf
tree75f97348ce198c9350c5a8b6dad13a49046c6303
parentcb432e7f522f2f052d561dad3b36bc97ca7868a4
Bug 22929: Allow SCI/SCO logins independent of GDPR_Policy

This patch makes Auth skip the GDPR policy check when get_template_and_user
is called for sci/sco (self checkin, checkout).
I do not really like the change in this form but the nature of self checkin
and checkout kind of dictate it (double hack).
I wanted to add a test but since that asks for mocking CGI, checkauth, etc.
the time needed for that is just too much for this simple change.

Test plan:
Enable GDPR_Policy and self checkin/checkout.
Verify that using patrons without consent is not blocking sci/sco.

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
C4/Auth.pm