[OpenbravoERP-commits] devel/pi: Fixes issue 27351: refactor (kernel, datasource) (II)

[hg]

details:   https://code.openbravo.com/erp/devel/pi/rev/a2f9c55b5ca9
changeset: 24309:a2f9c55b5ca9
user:      Rafa Alonso <rafael.alonso <at> openbravo.com>
date:      Sat Aug 23 01:31:40 2014 +0200
summary:   Fixes issue 27351: refactor (kernel, datasource) (II)

diffstat:

 
modules/org.openbravo.client.kernel/src/org/openbravo/client/kernel/KernelComponentProvider.java
               |  8 ++++----
 
modules/org.openbravo.service.datasource/src/org/openbravo/service/datasource/DataSourceComponentProvider.java
 |  8 ++++----
 2 files changed, 8 insertions(+), 8 deletions(-)

diffs (62 lines):

diff -r 077fca7b3c70 -r a2f9c55b5ca9 
modules/org.openbravo.client.kernel/src/org/openbravo/client/kernel/KernelComponentProvider.java
--- 
a/modules/org.openbravo.client.kernel/src/org/openbravo/client/kernel/KernelComponentProvider.java
  Sat Aug 23 09:05:26 2014 +0200
+++ 
b/modules/org.openbravo.client.kernel/src/org/openbravo/client/kernel/KernelComponentProvider.java
  Sat Aug 23 01:31:40 2014 +0200
@@ -37,6 +37,8 @@
 public class KernelComponentProvider extends BaseComponentProvider {
   public static final String QUALIFIER = KernelConstants.KERNEL_COMPONENT_TYPE;
 
+  private List<ComponentResource> globalResources = null;
+
   /*
    * (non-Javadoc)
    * 
@@ -80,14 +82,12 @@
     return versionParam;
   }
 
-  final List<ComponentResource> globalResources = new 
ArrayList<ComponentResource>();
-
   @Override
   public synchronized List<ComponentResource> getGlobalComponentResources() {
-    if (globalResources.size() > 0) {
+    if (globalResources != null) {
       return globalResources;
     }
-    globalResources.clear();
+    globalResources = new ArrayList<ComponentResource>();
 
     globalResources.add(createStaticResource("org.openbravo.client.kernel/"
         + KernelConstants.KERNEL_COMPONENT_TYPE + "/" + 
KernelConstants.APPLICATION_COMPONENT_ID,
diff -r 077fca7b3c70 -r a2f9c55b5ca9 
modules/org.openbravo.service.datasource/src/org/openbravo/service/datasource/DataSourceComponentProvider.java
--- 
a/modules/org.openbravo.service.datasource/src/org/openbravo/service/datasource/DataSourceComponentProvider.java
    Sat Aug 23 09:05:26 2014 +0200
+++ 
b/modules/org.openbravo.service.datasource/src/org/openbravo/service/datasource/DataSourceComponentProvider.java
    Sat Aug 23 01:31:40 2014 +0200
@@ -41,6 +41,8 @@
 public class DataSourceComponentProvider extends BaseComponentProvider {
   public final static String QUALIFIER = DataSourceConstants.DS_COMPONENT_TYPE;
 
+  private List<ComponentResource> globalResources = null;
+
   @Inject
   private DataSourceServiceProvider dataSourceServiceProvider;
 
@@ -75,18 +77,16 @@
     return dataSourceComponent;
   }
 
-  final List<ComponentResource> globalResources = new 
ArrayList<ComponentResource>();
-
   /**
    * @return an empty String (no global resources)
    * @see 
org.openbravo.client.kernel.ComponentProvider#getGlobalComponentResources()
    */
   @Override
   public synchronized List<ComponentResource> getGlobalComponentResources() {
-    if (globalResources.size() > 0) {
+    if (globalResources != null) {
       return globalResources;
     }
-    globalResources.clear();
+    globalResources = new ArrayList<ComponentResource>();
 
     globalResources.add(createStaticResource(
         "web/org.openbravo.service.datasource/js/ob-datasource-utilities.js", 
true));

------------------------------------------------------------------------------
Slashdot TV.  
Video for Nerds.  Stuff that matters.
http://tv.slashdot.org/
_______________________________________________
Openbravo-commits mailing list
Openbravo-commits@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openbravo-commits