diff --git a/rero_ils/modules/loans/logs/api.py b/rero_ils/modules/loans/logs/api.py index df0fedecac..6613a0c3cf 100644 --- a/rero_ils/modules/loans/logs/api.py +++ b/rero_ils/modules/loans/logs/api.py @@ -120,6 +120,6 @@ def anonymize_logs(cls, loan_pid): """ for log in cls.get_logs_by_record_pid(loan_pid): record = log.to_dict() - record['loan']['patron'].pop('name', None) - record['loan']['patron'].pop('pid', None) + record['loan']['patron']['name'] = 'anonymized' + record['loan']['patron']['pid'] = 'anonymized' cls.update(log.meta.id, log['date'], record) diff --git a/tests/ui/loans/test_loans_operation_logs.py b/tests/ui/loans/test_loans_operation_logs.py index b7ed5c6a05..333518565f 100644 --- a/tests/ui/loans/test_loans_operation_logs.py +++ b/tests/ui/loans/test_loans_operation_logs.py @@ -111,5 +111,5 @@ def test_anonymize_logs(item2_on_loan_martigny_patron_and_loan_on_loan): log = log.to_dict() md5_hash = hashlib.md5(patron['pid'].encode()).hexdigest() assert log['loan']['patron']['hashed_pid'] == f'{md5_hash}' - assert not log['loan']['patron'].get('name') - assert not log['loan']['patron'].get('pid') + assert log['loan']['patron'].get('name') == 'anonymized' + assert log['loan']['patron'].get('pid') == 'anonymized'