Details
-
Bug
-
Status: Closed
-
Should
-
Resolution: Fixed
-
None
-
None
Description
We have fixed as part of TRUNK-4348 the general case when one module name is a substring of another. But, as wyclif points out, this may still fail if for modules with periods in the name. For instance, if there was a reporting.ui module, OpenmrsClassLoader would potentially match it with the reporting module classloader.
Potential fix?
We could change it so that it takes the classname and truncates at the last dot and then do an equals comparison. Are there any cases when we think doing that wouldn't be the right thing?
For example:
int pos = className.lastIndexOf(".");
String classPackageName = className.substring(0, pos);
tryToLoad = classPackageName.equals(moduleClassLoader.getModul().getPackageName())
Gliffy Diagrams
Attachments
Issue Links
- relates to
-
TRUNK-4348 OpenmrsClassLoader chooses wrong ModuleClassLoader when one module name is a substring of another module name
-
- Closed
-
- mentioned in
-
Page Loading...