Anonymous | Login
Project:
RSS
  
News | My View | View Issues | Roadmap | Summary

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0017609
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Openbravo ERP] A. Platformminorhave not tried2011-06-10 19:342022-02-01 08:05
ReportershuehnerView Statuspublic 
Assigned ToTriage Platform Base 
PrioritynormalResolutionopenFixed in Version
StatusacknowledgedFix in branchFixed in SCM revision
ProjectionnoneETAnoneTarget Version
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Review Assigned To
Web browser
ModulesCore
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0017609: Code-review for issue: 16639

DescriptionThis issue list some things found while doing code-review for 16639:

a.) New column 'ISEXCLUDEAUDIT' in table ad_column is missing standard boolean constraint
b.) Help text for same new column mentions 'Y' value in a boolean column. Not understable by user, as char(1) with Y,N is technical low-level detail.
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.
d.) AuditTrailPopup.java, in code showing list of excluded columns, add ' ' space after , do make reading list with more than one element easier
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.
Steps To ReproduceN/A
TagsOB3-Reviewed
Attached Files

- Relationships Relation Graph ] Dependency Graph ]
has duplicate design defect 0031425 acknowledgedTriage Platform Base many boolean columns incorrectly defined in AD and DB 
related to feature request 0016639 closedthirumalaik Exclude column from audit 

-  Notes
There are no notes attached to this issue.

- Issue History
Date Modified Username Field Change
2011-06-10 19:34 shuehner New Issue
2011-06-10 19:34 shuehner Assigned To => alostale
2011-06-10 19:34 shuehner Modules => Core
2011-06-10 19:34 shuehner Relationship added related to 0016639
2011-06-14 06:02 thirumalaik Issue Monitored: thirumalaik
2011-06-22 19:41 dmitry_mezentsev Target Version 3.0MP1 => 3.0MP3
2011-07-26 17:28 alostale Status new => scheduled
2011-08-05 10:37 alostale Assigned To alostale => thirumalaik
2011-09-01 15:52 alostale Target Version 3.0MP3 => 3.0MP4
2011-09-27 10:30 iperdomo Target Version 3.0MP4 =>
2011-10-28 20:31 iciordia Tag Attached: OB3-Reviewed
2011-10-28 20:31 iciordia Assigned To thirumalaik => alostale
2015-03-11 15:31 alostale Status scheduled => feedback
2015-03-11 15:32 alostale Status feedback => new
2015-03-17 14:37 alostale Assigned To alostale => platform
2015-11-11 11:02 alostale Relationship added has duplicate 0031425
2015-11-11 11:03 alostale Status new => acknowledged
2022-02-01 08:05 alostale Assigned To platform => Triage Platform Base


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker