svn commit: r1862225 - /ofbiz/ofbiz-framework/trunk/framework/base/src/main/java/org/apache/ofbiz/base/component/ComponentConfig.java

[mthl]

Author: mthl
Date: Thu Jun 27 14:33:11 2019
New Revision: 1862225

URL: http://svn.apache.org/viewvc?rev=1862225&view=rev
Log:
Improved: Add ‘ComponentConfig#matchingComponentName’ static method
(OFBIZ-11101)

This factorizes the construction of a predicate filtering components by
their names.

Modified:
    ofbiz/ofbiz-framework/trunk/framework/base/src/main/java/org/apache/ofbiz/base/component/ComponentConfig.java

Modified: ofbiz/ofbiz-framework/trunk/framework/base/src/main/java/org/apache/ofbiz/base/component/ComponentConfig.java
URL: http://svn.apache.org/viewvc/ofbiz/ofbiz-framework/trunk/framework/base/src/main/java/org/apache/ofbiz/base/component/ComponentConfig.java?rev=1862225&r1=1862224&r2=1862225&view=diff
==============================================================================
--- ofbiz/ofbiz-framework/trunk/framework/base/src/main/java/org/apache/ofbiz/base/component/ComponentConfig.java (original)
+++ ofbiz/ofbiz-framework/trunk/framework/base/src/main/java/org/apache/ofbiz/base/component/ComponentConfig.java Thu Jun 27 14:33:11 2019
@@ -30,6 +30,7 @@ import java.util.LinkedHashMap;
 import java.util.List;
 import java.util.Map;
 import java.util.function.BiFunction;
+import java.util.function.Predicate;
 import java.util.stream.Collectors;
 
 import org.apache.ofbiz.base.container.ContainerConfig;
@@ -64,6 +65,17 @@ public final class ComponentConfig {
     }
 
     /**
+     * Constructs a component predicate checking if it corresponds to a specific name.
+     *
+     * @param cname  the name of the component to match which can be {@code null}
+     *               which means "any" component
+     * @return a component predicate for matching a specific component name.
+     */
+    private static Predicate<ComponentConfig> matchingComponentName(String cname) {
+        return cc -> cname == null || cname.equals(cc.getComponentName());
+    }
+
+    /**
      * Provides the list of all the classpath information available in components.
      *
      * @return a list of classpath information
@@ -102,7 +114,7 @@ public final class ComponentConfig {
      */
     public static List<EntityResourceInfo> getAllEntityResourceInfos(String type, String name) {
         return getAllComponents().stream()
-                .filter(cc -> name == null || name.equals(cc.getComponentName()))
+                .filter(matchingComponentName(name))
                 .flatMap(cc -> cc.getEntityResourceInfos().stream())
                 .filter(eri -> UtilValidate.isEmpty(type) || type.equals(eri.type))
                 .collect(Collectors.toList());
@@ -140,7 +152,7 @@ public final class ComponentConfig {
      */
     public static List<TestSuiteInfo> getAllTestSuiteInfos(String name) {
         return getAllComponents().stream()
-                .filter(cc -> name == null || name.equals(cc.getComponentName()))
+                .filter(matchingComponentName(name))
                 .flatMap(cc -> cc.getTestSuiteInfos().stream())
                 .collect(Collectors.toList());
     }