Openbravo Issue Tracking System - Openbravo ERP | |||||||||||||||||||
| View Issue Details | |||||||||||||||||||
| ID | Project | Category | View Status | Date Submitted | Last Update | ||||||||||||||
| 0017609 | Openbravo ERP | A. Platform | public | 2011-06-10 19:34 | 2022-02-01 08:05 | ||||||||||||||
| Reporter | shuehner | ||||||||||||||||||
| Assigned To | Triage Platform Base | ||||||||||||||||||
| Priority | normal | Severity | minor | Reproducibility | have not tried | ||||||||||||||
| Status | acknowledged | Resolution | open | ||||||||||||||||
| Platform | OS | 5 | OS Version | ||||||||||||||||
| Product Version | |||||||||||||||||||
| Target Version | Fixed in Version | ||||||||||||||||||
| Merge Request Status | |||||||||||||||||||
| Review Assigned To | |||||||||||||||||||
| OBNetwork customer | |||||||||||||||||||
| Web browser | |||||||||||||||||||
| Modules | Core | ||||||||||||||||||
| Support ticket | |||||||||||||||||||
| Regression level | |||||||||||||||||||
| Regression date | |||||||||||||||||||
| Regression introduced in release | |||||||||||||||||||
| Regression introduced by commit | |||||||||||||||||||
| Triggers an Emergency Pack | No | ||||||||||||||||||
| Summary | 0017609: Code-review for issue: 16639 | ||||||||||||||||||
| Description | This 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 Reproduce | N/A | ||||||||||||||||||
| Proposed Solution | |||||||||||||||||||
| Additional Information | |||||||||||||||||||
| Tags | OB3-Reviewed | ||||||||||||||||||
| Relationships |
| ||||||||||||||||||
| Attached Files | |||||||||||||||||||
| 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 | ||||||||||||||||
| There are no notes attached to this issue. |