You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Re-introducing a method in AuditLogListenerUtil seems not to be the right thing to do.
The AuditLogEventController in general got the problem with audit domain class type definition, which is evaluated during runtime. For this, @transactional does not work, as we do not know which datasource the AuditTrail domain class is bound to.
What would work is using .withTransaction{} wrapping, which makes the code ugly and also lead to wierd other problems (AuditTrail cannot be cast to AuditTrail)
With fix for #189 we moved static methods from AuditLogListenerUtil to AuditLogListener instance.
We forgot to fix the AuditLogEventController...
Thanks to David Brown for bringing that up (via Slack)
The text was updated successfully, but these errors were encountered: