Post Fri Aug 05, 2011 11:16 am

Audit trail code issue

Hi,

I started fixing of task 0017609: Code-review for issue: 16639[1].I need some assistance on point (c) and (e).

c.) SL_TableAudit callout, why does code-path for excluding-columns also check and modify SessionInfo.setAuditActive at all? Excluding individual columns does not change global audit state.


I assumed that,user may try to modify column as excludeaudit without modifying table(already audited).Please confirm my understanding.

e.) I AuditTrailPopup.java, extend utility function: getExcludeAuditColumnNames as the three places calling them look exactly identical. I.e. Let it return final translated string to be passed into the xmlEngine parameter.


Unable to understand.Will you please give some more detail?


https://issues.openbravo.com/view.php?id=17609