Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0035865Openbravo ERPA. Platformpublic2017-04-27 15:162017-05-29 17:43
Reporteralostale 
Assigned Toalostale 
PrioritynormalSeveritytrivialReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in Version3.0PR17Q3 
Merge Request Status
Review Assigned Tocaristu
OBNetwork customerNo
Web browser
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0035865: remove unused UsageAudit methods
DescriptionThere are some methods in UsageAudit class making use of DAL, they should be removed because:

* They are underperformant
* They are buggy
* They are not used since 2011 [1]

[1] https://code.openbravo.com/erp/devel/pi/rev/5cd9e415 [^]
Steps To Reproduce-
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
related to defect 0035855 closed alostale many standard requests borrow more than one connection from DB 
depends on defect 0035905 closed alostale API change: methods removed in UsageAudit class 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2017-04-27 15:16alostaleNew Issue
2017-04-27 15:16alostaleAssigned To => alostale
2017-04-27 15:16alostaleOBNetwork customer => No
2017-04-27 15:16alostaleModules => Core
2017-04-27 15:16alostaleTriggers an Emergency Pack => No
2017-04-27 15:17alostaleStatusnew => scheduled
2017-04-27 15:21alostaleRelationship addedrelated to 0035855
2017-05-03 10:22alostaleRelationship addeddepends on 0035905
2017-05-03 10:27hgbotCheckin
2017-05-03 10:27hgbotNote Added: 0096336
2017-05-03 10:27hgbotStatusscheduled => resolved
2017-05-03 10:27hgbotResolutionopen => fixed
2017-05-03 10:27hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/14c7240bde7a31e001d38c07fc9a91cb42f3ffef [^]
2017-05-03 10:35alostaleReview Assigned To => caristu
2017-05-09 10:19caristuNote Added: 0096478
2017-05-09 10:19caristuStatusresolved => closed
2017-05-09 10:19caristuFixed in Version => 3.0PR17Q3
2017-05-29 17:43hudsonbotCheckin
2017-05-29 17:43hudsonbotNote Added: 0096925

Notes
(0096336)
hgbot   
2017-05-03 10:27   
Repository: erp/devel/pi
Changeset: 14c7240bde7a31e001d38c07fc9a91cb42f3ffef
Author: Asier Lostalé <asier.lostale <at> openbravo.com>
Date: Thu Apr 27 15:21:17 2017 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/14c7240bde7a31e001d38c07fc9a91cb42f3ffef [^]

fixed bug 35865: remove unused UsageAudit methods

---
M src/org/openbravo/erpCommon/security/UsageAudit.java
---
(0096478)
caristu   
2017-05-09 10:19   
Reviewed
(0096925)
hudsonbot   
2017-05-29 17:43   
A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/1ee70113bdc4 [^]
Maturity status: Test