Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
0013849Openbravo ERPZ. Otherspublic2010-06-30 17:302022-02-01 08:09
mtaal 
Triage Platform Base 
normalmajorhave not tried
newout of date 
5
2.50 
 
User Interface Application
No
0013849: Code in some modules is copied from core, needs to be refactored in core to prevent duplicate code
Some modules have code duplicated from core. To prevent this code duplication the code in core needs to be refactored to separate methods which can be called from the modules.

Specifically the following code has to be checked/refactored:
- KernelServlet.logout method and its uses
- AlertActionHandler.execute method
No tags attached.
Issue History
2010-06-30 17:30mtaalNew Issue
2010-06-30 17:30mtaalAssigned To => mtaal
2010-06-30 17:37shuehnerIssue Monitored: shuehner
2010-06-30 17:47shuehnerNote Added: 0028931
2010-07-01 18:48shuehnerSeverityminor => major
2010-07-01 18:50shuehnerNote Added: 0028963
2010-07-07 13:31shuehnerNote Added: 0029124
2010-08-31 09:41hgbotCheckin
2010-08-31 09:41hgbotNote Added: 0030509
2010-08-31 10:00hgbotCheckin
2010-08-31 10:00hgbotNote Added: 0030516
2010-09-03 14:37hgbotCheckin
2010-09-03 14:37hgbotNote Added: 0030604
2010-12-06 07:48alostaleStatusnew => scheduled
2012-06-15 17:12alostaleTypedefect => design defect
2016-10-21 11:00mtaalStatusscheduled => closed
2016-10-21 11:00mtaalResolutionopen => out of date
2016-10-21 13:17mtaalTag Attached: MT_closed_out_of_date
2016-10-21 14:08mtaalAssigned Tomtaal => platform
2016-10-21 14:09mtaalStatusclosed => new
2016-10-21 14:12mtaalTag Detached: MT_closed_out_of_date
2022-02-01 08:09alostaleAssigned Toplatform => Triage Platform Base

Notes
(0028931)
shuehner   
2010-06-30 17:47   
More places to be checked:

UserInfoWidgetAction (.. UserSessionResetter (for i.e. RoleChange))
KernelServlet integration with core HSAS servlet
(0028963)
shuehner   
2010-07-01 18:50   
Also code for getRoles in UserInfoWidgetActionHandler
(0029124)
shuehner   
2010-07-07 13:31   
new code of hearbeat/registration popup is also copied from core file
(0030509)
hgbot   
2010-08-31 09:41   
Repository: erp/mods/org.openbravo.client.kernel
Changeset: 55365a0b4d68c9e97f8e7daeb5fd72e371c3d27a
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Tue Aug 31 09:40:12 2010 +0200
URL: http://code.openbravo.com/erp/mods/org.openbravo.client.kernel/rev/55365a0b4d68c9e97f8e7daeb5fd72e371c3d27a [^]

Renamed AuthenticatingServlet, fixes issue 14376: Move wrapping in contextual seem request from KernelServlet & DatasourceServlet to super class. Related to issue 13849: Code in some modules is copied from core, needs to be refactored in core to prevent duplicate code

---
M src/org/openbravo/client/kernel/KernelServlet.java
A src/org/openbravo/client/kernel/BaseKernelServlet.java
R src/org/openbravo/client/kernel/AuthenticatingServlet.java
---
(0030516)
hgbot   
2010-08-31 10:00   
Repository: erp/mods/org.openbravo.client.application
Changeset: 5a2040c640cc40d4715d93898c0e40f581c0b4f9
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Tue Aug 31 10:00:22 2010 +0200
URL: http://code.openbravo.com/erp/mods/org.openbravo.client.application/rev/5a2040c640cc40d4715d93898c0e40f581c0b4f9 [^]

Related to issue 13849: Code in some modules is copied from core, needs to be refactored in core to prevent duplicate code

---
M src/org/openbravo/client/application/LogOutActionHandler.java
---
(0030604)
hgbot   
2010-09-03 14:37   
Repository: erp/mods/org.openbravo.client.application-new-activation
Changeset: 5a2040c640cc40d4715d93898c0e40f581c0b4f9
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Tue Aug 31 10:00:22 2010 +0200
URL: http://code.openbravo.com/erp/mods/org.openbravo.client.application-new-activation/rev/5a2040c640cc40d4715d93898c0e40f581c0b4f9 [^]

Related to issue 13849: Code in some modules is copied from core, needs to be refactored in core to prevent duplicate code

---
M src/org/openbravo/client/application/LogOutActionHandler.java
---