rero / rero-ils / 27329665989

91%

Build Type

push

github

Committed by PascalRepond

Commit Message

fix(patrons): avoid duplicate operation log on update

Patron.update overrode the IlsRecord default with commit=True, so the
REST PUT handler committed the record twice (update() then commit()),
firing the operation log extension's pre_commit twice and creating two
UPDATE logs per edit. Align the default with the base class (commit=False)
so the handler's explicit commit() is the only one.

Add a REST PUT step to test_operation_log_on_patron to cover the path the
existing direct-update test missed.

Co-Authored-by: Pascal Repond <pascal.repond@rero.ch>

Coverage Stats

1 of 1 new or added line in 1 file covered. (100.0%)

8 existing lines in 1 file now uncovered.

24138 of 26435 relevant lines covered (91.31%)

0.91 hits per line