CIRC-2550 Fix circulation log showing incorrect source for loan renewals

[zakletskyi-epam]

CIRC-2550

Purpose

When a loan was renewed by one staff member but originally checked out by another, the Circulation Log incorrectly displayed the checkout staff as the source of the renewal action. This happened because the renewal event was published using the loan's metadata.updatedByUserId (set at checkout time) rather than the ID of the staff member who actually performed the renewal.

Approach

Instead of reusing the existing publishDueDateChangedEvent (which reads the source from loan metadata), a dedicated publishRenewedEvent method was introduced. This method explicitly passes the loggedInUserId from the RenewalContext as the updatedByUserId in the log record, ensuring the Circulation Log reflects the staff member who performed the renewal. A unit test was added to verify that the renewal log entry carries the renewal staff ID and not the checkout staff ID.

TODOS and Open Questions

• Check logging
• Use GitHub checklists. When solved, check the box and explain the answer.

Learning

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud