You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ofbiz.apache.org by mt...@apache.org on 2019/10/30 15:33:13 UTC
svn commit: r1869180 -
/ofbiz/ofbiz-framework/trunk/framework/base/src/main/java/org/apache/ofbiz/base/container/ComponentContainer.java
Author: mthl
Date: Wed Oct 30 15:33:13 2019
New Revision: 1869180
URL: http://svn.apache.org/viewvc?rev=1869180&view=rev
Log:
Improved: Import ‘ComponentDef’ and ‘DependsOnInfo’ inner-classes
(OFBIZ-11264)
This allows to use shorter name when using an variable or parameter of
those types.
Modified:
ofbiz/ofbiz-framework/trunk/framework/base/src/main/java/org/apache/ofbiz/base/container/ComponentContainer.java
Modified: ofbiz/ofbiz-framework/trunk/framework/base/src/main/java/org/apache/ofbiz/base/container/ComponentContainer.java
URL: http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/framework/base/src/main/java/org/apache/ofbiz/base/container/ComponentContainer.java?rev=1869180&r1=1869179&r2=1869180&view=diff
==============================================================================
--- ofbiz/ofbiz-framework/trunk/framework/base/src/main/java/org/apache/ofbiz/base/container/ComponentContainer.java (original)
+++ ofbiz/ofbiz-framework/trunk/framework/base/src/main/java/org/apache/ofbiz/base/container/ComponentContainer.java Wed Oct 30 15:33:13 2019
@@ -37,8 +37,10 @@ import java.util.concurrent.atomic.Atomi
import java.util.stream.Collectors;
import org.apache.ofbiz.base.component.ComponentConfig;
+import org.apache.ofbiz.base.component.ComponentConfig.DependsOnInfo;
import org.apache.ofbiz.base.component.ComponentException;
import org.apache.ofbiz.base.component.ComponentLoaderConfig;
+import org.apache.ofbiz.base.component.ComponentLoaderConfig.ComponentDef;
import org.apache.ofbiz.base.start.Start;
import org.apache.ofbiz.base.start.StartupCommand;
import org.apache.ofbiz.base.util.Debug;
@@ -61,7 +63,7 @@ public class ComponentContainer implemen
private String name;
private final AtomicBoolean loaded = new AtomicBoolean(false);
private final List<Classpath> componentsClassPath = new ArrayList<>();
- private static Map<String, List<ComponentConfig.DependsOnInfo>> toBeLoadedComponents = new ConcurrentHashMap<>();
+ private static Map<String, List<DependsOnInfo>> toBeLoadedComponents = new ConcurrentHashMap<>();
@Override
public void init(List<StartupCommand> ofbizCommands, String name, String configFile) throws ContainerException {
@@ -72,7 +74,7 @@ public class ComponentContainer implemen
// load the components from framework/base/config/component-load.xml (root components)
try {
- for (ComponentLoaderConfig.ComponentDef def: ComponentLoaderConfig.getRootComponents()) {
+ for (ComponentDef def: ComponentLoaderConfig.getRootComponents()) {
loadComponent(Start.getInstance().getConfig().ofbizHome, def);
}
} catch (IOException | ComponentException e) {
@@ -117,7 +119,7 @@ public class ComponentContainer implemen
* @throws IOException when component directory loading fails.
* @throws ComponentException when retrieving component configuration files fails.
*/
- private void loadComponent(Path dir, ComponentLoaderConfig.ComponentDef component)
+ private void loadComponent(Path dir, ComponentDef component)
throws IOException, ComponentException {
Path location = component.location.isAbsolute() ? component.location : dir.resolve(component.location);
switch (component.type) {
@@ -172,8 +174,8 @@ public class ComponentContainer implemen
URL configUrl = null;
try {
configUrl = componentLoadFile.toUri().toURL();
- List<ComponentLoaderConfig.ComponentDef> componentsToLoad = ComponentLoaderConfig.getComponentsFromConfig(configUrl);
- for (ComponentLoaderConfig.ComponentDef def: componentsToLoad) {
+ List<ComponentDef> componentsToLoad = ComponentLoaderConfig.getComponentsFromConfig(configUrl);
+ for (ComponentDef def: componentsToLoad) {
loadComponent(directoryPath, def);
}
} catch (MalformedURLException e) {
@@ -228,10 +230,10 @@ public class ComponentContainer implemen
if (UtilValidate.isEmpty(toBeLoadedComponents)) {
return;
} else {
- for (Map.Entry<String, List<ComponentConfig.DependsOnInfo>> entries : toBeLoadedComponents.entrySet()) {
+ for (Map.Entry<String, List<DependsOnInfo>> entries : toBeLoadedComponents.entrySet()) {
ComponentConfig config = retrieveComponentConfig(entries.getKey(), null);
if (config.enabled()) {
- List<ComponentConfig.DependsOnInfo> dependencyList = checkDependencyForComponent(config);
+ List<DependsOnInfo> dependencyList = checkDependencyForComponent(config);
if (UtilValidate.isNotEmpty(dependencyList)) {
toBeLoadedComponents.replace(config.getComponentName(), dependencyList);
String msg = "Not loading component [" + config.getComponentName() + "] because it's dependent Component is not loaded [ " + dependencyList + "]";
@@ -280,7 +282,7 @@ public class ComponentContainer implemen
*/
private void loadSingleComponent(ComponentConfig config) throws IOException, ComponentException {
if (config.enabled()) {
- List<ComponentConfig.DependsOnInfo> dependencyList = checkDependencyForComponent(config);
+ List<DependsOnInfo> dependencyList = checkDependencyForComponent(config);
if (UtilValidate.isEmpty(dependencyList)) {
componentsClassPath.add(buildClasspathFromComponentConfig(config));
Debug.logInfo("Added class path for component : [" + config.getComponentName() + "]", module);
@@ -299,11 +301,11 @@ public class ComponentContainer implemen
* @throws ComponentException
*
*/
- private List<ComponentConfig.DependsOnInfo> checkDependencyForComponent(ComponentConfig config) throws IOException, ComponentException {
- List<ComponentConfig.DependsOnInfo> dependencyList = new ArrayList<>(config.getDependsOn());
+ private List<DependsOnInfo> checkDependencyForComponent(ComponentConfig config) throws IOException, ComponentException {
+ List<DependsOnInfo> dependencyList = new ArrayList<>(config.getDependsOn());
if (UtilValidate.isNotEmpty(dependencyList)) {
- Set<ComponentConfig.DependsOnInfo> resolvedDependencyList = new HashSet<>();
- for (ComponentConfig.DependsOnInfo dependency : dependencyList) {
+ Set<DependsOnInfo> resolvedDependencyList = new HashSet<>();
+ for (DependsOnInfo dependency : dependencyList) {
Debug.logInfo("Component : " + config.getComponentName() + " is Dependent on " + dependency.componentName, module);
ComponentConfig componentConfig = ComponentConfig.getComponentConfig(String.valueOf(dependency.componentName));
Classpath dependentComponentClasspath = buildClasspathFromComponentConfig(componentConfig);