Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0016868Openbravo ERPA. Platformpublic2011-04-20 18:152012-03-02 15:36
Reporterrafademiguel 
Assigned Tomarvintm 
PrioritynormalSeverityminorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in Version3.0MP9 
Merge Request Status
Review Assigned To
OBNetwork customerNo
Web browser
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0016868: A component in a package that not match with the package of module doesn't work
DescriptionA component in a package that not match with the package of module doesn't work

Steps To ReproduceCreate a module with package name: org.openbravo.ui.workshop

Create a package named: org.openbravo.ui.workshop.toolbar

Create a Component that extends from BaseComponentProvider inside this package

The Component doesn't work until you override the method "getModulePackageName" with "org.openbravo.ui.workshop"
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2011-04-20 18:15rafademiguelNew Issue
2011-04-20 18:15rafademiguelAssigned To => alostale
2011-04-20 18:15rafademiguelModules => Core
2011-04-20 18:15rafademiguelOBNetwork customer => No
2011-05-02 10:51alostaleTarget Version3.0RC4 => 3.0RC7
2011-05-02 10:51alostaleStatusnew => scheduled
2011-05-02 10:51alostaleAssigned Toalostale => mtaal
2011-05-02 10:51alostalefix_in_branch => pi
2011-05-05 17:36dmitry_mezentsevNote Added: 0036410
2011-05-05 17:36dmitry_mezentsevTarget Version3.0RC7 => 3.0MP0
2011-05-05 17:36dmitry_mezentsevfix_in_branchpi =>
2011-05-23 13:26mtaalTarget Version3.0MP0 => 3.0MP1
2011-06-22 19:41dmitry_mezentsevTarget Version3.0MP1 => 3.0MP3
2011-08-31 09:48mtaalTarget Version3.0MP3 => 3.0MP4
2011-09-07 13:30mtaalNote Added: 0040790
2011-09-07 13:31mtaalAssigned Tomtaal => marvintm
2011-09-26 17:51marvintmTarget Version3.0MP4 => 3.0MP5
2011-10-26 17:08marvintmTarget Version3.0MP5 => 3.0MP6
2011-11-15 10:17iperdomoTarget Version3.0MP6 =>
2012-02-13 17:59hgbotCheckin
2012-02-13 17:59hgbotNote Added: 0045076
2012-02-13 17:59hgbotStatusscheduled => resolved
2012-02-13 17:59hgbotResolutionopen => fixed
2012-02-13 17:59hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/f52dabf2bdc4a5a3a80fa4a09c7fc02114e1def1 [^]
2012-02-28 11:50AugustoMauchNote Added: 0045521
2012-02-28 11:50AugustoMauchStatusresolved => closed
2012-02-28 11:51AugustoMauchFixed in Version => 3.0MP9
2012-03-02 15:36hudsonbotCheckin
2012-03-02 15:36hudsonbotNote Added: 0045838

Notes
(0036410)
dmitry_mezentsev   
2011-05-05 17:36   
Does not fit RC7
(0040790)
mtaal   
2011-09-07 13:30   
The code in BaseComponentProvider.getModule() should try parent package names (recursively) if it can't find a module with the classes package name.
(0045076)
hgbot   
2012-02-13 17:59   
Repository: erp/devel/pi
Changeset: f52dabf2bdc4a5a3a80fa4a09c7fc02114e1def1
Author: Antonio Moreno <antonio.moreno <at> openbravo.com>
Date: Mon Feb 13 17:58:29 2012 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/f52dabf2bdc4a5a3a80fa4a09c7fc02114e1def1 [^]

Fixed issue 16868. ComponentProviders will be matched with modules via the javapackage smartly:
- A class will be matched to a module if its javapackage starts with the javapackage of the module (we no longer require the matching to be exact).
- If more than one module matches, the one with the longest Javapackage will be picked. This is to resolve situations in which, for example, both org.openbravo and org.openbravo.client.kernel match.

---
M modules/org.openbravo.client.kernel/src/org/openbravo/client/kernel/KernelUtils.java
---
(0045521)
AugustoMauch   
2012-02-28 11:50   
Code reviewed and verified
(0045838)
hudsonbot   
2012-03-02 15:36   
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/544d64e0c159 [^]

Maturity status: Test