Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
0018978Openbravo ERPA. Platformpublic2011-11-05 02:582022-02-01 08:08
johnfandl 
Triage Platform Base 
normalminorhave not tried
acknowledgedopen 
5
 
 
Core
No
0018978: Audit Trail screen Old Value / New Value fields should have real values, not IDs
The Audit Trail screen (which shows ALL audit records) is not consistent with the window-specific audit screen, in terms of the values in the Old / New Value fields.

The window-specific audit screens have foreign key values properly resolved, while the Audit Trail screen itself shows only the unresolved record id, which is useless to the user.
 For example:

1. Go to a sales order line, and change the tac from Arrendamiento 18% (cobros) to
Arrendamiento 16% (cobros) - IVA Normal
2. On the Sales Order Audit Trail screen, the old and new values match what the user selected on the screen
3. But, go to the Master Audit Trail screen and see that the old value is 4DB3919EB42E485FBFB6EA1342C07C04 and the new value is 8AA93DB8D416480E9A54D79A25403E7F.

These are not useful values. If the foreign key IDs were properly resolved to real values (like on the table-specific audit screens), it would be very, very useful for searching. For example, consider this common scenario:

- employee maintains an order line, to change the tax (for example). It is actually the WRONG order line, a simple mistake.
- supervisor looks at the intended order line, sees that it has not been changed.
- Asks employee about it. Employee says "yes, I changed an order line to 18%".
- They look at the record and employee realizes he changed the wrong order
- If the real values were properly shown on the Audit Trail screen, it would be a very simple matter to FIND the order line that was incorrectly changed (so it could be changed back)--just filter on the New Value. As the screen is now, it is useless for this type of research
- Additionally, the Audit Trail screen must include a link to open the maintenance screen up on a new application tab (I will enter a feature request for that as well).
Resolve the foreign key values on the Master Audit Trail screen, to be consistent with the table-specific audit screens. The real point in time values should be captured in the audit tables, not just the IDs.

This would make the Audit Trail much more useful for easily finding any information that was changed (not just literal values).
No tags attached.
Issue History
2011-11-05 02:58johnfandlNew Issue
2011-11-05 02:58johnfandlAssigned To => alostale
2011-11-05 02:58johnfandlModules => Core
2011-11-16 16:38alostaleNote Added: 0042882
2011-11-16 16:38alostaleStatusnew => acknowledged
2011-11-16 16:38alostaleTypedefect => design defect
2011-11-16 16:38alostaleSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=2911#r2911
2011-11-16 16:38alostaleProposed Solution updated
2012-09-24 21:03AugustoMauchNote Added: 0052416
2012-09-24 21:03AugustoMauchStatusacknowledged => scheduled
2012-09-24 21:03AugustoMauchSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=3891#r3891
2017-03-31 14:36alostaleStatusscheduled => acknowledged
2017-04-10 14:33alostaleAssigned Toalostale => platform
2022-02-01 08:08alostaleAssigned Toplatform => Triage Platform Base

Notes
(0042882)
alostale   
2011-11-16 16:38   
Moved to design defect.

With current references this cannot be implemented. It is needed to create first "Dynamic Link reference" (scheduled in the backlog) because reference in this case depends on the column modified.
(0052416)
AugustoMauch   
2012-09-24 21:03   
Impact: mid
Plan: mid