You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by ja...@apache.org on 2011/01/05 12:05:13 UTC
svn commit: r1055411 -
/ofbiz/trunk/framework/base/src/org/ofbiz/base/container/ComponentContainer.java
Author: jacopoc
Date: Wed Jan 5 11:05:13 2011
New Revision: 1055411
URL: http://svn.apache.org/viewvc?rev=1055411&view=rev
Log:
hot-deploy component are now always loaded in alphanumeric order; before this change they were loaded randomly (in a platform dependedent way).
Modified:
ofbiz/trunk/framework/base/src/org/ofbiz/base/container/ComponentContainer.java
Modified: ofbiz/trunk/framework/base/src/org/ofbiz/base/container/ComponentContainer.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/base/src/org/ofbiz/base/container/ComponentContainer.java?rev=1055411&r1=1055410&r2=1055411&view=diff
==============================================================================
--- ofbiz/trunk/framework/base/src/org/ofbiz/base/container/ComponentContainer.java (original)
+++ ofbiz/trunk/framework/base/src/org/ofbiz/base/container/ComponentContainer.java Wed Jan 5 11:05:13 2011
@@ -22,6 +22,7 @@ import java.io.File;
import java.io.IOException;
import java.net.MalformedURLException;
import java.net.URL;
+import java.util.Arrays;
import java.util.List;
import org.ofbiz.base.component.AlreadyLoadedException;
@@ -195,7 +196,9 @@ public class ComponentContainer implemen
Debug.logError(e, "Unable to load components from URL: " + configUrl.toExternalForm(), module);
}
} else {
- for (String sub: parentPath.list()) {
+ String[] fileNames = parentPath.list();
+ Arrays.sort(fileNames);
+ for (String sub: fileNames) {
try {
File componentPath = FileUtil.getFile(parentPath.getCanonicalPath() + "/" + sub);
if (componentPath.isDirectory() && !sub.equals("CVS") && !sub.equals(".svn")) {