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 2014/11/01 08:15:12 UTC

svn commit: r1635897 [3/4] - in /ofbiz/branches/json-integration-refactoring: ./ applications/order/src/org/ofbiz/order/order/ framework/base/src/org/ofbiz/base/concurrent/ framework/base/src/org/ofbiz/base/config/ framework/base/src/org/ofbiz/base/uti...

Modified: ofbiz/branches/json-integration-refactoring/framework/service/src/org/ofbiz/service/group/GroupModel.java
URL: http://svn.apache.org/viewvc/ofbiz/branches/json-integration-refactoring/framework/service/src/org/ofbiz/service/group/GroupModel.java?rev=1635897&r1=1635896&r2=1635897&view=diff
==============================================================================
--- ofbiz/branches/json-integration-refactoring/framework/service/src/org/ofbiz/service/group/GroupModel.java (original)
+++ ofbiz/branches/json-integration-refactoring/framework/service/src/org/ofbiz/service/group/GroupModel.java Sat Nov  1 07:15:09 2014
@@ -20,9 +20,7 @@ package org.ofbiz.service.group;
 
 import java.util.LinkedList;
 import java.util.List;
-import java.util.Map;
-
-import javolution.util.FastMap;
+import java.util.*;
 
 import org.ofbiz.base.util.Debug;
 import org.ofbiz.base.util.UtilMisc;
@@ -58,19 +56,21 @@ public class GroupModel {
             throw new IllegalArgumentException("Group Definition found with no name attribute! : " + group);
         }
 
-        for (Element service: UtilXml.childElementList(group, "invoke")) {
+        for (Element service : UtilXml.childElementList(group, "invoke")) {
             services.add(new GroupServiceModel(service));
         }
 
         List<? extends Element> oldServiceTags = UtilXml.childElementList(group, "service");
         if (oldServiceTags.size() > 0) {
-            for (Element service: oldServiceTags) {
+            for (Element service : oldServiceTags) {
                 services.add(new GroupServiceModel(service));
             }
-            Debug.logWarning("Service Group Definition : [" + group.getAttribute("name") + "] found with OLD 'service' attribute, change to use 'invoke'", module);
+            Debug.logWarning("Service Group Definition : [" + group.getAttribute("name")
+                    + "] found with OLD 'service' attribute, change to use 'invoke'", module);
         }
 
-        if (Debug.verboseOn()) Debug.logVerbose("Created Service Group Model --> " + this, module);
+        if (Debug.verboseOn())
+            Debug.logVerbose("Created Service Group Model --> " + this, module);
     }
 
     /**
@@ -109,6 +109,7 @@ public class GroupModel {
     public List<GroupServiceModel> getServices() {
         return this.services;
     }
+
     public boolean isOptional() {
         return optional;
     }
@@ -121,7 +122,8 @@ public class GroupModel {
      * @return Map Result Map
      * @throws GenericServiceException
      */
-    public Map<String, Object> run(ServiceDispatcher dispatcher, String localName, Map<String, Object> context) throws GenericServiceException {
+    public Map<String, Object> run(ServiceDispatcher dispatcher, String localName, Map<String, Object> context)
+            throws GenericServiceException {
         if (this.getSendMode().equals("all")) {
             return runAll(dispatcher, localName, context);
         } else if (this.getSendMode().equals("round-robin")) {
@@ -132,7 +134,7 @@ public class GroupModel {
         } else if (this.getSendMode().equals("first-available")) {
             return runOne(dispatcher, localName, context);
         } else if (this.getSendMode().equals("none")) {
-            return FastMap.newInstance();
+            return new HashMap<String, Object>();
         } else {
             throw new GenericServiceException("This mode is not currently supported");
         }
@@ -152,13 +154,16 @@ public class GroupModel {
         return str.toString();
     }
 
-    private Map<String, Object> runAll(ServiceDispatcher dispatcher, String localName, Map<String, Object> context) throws GenericServiceException {
+    private Map<String, Object> runAll(ServiceDispatcher dispatcher, String localName, Map<String, Object> context)
+            throws GenericServiceException {
         Map<String, Object> runContext = UtilMisc.makeMapWritable(context);
-        Map<String, Object> result = FastMap.newInstance();
-        for (GroupServiceModel model: services) {
-            if (Debug.verboseOn()) Debug.logVerbose("Using Context: " + runContext, module);
+        Map<String, Object> result = new HashMap<String, Object>();
+        for (GroupServiceModel model : services) {
+            if (Debug.verboseOn())
+                Debug.logVerbose("Using Context: " + runContext, module);
             Map<String, Object> thisResult = model.invoke(dispatcher, localName, runContext);
-            if (Debug.verboseOn()) Debug.logVerbose("Result: " + thisResult, module);
+            if (Debug.verboseOn())
+                Debug.logVerbose("Result: " + thisResult, module);
 
             // make sure we didn't fail
             if (ServiceUtil.isError(thisResult)) {
@@ -175,14 +180,16 @@ public class GroupModel {
         return result;
     }
 
-    private Map<String, Object> runIndex(ServiceDispatcher dispatcher, String localName, Map<String, Object> context, int index) throws GenericServiceException {
+    private Map<String, Object> runIndex(ServiceDispatcher dispatcher, String localName, Map<String, Object> context, int index)
+            throws GenericServiceException {
         GroupServiceModel model = services.get(index);
         return model.invoke(dispatcher, localName, context);
     }
 
-    private Map<String, Object> runOne(ServiceDispatcher dispatcher, String localName, Map<String, Object> context) throws GenericServiceException {
+    private Map<String, Object> runOne(ServiceDispatcher dispatcher, String localName, Map<String, Object> context)
+            throws GenericServiceException {
         Map<String, Object> result = null;
-        for (GroupServiceModel model: services) {
+        for (GroupServiceModel model : services) {
             try {
                 result = model.invoke(dispatcher, localName, context);
             } catch (GenericServiceException e) {

Modified: ofbiz/branches/json-integration-refactoring/framework/service/src/org/ofbiz/service/group/GroupServiceModel.java
URL: http://svn.apache.org/viewvc/ofbiz/branches/json-integration-refactoring/framework/service/src/org/ofbiz/service/group/GroupServiceModel.java?rev=1635897&r1=1635896&r2=1635897&view=diff
==============================================================================
--- ofbiz/branches/json-integration-refactoring/framework/service/src/org/ofbiz/service/group/GroupServiceModel.java (original)
+++ ofbiz/branches/json-integration-refactoring/framework/service/src/org/ofbiz/service/group/GroupServiceModel.java Sat Nov  1 07:15:09 2014
@@ -18,16 +18,15 @@
  *******************************************************************************/
 package org.ofbiz.service.group;
 
+import java.util.HashMap;
 import java.util.List;
 import java.util.Map;
 
-import javolution.util.FastMap;
-
+import org.ofbiz.base.util.Debug;
 import org.ofbiz.service.DispatchContext;
 import org.ofbiz.service.GenericServiceException;
 import org.ofbiz.service.ModelService;
 import org.ofbiz.service.ServiceDispatcher;
-import org.ofbiz.base.util.Debug;
 import org.w3c.dom.Element;
 
 /**
@@ -117,7 +116,7 @@ public class GroupServiceModel {
                 return dispatcher.runSync(localName, model, thisContext);
             } else {
                 dispatcher.runAsync(localName, model, thisContext, false);
-                return FastMap.newInstance();
+                return new HashMap<String, Object>();
             }
         } else {
             return dispatcher.runSync(localName, model, thisContext);

Modified: ofbiz/branches/json-integration-refactoring/framework/service/src/org/ofbiz/service/group/ServiceGroupReader.java
URL: http://svn.apache.org/viewvc/ofbiz/branches/json-integration-refactoring/framework/service/src/org/ofbiz/service/group/ServiceGroupReader.java?rev=1635897&r1=1635896&r2=1635897&view=diff
==============================================================================
--- ofbiz/branches/json-integration-refactoring/framework/service/src/org/ofbiz/service/group/ServiceGroupReader.java (original)
+++ ofbiz/branches/json-integration-refactoring/framework/service/src/org/ofbiz/service/group/ServiceGroupReader.java Sat Nov  1 07:15:09 2014
@@ -20,8 +20,7 @@ package org.ofbiz.service.group;
 
 import java.util.List;
 import java.util.Map;
-
-import javolution.util.FastMap;
+import java.util.concurrent.ConcurrentHashMap;
 
 import org.ofbiz.base.component.ComponentConfig;
 import org.ofbiz.base.config.GenericConfigException;
@@ -41,7 +40,7 @@ public class ServiceGroupReader {
     public static final String module = ServiceGroupReader.class.getName();
 
     // using a cache is dangerous here because if someone clears it the groups won't work at all: public static UtilCache groupsCache = new UtilCache("service.ServiceGroups", 0, 0, false);
-    public static Map<String, GroupModel> groupsCache = FastMap.newInstance();
+    public static Map<String, GroupModel> groupsCache = new ConcurrentHashMap<String, GroupModel>();
 
     public static void readConfig() {
         List<ServiceGroups> serviceGroupsList = null;

Modified: ofbiz/branches/json-integration-refactoring/framework/service/src/org/ofbiz/service/jms/JmsListenerFactory.java
URL: http://svn.apache.org/viewvc/ofbiz/branches/json-integration-refactoring/framework/service/src/org/ofbiz/service/jms/JmsListenerFactory.java?rev=1635897&r1=1635896&r2=1635897&view=diff
==============================================================================
--- ofbiz/branches/json-integration-refactoring/framework/service/src/org/ofbiz/service/jms/JmsListenerFactory.java (original)
+++ ofbiz/branches/json-integration-refactoring/framework/service/src/org/ofbiz/service/jms/JmsListenerFactory.java Sat Nov  1 07:15:09 2014
@@ -21,10 +21,9 @@ package org.ofbiz.service.jms;
 import java.lang.reflect.Constructor;
 import java.util.List;
 import java.util.Map;
+import java.util.concurrent.ConcurrentHashMap;
 import java.util.concurrent.atomic.AtomicReference;
 
-import javolution.util.FastMap;
-
 import org.ofbiz.base.util.Debug;
 import org.ofbiz.base.util.UtilGenerics;
 import org.ofbiz.base.util.UtilMisc;
@@ -45,8 +44,8 @@ public class JmsListenerFactory implemen
     public static final String TOPIC_LISTENER_CLASS = "org.ofbiz.service.jms.JmsTopicListener";
     public static final String QUEUE_LISTENER_CLASS = "org.ofbiz.service.jms.JmsQueueListener";
 
-    protected static Map<String, GenericMessageListener> listeners = FastMap.newInstance();
-    protected static Map<String, Server> servers = FastMap.newInstance();
+    protected static Map<String, GenericMessageListener> listeners = new ConcurrentHashMap<String, GenericMessageListener>();
+    protected static Map<String, Server> servers = new ConcurrentHashMap<String, Server>();
 
     private static final AtomicReference<JmsListenerFactory> jlFactoryRef = new AtomicReference<JmsListenerFactory>(null);
 

Modified: ofbiz/branches/json-integration-refactoring/framework/service/src/org/ofbiz/service/jms/JmsServiceEngine.java
URL: http://svn.apache.org/viewvc/ofbiz/branches/json-integration-refactoring/framework/service/src/org/ofbiz/service/jms/JmsServiceEngine.java?rev=1635897&r1=1635896&r2=1635897&view=diff
==============================================================================
--- ofbiz/branches/json-integration-refactoring/framework/service/src/org/ofbiz/service/jms/JmsServiceEngine.java (original)
+++ ofbiz/branches/json-integration-refactoring/framework/service/src/org/ofbiz/service/jms/JmsServiceEngine.java Sat Nov  1 07:15:09 2014
@@ -19,6 +19,7 @@
 package org.ofbiz.service.jms;
 
 import java.util.ArrayList;
+import java.util.HashMap;
 import java.util.List;
 import java.util.Map;
 
@@ -43,8 +44,6 @@ import javax.naming.InitialContext;
 import javax.naming.NamingException;
 import javax.transaction.xa.XAResource;
 
-import javolution.util.FastMap;
-
 import org.ofbiz.base.config.GenericConfigException;
 import org.ofbiz.base.util.Debug;
 import org.ofbiz.base.util.GeneralException;
@@ -310,7 +309,7 @@ public class JmsServiceEngine extends Ab
         JmsService serviceElement = getServiceElement(modelService);
         List<Server> serverList = serviceElement.getServers();
 
-        Map<String, Object> result = FastMap.newInstance();
+        Map<String, Object> result = new HashMap<String, Object>();
         for (Server server: serverList) {
             String serverType = server.getType();
             if (serverType.equals("topic"))

Modified: ofbiz/branches/json-integration-refactoring/framework/service/src/org/ofbiz/service/job/PersistedServiceJob.java
URL: http://svn.apache.org/viewvc/ofbiz/branches/json-integration-refactoring/framework/service/src/org/ofbiz/service/job/PersistedServiceJob.java?rev=1635897&r1=1635896&r2=1635897&view=diff
==============================================================================
--- ofbiz/branches/json-integration-refactoring/framework/service/src/org/ofbiz/service/job/PersistedServiceJob.java (original)
+++ ofbiz/branches/json-integration-refactoring/framework/service/src/org/ofbiz/service/job/PersistedServiceJob.java Sat Nov  1 07:15:09 2014
@@ -21,12 +21,11 @@ package org.ofbiz.service.job;
 import java.io.IOException;
 import java.sql.Timestamp;
 import java.util.Date;
+import java.util.HashMap;
 import java.util.Map;
 
 import javax.xml.parsers.ParserConfigurationException;
 
-import javolution.util.FastMap;
-
 import org.apache.commons.lang.StringUtils;
 import org.ofbiz.base.config.GenericConfigException;
 import org.ofbiz.base.util.Debug;
@@ -291,7 +290,7 @@ public class PersistedServiceJob extends
                 }
             }
             if (context == null) {
-                context = FastMap.newInstance();
+                context = new HashMap<String, Object>();
             }
             // check the runAsUser
             if (!UtilValidate.isEmpty(jobValue.getString("runAsUser"))) {

Modified: ofbiz/branches/json-integration-refactoring/framework/service/src/org/ofbiz/service/mail/MimeMessageWrapper.java
URL: http://svn.apache.org/viewvc/ofbiz/branches/json-integration-refactoring/framework/service/src/org/ofbiz/service/mail/MimeMessageWrapper.java?rev=1635897&r1=1635896&r2=1635897&view=diff
==============================================================================
--- ofbiz/branches/json-integration-refactoring/framework/service/src/org/ofbiz/service/mail/MimeMessageWrapper.java (original)
+++ ofbiz/branches/json-integration-refactoring/framework/service/src/org/ofbiz/service/mail/MimeMessageWrapper.java Sat Nov  1 07:15:09 2014
@@ -18,12 +18,13 @@
  *******************************************************************************/
 package org.ofbiz.service.mail;
 
+import java.io.ByteArrayInputStream;
 import java.io.ByteArrayOutputStream;
 import java.io.IOException;
-import java.io.ByteArrayInputStream;
 import java.io.InputStream;
 import java.nio.ByteBuffer;
 import java.sql.Timestamp;
+import java.util.LinkedList;
 import java.util.List;
 import java.util.Properties;
 
@@ -36,8 +37,6 @@ import javax.mail.Part;
 import javax.mail.Session;
 import javax.mail.internet.MimeMessage;
 
-import javolution.util.FastList;
-
 import org.ofbiz.base.conversion.AbstractConverter;
 import org.ofbiz.base.conversion.ConversionException;
 import org.ofbiz.base.conversion.Converters;
@@ -248,7 +247,7 @@ public class MimeMessageWrapper implemen
     }
 
     public List<String> getAttachmentIndexes() {
-        List<String> attachments = FastList.newInstance();
+        List<String> attachments = new LinkedList<String>();
         if (getMainPartCount() == 0) { // single part message (no attachments)
             return attachments;
         } else {

Modified: ofbiz/branches/json-integration-refactoring/framework/service/src/org/ofbiz/service/mail/ServiceMcaAction.java
URL: http://svn.apache.org/viewvc/ofbiz/branches/json-integration-refactoring/framework/service/src/org/ofbiz/service/mail/ServiceMcaAction.java?rev=1635897&r1=1635896&r2=1635897&view=diff
==============================================================================
--- ofbiz/branches/json-integration-refactoring/framework/service/src/org/ofbiz/service/mail/ServiceMcaAction.java (original)
+++ ofbiz/branches/json-integration-refactoring/framework/service/src/org/ofbiz/service/mail/ServiceMcaAction.java Sat Nov  1 07:15:09 2014
@@ -18,18 +18,16 @@
  *******************************************************************************/
 package org.ofbiz.service.mail;
 
+import java.util.HashMap;
 import java.util.Map;
 
-import javolution.util.FastMap;
-
-import org.ofbiz.service.GenericServiceException;
-import org.ofbiz.service.LocalDispatcher;
-import org.ofbiz.service.ServiceUtil;
-import org.ofbiz.base.util.UtilMisc;
 import org.ofbiz.base.util.Debug;
+import org.ofbiz.base.util.UtilMisc;
 import org.ofbiz.base.util.UtilValidate;
 import org.ofbiz.entity.GenericValue;
-
+import org.ofbiz.service.GenericServiceException;
+import org.ofbiz.service.LocalDispatcher;
+import org.ofbiz.service.ServiceUtil;
 import org.w3c.dom.Element;
 
 @SuppressWarnings("serial")
@@ -54,7 +52,7 @@ public class ServiceMcaAction implements
     }
 
     public boolean runAction(LocalDispatcher dispatcher, MimeMessageWrapper messageWrapper, GenericValue userLogin) throws GenericServiceException {
-        Map<String, Object> serviceContext = FastMap.newInstance();
+        Map<String, Object> serviceContext = new HashMap<String, Object>();
         serviceContext.putAll(UtilMisc.toMap("messageWrapper", messageWrapper, "userLogin", userLogin));
         serviceContext.put("userLogin", ServiceUtil.getUserLogin(dispatcher.getDispatchContext(), serviceContext, runAsUser));
 

Modified: ofbiz/branches/json-integration-refactoring/framework/service/src/org/ofbiz/service/mail/ServiceMcaCondition.java
URL: http://svn.apache.org/viewvc/ofbiz/branches/json-integration-refactoring/framework/service/src/org/ofbiz/service/mail/ServiceMcaCondition.java?rev=1635897&r1=1635896&r2=1635897&view=diff
==============================================================================
--- ofbiz/branches/json-integration-refactoring/framework/service/src/org/ofbiz/service/mail/ServiceMcaCondition.java (original)
+++ ofbiz/branches/json-integration-refactoring/framework/service/src/org/ofbiz/service/mail/ServiceMcaCondition.java Sat Nov  1 07:15:09 2014
@@ -19,8 +19,10 @@
 package org.ofbiz.service.mail;
 
 import java.io.IOException;
+import java.util.LinkedList;
 import java.util.List;
 import java.util.Map;
+
 import javax.mail.Address;
 import javax.mail.BodyPart;
 import javax.mail.MessagingException;
@@ -28,15 +30,12 @@ import javax.mail.Multipart;
 import javax.mail.Part;
 import javax.mail.internet.MimeMessage;
 
-import javolution.util.FastList;
-
 import org.ofbiz.base.util.Debug;
 import org.ofbiz.base.util.UtilMisc;
-import org.ofbiz.service.LocalDispatcher;
+import org.ofbiz.entity.GenericValue;
 import org.ofbiz.service.GenericServiceException;
+import org.ofbiz.service.LocalDispatcher;
 import org.ofbiz.service.ServiceUtil;
-import org.ofbiz.entity.GenericValue;
-
 import org.w3c.dom.Element;
 
 @SuppressWarnings("serial")
@@ -250,7 +249,7 @@ public class ServiceMcaCondition impleme
         if (c instanceof String) {
             return UtilMisc.toList((String) c);
         } else if (c instanceof Multipart) {
-            List<String> textContent = FastList.newInstance();
+            List<String> textContent = new LinkedList<String>();
             int count = ((Multipart) c).getCount();
             for (int i = 0; i < count; i++) {
                 BodyPart bp = ((Multipart) c).getBodyPart(i);
@@ -258,7 +257,7 @@ public class ServiceMcaCondition impleme
             }
             return textContent;
         } else {
-            return FastList.newInstance();
+            return new LinkedList<String>();
         }
     }
 }

Modified: ofbiz/branches/json-integration-refactoring/framework/service/src/org/ofbiz/service/rmi/ExampleRemoteClient.java
URL: http://svn.apache.org/viewvc/ofbiz/branches/json-integration-refactoring/framework/service/src/org/ofbiz/service/rmi/ExampleRemoteClient.java?rev=1635897&r1=1635896&r2=1635897&view=diff
==============================================================================
--- ofbiz/branches/json-integration-refactoring/framework/service/src/org/ofbiz/service/rmi/ExampleRemoteClient.java (original)
+++ ofbiz/branches/json-integration-refactoring/framework/service/src/org/ofbiz/service/rmi/ExampleRemoteClient.java Sat Nov  1 07:15:09 2014
@@ -18,15 +18,13 @@
  *******************************************************************************/
 package org.ofbiz.service.rmi;
 
+import java.net.MalformedURLException;
 import java.rmi.Naming;
 import java.rmi.NotBoundException;
 import java.rmi.RemoteException;
-import java.net.MalformedURLException;
+import java.util.HashMap;
 import java.util.Map;
 
-import javolution.util.FastMap;
-
-import org.ofbiz.service.rmi.RemoteDispatcher;
 import org.ofbiz.service.GenericServiceException;
 
 /** An example of how to remotely access the Service Engine's RemoteDispatcher.
@@ -67,7 +65,7 @@ public class ExampleRemoteClient {
     }
 
     public Map<String, Object> runTestService() throws RemoteException, GenericServiceException {
-        Map<String, Object> context = FastMap.newInstance();
+        Map<String, Object> context = new HashMap<String, Object>();
         context.put("message", "Remote Service Test");
         return rd.runSync("testScv", context);
     }

Modified: ofbiz/branches/json-integration-refactoring/framework/service/src/org/ofbiz/service/test/ServiceEngineTestServices.java
URL: http://svn.apache.org/viewvc/ofbiz/branches/json-integration-refactoring/framework/service/src/org/ofbiz/service/test/ServiceEngineTestServices.java?rev=1635897&r1=1635896&r2=1635897&view=diff
==============================================================================
--- ofbiz/branches/json-integration-refactoring/framework/service/src/org/ofbiz/service/test/ServiceEngineTestServices.java (original)
+++ ofbiz/branches/json-integration-refactoring/framework/service/src/org/ofbiz/service/test/ServiceEngineTestServices.java Sat Nov  1 07:15:09 2014
@@ -18,12 +18,11 @@
  */
 package org.ofbiz.service.test;
 
+import java.util.LinkedList;
 import java.util.List;
 import java.util.Locale;
 import java.util.Map;
 
-import javolution.util.FastList;
-
 import org.ofbiz.base.util.Debug;
 import org.ofbiz.base.util.UtilMisc;
 import org.ofbiz.base.util.UtilProperties;
@@ -53,7 +52,7 @@ public class ServiceEngineTestServices {
             // make sure to wait for these to both finish to make sure results aren't checked until they are done
             Map<String, Object> threadAResult = threadAWaiter.waitForResult();
             Map<String, Object> threadBResult = threadBWaiter.waitForResult();
-            List<Object> errorList = FastList.newInstance();
+            List<Object> errorList = new LinkedList<Object>();
             if (ServiceUtil.isError(threadAResult)) {
                 errorList.add(UtilProperties.getMessage(resource, "ServiceTestDeadLockThreadA", UtilMisc.toMap("errorString", ServiceUtil.getErrorMessage(threadAResult)), locale));
             }
@@ -148,7 +147,7 @@ public class ServiceEngineTestServices {
             // make sure to wait for these to both finish to make sure results aren't checked until they are done
             Map<String, Object> grabberResult = grabberWaiter.waitForResult();
             Map<String, Object> waiterResult = waiterWaiter.waitForResult();
-            List<Object> errorList = FastList.newInstance();
+            List<Object> errorList = new LinkedList<Object>();
             if (ServiceUtil.isError(grabberResult)) {
                 errorList.add("Error running testServiceLockWaitTimeoutRetryGrabber: " + ServiceUtil.getErrorMessage(grabberResult));
             }

Modified: ofbiz/branches/json-integration-refactoring/framework/service/src/org/ofbiz/service/test/ServiceSOAPTests.java
URL: http://svn.apache.org/viewvc/ofbiz/branches/json-integration-refactoring/framework/service/src/org/ofbiz/service/test/ServiceSOAPTests.java?rev=1635897&r1=1635896&r2=1635897&view=diff
==============================================================================
--- ofbiz/branches/json-integration-refactoring/framework/service/src/org/ofbiz/service/test/ServiceSOAPTests.java (original)
+++ ofbiz/branches/json-integration-refactoring/framework/service/src/org/ofbiz/service/test/ServiceSOAPTests.java Sat Nov  1 07:15:09 2014
@@ -18,11 +18,10 @@
  */
 package org.ofbiz.service.test;
 
+import java.util.HashMap;
 import java.util.List;
 import java.util.Map;
 
-import javolution.util.FastMap;
-
 import org.ofbiz.base.util.UtilDateTime;
 import org.ofbiz.base.util.UtilGenerics;
 import org.ofbiz.entity.GenericValue;
@@ -38,14 +37,14 @@ public class ServiceSOAPTests extends OF
     }
 
     public void testSOAPSimpleService() throws Exception {
-        Map<String, Object> serviceContext = FastMap.newInstance();
+        Map<String, Object> serviceContext = new HashMap<String, Object>();
         serviceContext.put("defaultValue", new Double("123.4567"));
         serviceContext.put("message", "Test Message !!!");
         dispatcher.runSync("testSoapSimple", serviceContext);
     }
 
     public void testSOAPService() throws Exception {
-        Map<String, Object> serviceContext = FastMap.newInstance();
+        Map<String, Object> serviceContext = new HashMap<String, Object>();
         GenericValue testing = delegator.makeValue("Testing");
         testing.put("testingId", "COMPLEX_TYPE_TEST");
         testing.put("testingTypeId", "SOAP_TEST");

Modified: ofbiz/branches/json-integration-refactoring/framework/widget/src/org/ofbiz/widget/ModelWidgetAction.java
URL: http://svn.apache.org/viewvc/ofbiz/branches/json-integration-refactoring/framework/widget/src/org/ofbiz/widget/ModelWidgetAction.java?rev=1635897&r1=1635896&r2=1635897&view=diff
==============================================================================
--- ofbiz/branches/json-integration-refactoring/framework/widget/src/org/ofbiz/widget/ModelWidgetAction.java (original)
+++ ofbiz/branches/json-integration-refactoring/framework/widget/src/org/ofbiz/widget/ModelWidgetAction.java Sat Nov  1 07:15:09 2014
@@ -21,6 +21,8 @@ package org.ofbiz.widget;
 import java.io.Serializable;
 import java.text.MessageFormat;
 import java.util.ArrayList;
+import java.util.HashMap;
+import java.util.LinkedList;
 import java.util.List;
 import java.util.Locale;
 import java.util.Map;
@@ -30,10 +32,6 @@ import java.util.regex.PatternSyntaxExce
 import javax.servlet.ServletContext;
 import javax.servlet.http.HttpSession;
 
-import javolution.util.FastList;
-import javolution.util.FastMap;
-
-import org.w3c.dom.Element;
 import org.ofbiz.base.util.Debug;
 import org.ofbiz.base.util.GeneralException;
 import org.ofbiz.base.util.ObjectType;
@@ -59,6 +57,7 @@ import org.ofbiz.minilang.method.MethodC
 import org.ofbiz.service.DispatchContext;
 import org.ofbiz.service.GenericServiceException;
 import org.ofbiz.service.ModelService;
+import org.w3c.dom.Element;
 
 @SuppressWarnings("serial")
 public abstract class ModelWidgetAction implements Serializable {
@@ -140,6 +139,7 @@ public abstract class ModelWidgetAction 
             }
         }
 
+        @SuppressWarnings("rawtypes")
         @Override
         public void runAction(Map<String, Object> context) {
             String globalStr = this.globalExdr.expandString(context);
@@ -179,9 +179,9 @@ public abstract class ModelWidgetAction 
 
             if (UtilValidate.isNotEmpty(this.type)) {
                 if ("NewMap".equals(this.type)) {
-                    newValue = FastMap.newInstance();
+                    newValue = new HashMap();
                 } else if ("NewList".equals(this.type)) {
-                    newValue = FastList.newInstance();
+                    newValue = new LinkedList();
                 } else {
                     try {
                         newValue = ObjectType.simpleTypeConvert(newValue, this.type, null, (TimeZone) context.get("timeZone"), (Locale) context.get("locale"), true);
@@ -400,7 +400,7 @@ public abstract class ModelWidgetAction 
         @Override
         public void runAction(Map<String, Object> context) throws GeneralException {
             if (location.endsWith(".xml")) {
-                Map<String, Object> localContext = FastMap.newInstance();
+                Map<String, Object> localContext = new HashMap<String, Object>();
                 localContext.putAll(context);
                 DispatchContext ctx = WidgetWorker.getDispatcher(context).getDispatchContext();
                 MethodContext methodContext = new MethodContext(ctx, localContext, null);
@@ -445,7 +445,7 @@ public abstract class ModelWidgetAction 
                 if ("true".equals(autoFieldMapString)) {
                     DispatchContext dc = WidgetWorker.getDispatcher(context).getDispatchContext();
                     // try a map called "parameters", try it first so values from here are overriden by values in the main context
-                    Map<String, Object> combinedMap = FastMap.newInstance();
+                    Map<String, Object> combinedMap = new HashMap<String, Object>();
                     Map<String, Object> parametersObj = UtilGenerics.toMap(context.get("parameters"));
                     if (parametersObj != null) {
                         combinedMap.putAll(parametersObj);
@@ -460,7 +460,7 @@ public abstract class ModelWidgetAction 
                     }
                 }
                 if (serviceContext == null) {
-                    serviceContext = FastMap.newInstance();
+                    serviceContext = new HashMap<String, Object>();
                 }
 
                 if (this.fieldMap != null) {

Modified: ofbiz/branches/json-integration-refactoring/framework/widget/src/org/ofbiz/widget/WidgetFactory.java
URL: http://svn.apache.org/viewvc/ofbiz/branches/json-integration-refactoring/framework/widget/src/org/ofbiz/widget/WidgetFactory.java?rev=1635897&r1=1635896&r2=1635897&view=diff
==============================================================================
--- ofbiz/branches/json-integration-refactoring/framework/widget/src/org/ofbiz/widget/WidgetFactory.java (original)
+++ ofbiz/branches/json-integration-refactoring/framework/widget/src/org/ofbiz/widget/WidgetFactory.java Sat Nov  1 07:15:09 2014
@@ -24,8 +24,7 @@ import java.lang.reflect.Modifier;
 import java.util.Iterator;
 import java.util.Map;
 import java.util.ServiceLoader;
-
-import javolution.util.FastMap;
+import java.util.concurrent.ConcurrentHashMap;
 
 import org.ofbiz.base.util.Assert;
 import org.ofbiz.base.util.Debug;
@@ -42,7 +41,7 @@ import org.w3c.dom.Element;
 public class WidgetFactory {
 
     public static final String module = WidgetFactory.class.getName();
-    protected static final Map<String, Constructor<? extends ModelScreenWidget>> screenWidgets = FastMap.newInstance();
+    protected static final Map<String, Constructor<? extends ModelScreenWidget>> screenWidgets = new ConcurrentHashMap<String, Constructor<? extends ModelScreenWidget>>();
 
     static {
         loadStandardWidgets();

Modified: ofbiz/branches/json-integration-refactoring/framework/widget/src/org/ofbiz/widget/WidgetWorker.java
URL: http://svn.apache.org/viewvc/ofbiz/branches/json-integration-refactoring/framework/widget/src/org/ofbiz/widget/WidgetWorker.java?rev=1635897&r1=1635896&r2=1635897&view=diff
==============================================================================
--- ofbiz/branches/json-integration-refactoring/framework/widget/src/org/ofbiz/widget/WidgetWorker.java (original)
+++ ofbiz/branches/json-integration-refactoring/framework/widget/src/org/ofbiz/widget/WidgetWorker.java Sat Nov  1 07:15:09 2014
@@ -25,6 +25,8 @@ import java.math.BigDecimal;
 import java.net.URLEncoder;
 import java.nio.charset.Charset;
 import java.text.DateFormat;
+import java.util.ArrayList;
+import java.util.HashMap;
 import java.util.Iterator;
 import java.util.List;
 import java.util.Map;
@@ -34,9 +36,6 @@ import javax.servlet.ServletContext;
 import javax.servlet.http.HttpServletRequest;
 import javax.servlet.http.HttpServletResponse;
 
-import javolution.util.FastList;
-import javolution.util.FastMap;
-
 import org.ofbiz.base.util.Debug;
 import org.ofbiz.base.util.StringUtil;
 import org.ofbiz.base.util.UtilDateTime;
@@ -414,7 +413,7 @@ public class WidgetWorker {
 
     public static class AutoServiceParameters {
         private String serviceName;
-        List<String> excludeList = FastList.newInstance();
+        List<String> excludeList = new ArrayList<String>();
         boolean includePk;
         boolean includeNonPk;
         boolean sendIfEmpty;
@@ -433,7 +432,7 @@ public class WidgetWorker {
 
         @SuppressWarnings("unchecked")
         public Map<String, String> getParametersMap(Map<String, Object> context, String defaultServiceName) {
-            Map<String, String> autServiceParams = FastMap.newInstance();
+            Map<String, String> autServiceParams = new HashMap<String, String>();
             LocalDispatcher dispatcher = (LocalDispatcher) context.get("dispatcher");
             if (dispatcher == null) {
                 Debug.logError("We can not append auto service Parameters since we could not find dispatcher in the current context", module);
@@ -477,7 +476,7 @@ public class WidgetWorker {
     public static class AutoEntityParameters {
         private String entityName;
         private String includeType;
-        List<String> excludeList = FastList.newInstance();
+        List<String> excludeList = new ArrayList<String>();
         boolean includePk;
         boolean includeNonPk;
         boolean sendIfEmpty;
@@ -499,7 +498,7 @@ public class WidgetWorker {
 
         @SuppressWarnings("unchecked")
         public Map<String, String> getParametersMap(Map<String, Object> context, String defaultEntityName) {
-            Map<String, String> autEntityParams = FastMap.newInstance();
+            Map<String, String> autEntityParams = new HashMap<String, String>();
             Delegator delegator = (Delegator) context.get("delegator");
             if (delegator == null) {
                 Debug.logError("We can not append auto entity Parameters since we could not find delegator in the current context", module);

Modified: ofbiz/branches/json-integration-refactoring/framework/widget/src/org/ofbiz/widget/cache/WidgetContextCacheKey.java
URL: http://svn.apache.org/viewvc/ofbiz/branches/json-integration-refactoring/framework/widget/src/org/ofbiz/widget/cache/WidgetContextCacheKey.java?rev=1635897&r1=1635896&r2=1635897&view=diff
==============================================================================
--- ofbiz/branches/json-integration-refactoring/framework/widget/src/org/ofbiz/widget/cache/WidgetContextCacheKey.java (original)
+++ ofbiz/branches/json-integration-refactoring/framework/widget/src/org/ofbiz/widget/cache/WidgetContextCacheKey.java Sat Nov  1 07:15:09 2014
@@ -18,24 +18,24 @@
  *******************************************************************************/
 package org.ofbiz.widget.cache;
 
+import java.util.Collections;
+import java.util.HashMap;
+import java.util.HashSet;
 import java.util.Map;
 import java.util.Set;
 
-import javolution.util.FastMap;
-import javolution.util.FastSet;
-
 import org.ofbiz.base.util.Debug;
 import org.ofbiz.base.util.UtilGenerics;
 import org.ofbiz.base.util.UtilMisc;
 
-public class WidgetContextCacheKey {
+public final class WidgetContextCacheKey {
 
     public static final String module = WidgetContextCacheKey.class.getName();
 
-    private static Set<String> fieldNamesToSkip;
+    private static Set<String> fieldNamesToSkip = createFieldNamesToSkip();
 
-    static {
-        fieldNamesToSkip = FastSet.newInstance();
+    private static Set<String> createFieldNamesToSkip(){
+        Set<String> fieldNamesToSkip = new HashSet<String>();
         fieldNamesToSkip.add("globalContext");
         fieldNamesToSkip.add("delegator");
         fieldNamesToSkip.add("dispatcher");
@@ -76,13 +76,13 @@ public class WidgetContextCacheKey {
         // parameters
         fieldNamesToSkip.add("visit");
         fieldNamesToSkip.add("visitor");
+        return Collections.unmodifiableSet(fieldNamesToSkip);
     }
 
-    protected Map<String, Object> context;
+    private final Map<String, Object> context;
 
     public WidgetContextCacheKey(Map<String, ? extends Object> context) {
-        this.context = FastMap.newInstance();
-        this.context.putAll(context);
+        this.context = Collections.unmodifiableMap(new HashMap<String, Object>(context));
     }
 
     @Override
@@ -103,7 +103,7 @@ public class WidgetContextCacheKey {
             return false;
         }
 
-        Set<String> unifiedContext = FastSet.newInstance();
+        Set<String> unifiedContext = new HashSet<String>();
         unifiedContext.addAll(this.context.keySet());
         unifiedContext.addAll(key.context.keySet());
         for (String fieldName: unifiedContext) {
@@ -135,7 +135,7 @@ public class WidgetContextCacheKey {
 
     @Override
     public String toString() {
-        Map<String, Object> printableMap = FastMap.newInstance();
+        Map<String, Object> printableMap = new HashMap<String, Object>();
         for (String fieldName: this.context.keySet()) {
             if (!fieldNamesToSkip.contains(fieldName) && !"parameters".equals(fieldName)) {
                 printableMap.put(fieldName, this.context.get(fieldName));
@@ -146,7 +146,7 @@ public class WidgetContextCacheKey {
     }
 
     public static String printMap(Map<String, ? extends Object> map) {
-        Map<String, Object> printableMap = FastMap.newInstance();
+        Map<String, Object> printableMap = new HashMap<String, Object>();
         for (Map.Entry<String, ? extends Object> entry : map.entrySet()) {
             String fieldName = entry.getKey();
             if (!fieldNamesToSkip.contains(fieldName) &&
@@ -160,7 +160,7 @@ public class WidgetContextCacheKey {
     }
 
     public static boolean parametersAreEqual(Map<String, ? extends Object> map1, Map<String, ? extends Object> map2) {
-        Set<String> unifiedContext = FastSet.newInstance();
+        Set<String> unifiedContext = new HashSet<String>();
         unifiedContext.addAll(map1.keySet());
         unifiedContext.addAll(map2.keySet());
         for (String fieldName: unifiedContext) {

Modified: ofbiz/branches/json-integration-refactoring/framework/widget/src/org/ofbiz/widget/form/MacroFormRenderer.java
URL: http://svn.apache.org/viewvc/ofbiz/branches/json-integration-refactoring/framework/widget/src/org/ofbiz/widget/form/MacroFormRenderer.java?rev=1635897&r1=1635896&r2=1635897&view=diff
==============================================================================
--- ofbiz/branches/json-integration-refactoring/framework/widget/src/org/ofbiz/widget/form/MacroFormRenderer.java (original)
+++ ofbiz/branches/json-integration-refactoring/framework/widget/src/org/ofbiz/widget/form/MacroFormRenderer.java Sat Nov  1 07:15:09 2014
@@ -26,6 +26,7 @@ import java.rmi.server.UID;
 import java.sql.Timestamp;
 import java.util.HashSet;
 import java.util.Iterator;
+import java.util.LinkedList;
 import java.util.List;
 import java.util.Locale;
 import java.util.Map;
@@ -37,8 +38,6 @@ import javax.servlet.ServletContext;
 import javax.servlet.http.HttpServletRequest;
 import javax.servlet.http.HttpServletResponse;
 
-import javolution.util.FastList;
-
 import org.ofbiz.base.util.Debug;
 import org.ofbiz.base.util.StringUtil;
 import org.ofbiz.base.util.UtilFormatOut;
@@ -87,18 +86,18 @@ import freemarker.template.TemplateExcep
  * Widget Library - Form Renderer implementation based on Freemarker macros
  *
  */
-public class MacroFormRenderer implements FormStringRenderer {
+public final class MacroFormRenderer implements FormStringRenderer {
 
     public static final String module = MacroFormRenderer.class.getName();
-    private Template macroLibrary;
-    private WeakHashMap<Appendable, Environment> environments = new WeakHashMap<Appendable, Environment>();
-    private StringUtil.SimpleEncoder internalEncoder;
-    protected RequestHandler rh;
-    protected HttpServletRequest request;
-    protected HttpServletResponse response;
-    protected boolean javaScriptEnabled = false;
-    protected boolean renderPagination = true;
-    protected boolean widgetCommentsEnabled = false;
+    private final Template macroLibrary;
+    private final WeakHashMap<Appendable, Environment> environments = new WeakHashMap<Appendable, Environment>();
+    private final StringUtil.SimpleEncoder internalEncoder;
+    private final RequestHandler rh;
+    private final HttpServletRequest request;
+    private final HttpServletResponse response;
+    private final boolean javaScriptEnabled;
+    private boolean renderPagination = true;
+    private boolean widgetCommentsEnabled = false;
 
     public MacroFormRenderer(String macroLibraryPath, HttpServletRequest request, HttpServletResponse response) throws TemplateException, IOException {
         macroLibrary = FreeMarkerWorker.getTemplate(macroLibraryPath);
@@ -1087,7 +1086,7 @@ public class MacroFormRenderer implement
         String backgroundSubmitRefreshTarget = submitField.getBackgroundSubmitRefreshTarget(context);
         if (UtilValidate.isNotEmpty(backgroundSubmitRefreshTarget)) {
             if (updateAreas == null) {
-                updateAreas = FastList.newInstance();
+                updateAreas = new LinkedList<ModelForm.UpdateArea>();
             }
             updateAreas.add(new ModelForm.UpdateArea("submit", formId, backgroundSubmitRefreshTarget));
         }
@@ -1414,8 +1413,8 @@ public class MacroFormRenderer implement
             this.renderNextPrev(writer, context, modelForm);
         }
         List<ModelFormField> childFieldList = modelForm.getFieldList();
-        List<String> columnStyleList = FastList.newInstance();
-        List<String> fieldNameList = FastList.newInstance();
+        List<String> columnStyleList = new LinkedList<String>();
+        List<String> fieldNameList = new LinkedList<String>();
         for (ModelFormField childField : childFieldList) {
             int childFieldType = childField.getFieldInfo().getFieldType();
             if (childFieldType == ModelFormField.FieldInfo.HIDDEN || childFieldType == ModelFormField.FieldInfo.IGNORED) {
@@ -2012,7 +2011,7 @@ public class MacroFormRenderer implement
                 autoCompleterTarget = lookupFieldFormName + "&amp;amp;";
             }
             autoCompleterTarget = autoCompleterTarget + "ajaxLookup=Y";
-            updateAreas = FastList.newInstance();
+            updateAreas = new LinkedList<ModelForm.UpdateArea>();
             updateAreas.add(new ModelForm.UpdateArea("change", id, autoCompleterTarget));
         }
         boolean ajaxEnabled = updateAreas != null && this.javaScriptEnabled;

Modified: ofbiz/branches/json-integration-refactoring/framework/widget/src/org/ofbiz/widget/form/ModelForm.java
URL: http://svn.apache.org/viewvc/ofbiz/branches/json-integration-refactoring/framework/widget/src/org/ofbiz/widget/form/ModelForm.java?rev=1635897&r1=1635896&r2=1635897&view=diff
==============================================================================
--- ofbiz/branches/json-integration-refactoring/framework/widget/src/org/ofbiz/widget/form/ModelForm.java (original)
+++ ofbiz/branches/json-integration-refactoring/framework/widget/src/org/ofbiz/widget/form/ModelForm.java Sat Nov  1 07:15:09 2014
@@ -21,7 +21,10 @@ package org.ofbiz.widget.form;
 import java.io.IOException;
 import java.util.ArrayList;
 import java.util.Collection;
+import java.util.HashMap;
+import java.util.HashSet;
 import java.util.Iterator;
+import java.util.LinkedList;
 import java.util.List;
 import java.util.Locale;
 import java.util.Map;
@@ -30,10 +33,6 @@ import java.util.Set;
 import java.util.TreeMap;
 import java.util.TreeSet;
 
-import javolution.util.FastList;
-import javolution.util.FastMap;
-import javolution.util.FastSet;
-
 import org.ofbiz.base.util.BshUtil;
 import org.ofbiz.base.util.Debug;
 import org.ofbiz.base.util.GeneralException;
@@ -131,12 +130,12 @@ public class ModelForm extends ModelWidg
     protected boolean overridenListSize = false;
     protected boolean clientAutocompleteFields = true;
 
-    protected List<AltTarget> altTargets = FastList.newInstance();
-    protected List<AutoFieldsService> autoFieldsServices = FastList.newInstance();
-    protected List<AutoFieldsEntity> autoFieldsEntities = FastList.newInstance();
-    protected List<String> lastOrderFields = FastList.newInstance();
-    protected List<SortField> sortOrderFields = FastList.newInstance();
-    protected List<AltRowStyle> altRowStyles = FastList.newInstance();
+    protected List<AltTarget> altTargets = new ArrayList<AltTarget>();
+    protected List<AutoFieldsService> autoFieldsServices = new ArrayList<AutoFieldsService>();
+    protected List<AutoFieldsEntity> autoFieldsEntities = new ArrayList<AutoFieldsEntity>();
+    protected List<String> lastOrderFields = new ArrayList<String>();
+    protected List<SortField> sortOrderFields = new ArrayList<SortField>();
+    protected List<AltRowStyle> altRowStyles = new ArrayList<AltRowStyle>();
 
     /** This List will contain one copy of each field for each field name in the order
      * they were encountered in the service, entity, or form definition; field definitions
@@ -147,25 +146,25 @@ public class ModelForm extends ModelWidg
      * necessary to use the Map. The Map is used when loading the form definition to keep the
      * list clean and implement the override features for field definitions.
      */
-    protected List<ModelFormField> fieldList = FastList.newInstance();
+    protected List<ModelFormField> fieldList = new ArrayList<ModelFormField>();
 
     /** This Map is keyed with the field name and has a ModelFormField for the value.
      */
-    protected Map<String, ModelFormField> fieldMap = FastMap.newInstance();
+    protected Map<String, ModelFormField> fieldMap = new HashMap<String, ModelFormField>();
 
     /** Keeps track of conditional fields to help ensure that only one is rendered
      */
-    protected Set<String> useWhenFields = FastSet.newInstance();
+    protected Set<String> useWhenFields = new HashSet<String>();
 
     /** This is a list of FieldGroups in the order they were created.
      * Can also include Banner objects.
      */
-    protected List<FieldGroupBase> fieldGroupList = FastList.newInstance();
+    protected List<FieldGroupBase> fieldGroupList = new ArrayList<FieldGroupBase>();
 
     /** This Map is keyed with the field name and has a FieldGroup for the value.
      * Can also include Banner objects.
      */
-    protected Map<String, FieldGroupBase> fieldGroupMap = FastMap.newInstance();
+    protected Map<String, FieldGroupBase> fieldGroupMap = new HashMap<String, FieldGroupBase>();
 
     /** This field group will be the "catch-all" group for fields that are not
      *  included in an explicit field-group.
@@ -192,7 +191,7 @@ public class ModelForm extends ModelWidg
     protected List<ModelFormAction> actions;
     protected List<ModelFormAction> rowActions;
     protected FlexibleStringExpander rowCountExdr;
-    protected List<ModelFormField> multiSubmitFields = FastList.newInstance();
+    protected List<ModelFormField> multiSubmitFields = new ArrayList<ModelFormField>();
     protected int rowCount = 0;
     private String sortFieldParameterName = "sortField";
 
@@ -571,7 +570,7 @@ public class ModelForm extends ModelWidg
 
         // reorder fields according to sort order
         if (sortOrderFields.size() > 0) {
-            List<ModelFormField> sortedFields = FastList.newInstance();
+            List<ModelFormField> sortedFields = new LinkedList<ModelFormField>();
             for (SortField sortField: this.sortOrderFields) {
                 String fieldName = sortField.getFieldName();
                 if (UtilValidate.isEmpty(fieldName)) {
@@ -599,7 +598,7 @@ public class ModelForm extends ModelWidg
         }
 
         if (UtilValidate.isNotEmpty(this.lastOrderFields)) {
-            List<ModelFormField> lastedFields = FastList.newInstance();
+            List<ModelFormField> lastedFields = new LinkedList<ModelFormField>();
             for (String fieldName: this.lastOrderFields) {
                 if (UtilValidate.isEmpty(fieldName)) {
                     continue;
@@ -695,7 +694,7 @@ public class ModelForm extends ModelWidg
 
     protected void addOnSubmitUpdateArea(UpdateArea updateArea) {
         if (onSubmitUpdateAreas == null) {
-            onSubmitUpdateAreas = FastList.newInstance();
+            onSubmitUpdateAreas = new ArrayList<UpdateArea>();
         }
         int index = onSubmitUpdateAreas.indexOf(updateArea);
         if (index != -1) {
@@ -707,7 +706,7 @@ public class ModelForm extends ModelWidg
 
     protected void addOnPaginateUpdateArea(UpdateArea updateArea) {
         if (onPaginateUpdateAreas == null) {
-            onPaginateUpdateAreas = FastList.newInstance();
+            onPaginateUpdateAreas = new ArrayList<UpdateArea>();
         }
         int index = onPaginateUpdateAreas.indexOf(updateArea);
         if (index != -1) {
@@ -724,7 +723,7 @@ public class ModelForm extends ModelWidg
 
     protected void addOnSortColumnUpdateArea(UpdateArea updateArea) {
         if (onSortColumnUpdateAreas == null) {
-            onSortColumnUpdateAreas = FastList.newInstance();
+            onSortColumnUpdateAreas = new ArrayList<UpdateArea>();
         }
         int index = onSortColumnUpdateAreas.indexOf(updateArea);
         if (index != -1) {
@@ -916,7 +915,7 @@ public class ModelForm extends ModelWidg
     }
 
     public void renderSingleFormString(Appendable writer, Map<String, Object> context, FormStringRenderer formStringRenderer, int positions) throws IOException {
-        List<ModelFormField> tempFieldList = FastList.newInstance();
+        List<ModelFormField> tempFieldList = new LinkedList<ModelFormField>();
         tempFieldList.addAll(this.fieldList);
 
         // Check to see if there is a field, same name and same use-when (could come from extended form)
@@ -1212,7 +1211,7 @@ public class ModelForm extends ModelWidg
         // in this model: we can have more fields with the same name when use-when
         // conditions are used or when a form is extended or when the fields are
         // automatically retrieved by a service or entity definition.
-        List<ModelFormField> tempFieldList = FastList.newInstance();
+        List<ModelFormField> tempFieldList = new LinkedList<ModelFormField>();
         tempFieldList.addAll(this.fieldList);
         for (int j = 0; j < tempFieldList.size(); j++) {
             ModelFormField modelFormField = tempFieldList.get(j);
@@ -1230,13 +1229,13 @@ public class ModelForm extends ModelWidg
         // We get a sorted (by position, ascending) set of lists;
         // each list contains all the fields with that position.
         Collection<List<ModelFormField>> fieldListsByPosition = this.getFieldListsByPosition(tempFieldList);
-        List<Map<String, List<ModelFormField>>> fieldRowsByPosition = FastList.newInstance(); // this list will contain maps, each one containing the list of fields for a position
+        List<Map<String, List<ModelFormField>>> fieldRowsByPosition = new LinkedList<Map<String, List<ModelFormField>>>(); // this list will contain maps, each one containing the list of fields for a position
         for (List<ModelFormField> mainFieldList: fieldListsByPosition) {
             int numOfColumns = 0;
 
-            List<ModelFormField> innerDisplayHyperlinkFieldsBegin = FastList.newInstance();
-            List<ModelFormField> innerFormFields = FastList.newInstance();
-            List<ModelFormField> innerDisplayHyperlinkFieldsEnd = FastList.newInstance();
+            List<ModelFormField> innerDisplayHyperlinkFieldsBegin = new LinkedList<ModelFormField>();
+            List<ModelFormField> innerFormFields = new LinkedList<ModelFormField>();
+            List<ModelFormField> innerDisplayHyperlinkFieldsEnd = new LinkedList<ModelFormField>();
 
             // render title for each field, except hidden & ignored, etc
 
@@ -1522,7 +1521,7 @@ public class ModelForm extends ModelWidg
             int itemIndex = -1;
             Object item = null;
             context.put("wholeFormContext", context);
-            Map<String, Object> previousItem = FastMap.newInstance();
+            Map<String, Object> previousItem = new HashMap<String, Object>();
             while ((item = this.safeNext(iter)) != null) {
                 itemIndex++;
                 if (itemIndex >= highIndex) {
@@ -1556,7 +1555,7 @@ public class ModelForm extends ModelWidg
                 this.resetBshInterpreter(localContext);
                 localContext.push();
                 localContext.put("previousItem", previousItem);
-                previousItem = FastMap.newInstance();
+                previousItem = new HashMap<String, Object>();
                 previousItem.putAll(itemMap);
 
                 ModelFormAction.runSubActions(this.rowActions, localContext);
@@ -1571,7 +1570,7 @@ public class ModelForm extends ModelWidg
                 if (Debug.verboseOn()) Debug.logVerbose("In form got another row, context is: " + localContext, module);
 
                 // Check to see if there is a field, same name and same use-when (could come from extended form)
-                List<ModelFormField> tempFieldList = FastList.newInstance();
+                List<ModelFormField> tempFieldList = new LinkedList<ModelFormField>();
                 tempFieldList.addAll(this.fieldList);
                 for (int j = 0; j < tempFieldList.size(); j++) {
                     ModelFormField modelFormField = tempFieldList.get(j);
@@ -1608,9 +1607,9 @@ public class ModelForm extends ModelWidg
                     // we have two phases: preprocessing and rendering
                     this.rowCount++;
 
-                    List<ModelFormField> innerDisplayHyperlinkFieldsBegin = FastList.newInstance();
-                    List<ModelFormField> innerFormFields = FastList.newInstance();
-                    List<ModelFormField> innerDisplayHyperlinkFieldsEnd = FastList.newInstance();
+                    List<ModelFormField> innerDisplayHyperlinkFieldsBegin = new LinkedList<ModelFormField>();
+                    List<ModelFormField> innerFormFields = new LinkedList<ModelFormField>();
+                    List<ModelFormField> innerDisplayHyperlinkFieldsEnd = new LinkedList<ModelFormField>();
 
                     // Preprocessing:
                     // all the form fields are evaluated and the ones that will
@@ -1737,7 +1736,7 @@ public class ModelForm extends ModelWidg
         // render row formatting open
         formStringRenderer.renderFormatItemRowOpen(writer, localContext, this);
         Iterator<ModelFormField> innerDisplayHyperlinkFieldsBeginIter = innerDisplayHyperlinkFieldsBegin.iterator();
-        Map<String, Integer> fieldCount = FastMap.newInstance();
+        Map<String, Integer> fieldCount = new HashMap<String, Integer>();
         while(innerDisplayHyperlinkFieldsBeginIter.hasNext()){
             ModelFormField modelFormField = innerDisplayHyperlinkFieldsBeginIter.next();
             if(fieldCount.containsKey(modelFormField.getFieldName())){
@@ -1862,7 +1861,7 @@ public class ModelForm extends ModelWidg
     }
 
     public List<ModelFormField> getHiddenIgnoredFields(Map<String, Object> context, Set<String> alreadyRendered, List<ModelFormField> fieldList, int position) {
-        List<ModelFormField> hiddenIgnoredFieldList = FastList.newInstance();
+        List<ModelFormField> hiddenIgnoredFieldList = new LinkedList<ModelFormField>();
         for (ModelFormField modelFormField: fieldList) {
             // with position == -1 then gets all the hidden fields
             if (position != -1 && modelFormField.getPosition() != position) {
@@ -1927,7 +1926,7 @@ public class ModelForm extends ModelWidg
             Integer position = Integer.valueOf(modelFormField.getPosition());
             List<ModelFormField> fieldListByPosition = fieldsByPosition.get(position);
             if (fieldListByPosition == null) {
-                fieldListByPosition = FastList.newInstance();
+                fieldListByPosition = new LinkedList<ModelFormField>();
                 fieldsByPosition.put(position, fieldListByPosition);
             }
             fieldListByPosition.add(modelFormField);
@@ -1936,7 +1935,7 @@ public class ModelForm extends ModelWidg
     }
 
     public List<ModelFormField> getFieldListByPosition(List<ModelFormField> modelFormFieldList, int position) {
-        List<ModelFormField> fieldListByPosition = FastList.newInstance();
+        List<ModelFormField> fieldListByPosition = new LinkedList<ModelFormField>();
         for (ModelFormField modelFormField: modelFormFieldList) {
             if (modelFormField.getPosition() == position) {
                 fieldListByPosition.add(modelFormField);
@@ -2812,7 +2811,7 @@ public class ModelForm extends ModelWidg
     }
 
     public void setDefaultEntityNameOnUpdateAreas() {
-        List<UpdateArea> allUpdateAreas = FastList.newInstance();
+        List<UpdateArea> allUpdateAreas = new LinkedList<UpdateArea>();
         if (UtilValidate.isNotEmpty(this.onSubmitUpdateAreas)) allUpdateAreas.addAll(this.onSubmitUpdateAreas);
         if (UtilValidate.isNotEmpty(this.onPaginateUpdateAreas)) allUpdateAreas.addAll(this.onPaginateUpdateAreas);
         for (UpdateArea updateArea : allUpdateAreas) {
@@ -2823,7 +2822,7 @@ public class ModelForm extends ModelWidg
     }
 
     public void setDefaultServiceNameOnUpdateAreas() {
-        List<UpdateArea> allUpdateAreas = FastList.newInstance();
+        List<UpdateArea> allUpdateAreas = new LinkedList<UpdateArea>();
         if (UtilValidate.isNotEmpty(this.onSubmitUpdateAreas)) allUpdateAreas.addAll(this.onSubmitUpdateAreas);
         if (UtilValidate.isNotEmpty(this.onPaginateUpdateAreas)) allUpdateAreas.addAll(this.onPaginateUpdateAreas);
         for (UpdateArea updateArea : allUpdateAreas) {
@@ -2901,7 +2900,7 @@ public class ModelForm extends ModelWidg
         protected String defaultEntityName;
         protected WidgetWorker.AutoEntityParameters autoEntityParameters;
         protected WidgetWorker.AutoEntityParameters autoServiceParameters;
-        List<WidgetWorker.Parameter> parameterList = FastList.newInstance();
+        protected List<WidgetWorker.Parameter> parameterList = new ArrayList<WidgetWorker.Parameter>();
         /** XML constructor.
          * @param updateAreaElement The <code>&lt;on-xxx-update-area&gt;</code>
          * XML element.
@@ -2950,7 +2949,7 @@ public class ModelForm extends ModelWidg
             return FlexibleStringExpander.expandString(areaTarget, context);
         }
         public Map<String, String> getParameterMap(Map<String, Object> context) {
-            Map<String, String> fullParameterMap = FastMap.newInstance();
+            Map<String, String> fullParameterMap = new HashMap<String, String>();
             if (autoServiceParameters != null) {
                 fullParameterMap.putAll(autoServiceParameters.getParametersMap(context, defaultServiceName));
             }
@@ -3175,7 +3174,7 @@ public class ModelForm extends ModelWidg
     }
 
     public Set<String> getAllEntityNamesUsed() {
-        Set<String> allEntityNamesUsed = FastSet.newInstance();
+        Set<String> allEntityNamesUsed = new HashSet<String>();
         for (AutoFieldsEntity autoFieldsEntity: this.autoFieldsEntities) {
             allEntityNamesUsed.add(autoFieldsEntity.entityName);
         }
@@ -3221,7 +3220,7 @@ public class ModelForm extends ModelWidg
     }
 
     public Set<String> getAllServiceNamesUsed() {
-        Set<String> allServiceNamesUsed = FastSet.newInstance();
+        Set<String> allServiceNamesUsed = new HashSet<String>();
         for (AutoFieldsService autoFieldsService: this.autoFieldsServices) {
             allServiceNamesUsed.add(autoFieldsService.serviceName);
         }
@@ -3254,7 +3253,7 @@ public class ModelForm extends ModelWidg
     }
 
     public Set<String> getLinkedRequestsLocationAndUri() throws GeneralException {
-        Set<String> allRequestsUsed = FastSet.newInstance();
+        Set<String> allRequestsUsed = new HashSet<String>();
 
         if (this.fieldList != null) {
             for (ModelFormField modelFormField: this.fieldList) {
@@ -3306,7 +3305,7 @@ public class ModelForm extends ModelWidg
     }
 
     public Set<String> getTargetedRequestsLocationAndUri() throws GeneralException {
-        Set<String> allRequestsUsed = FastSet.newInstance();
+        Set<String> allRequestsUsed = new HashSet<String>();
 
         if (this.altTargets != null) {
             for (AltTarget altTarget: this.altTargets) {

Modified: ofbiz/branches/json-integration-refactoring/framework/widget/src/org/ofbiz/widget/form/ModelFormAction.java
URL: http://svn.apache.org/viewvc/ofbiz/branches/json-integration-refactoring/framework/widget/src/org/ofbiz/widget/form/ModelFormAction.java?rev=1635897&r1=1635896&r2=1635897&view=diff
==============================================================================
--- ofbiz/branches/json-integration-refactoring/framework/widget/src/org/ofbiz/widget/form/ModelFormAction.java (original)
+++ ofbiz/branches/json-integration-refactoring/framework/widget/src/org/ofbiz/widget/form/ModelFormAction.java Sat Nov  1 07:15:09 2014
@@ -28,9 +28,6 @@ import java.util.Map;
 import java.util.TimeZone;
 import java.util.regex.PatternSyntaxException;
 
-import javolution.util.FastList;
-import javolution.util.FastMap;
-
 import org.ofbiz.base.util.Debug;
 import org.ofbiz.base.util.GeneralException;
 import org.ofbiz.base.util.ObjectType;
@@ -131,6 +128,7 @@ public abstract class ModelFormAction {
             }
         }
 
+        @SuppressWarnings("rawtypes")
         @Override
         public void runAction(Map<String, Object> context) {
             String globalStr = this.globalExdr.expandString(context);
@@ -152,9 +150,9 @@ public abstract class ModelFormAction {
 
             if (UtilValidate.isNotEmpty(this.type)) {
                 if ("NewMap".equals(this.type)) {
-                    newValue = FastMap.newInstance();
+                    newValue = new HashMap();
                 } else if ("NewList".equals(this.type)) {
-                    newValue = FastList.newInstance();
+                    newValue = new LinkedList();
                 } else {
                     try {
                         newValue = ObjectType.simpleTypeConvert(newValue, this.type, null, (TimeZone) context.get("timeZone"), (Locale) context.get("locale"), true);
@@ -286,7 +284,7 @@ public abstract class ModelFormAction {
         @Override
         public void runAction(Map<String, Object> context) {
             if (location.endsWith(".xml")) {
-                Map<String, Object> localContext = FastMap.newInstance();
+                Map<String, Object> localContext = new HashMap<String, Object>();
                 localContext.putAll(context);
                 DispatchContext ctx = this.modelForm.dispatchContext;
                 MethodContext methodContext = new MethodContext(ctx, localContext, null);

Modified: ofbiz/branches/json-integration-refactoring/framework/widget/src/org/ofbiz/widget/form/ModelFormField.java
URL: http://svn.apache.org/viewvc/ofbiz/branches/json-integration-refactoring/framework/widget/src/org/ofbiz/widget/form/ModelFormField.java?rev=1635897&r1=1635896&r2=1635897&view=diff
==============================================================================
--- ofbiz/branches/json-integration-refactoring/framework/widget/src/org/ofbiz/widget/form/ModelFormField.java (original)
+++ ofbiz/branches/json-integration-refactoring/framework/widget/src/org/ofbiz/widget/form/ModelFormField.java Sat Nov  1 07:15:09 2014
@@ -23,6 +23,7 @@ import java.math.BigDecimal;
 import java.sql.Timestamp;
 import java.text.DateFormat;
 import java.text.NumberFormat;
+import java.util.ArrayList;
 import java.util.Date;
 import java.util.HashMap;
 import java.util.LinkedList;
@@ -32,9 +33,6 @@ import java.util.Map;
 import java.util.StringTokenizer;
 import java.util.TimeZone;
 
-import javolution.util.FastList;
-import javolution.util.FastMap;
-
 import org.ofbiz.base.conversion.ConversionException;
 import org.ofbiz.base.conversion.DateTimeConverters;
 import org.ofbiz.base.conversion.DateTimeConverters.StringToTimestamp;
@@ -218,13 +216,13 @@ public class ModelFormField {
     }
 
     protected void addOnChangeUpdateArea(UpdateArea updateArea) {
-        if (onChangeUpdateAreas == null) onChangeUpdateAreas = FastList.newInstance();
+        if (onChangeUpdateAreas == null) onChangeUpdateAreas = new ArrayList<UpdateArea>();
         onChangeUpdateAreas.add(updateArea);
         Debug.logInfo(this.modelForm.getName() + ":" + this.name + " onChangeUpdateAreas size = " + onChangeUpdateAreas.size(), module);
     }
 
     protected void addOnClickUpdateArea(UpdateArea updateArea) {
-        if (onClickUpdateAreas == null) onClickUpdateAreas = FastList.newInstance();
+        if (onClickUpdateAreas == null) onClickUpdateAreas = new ArrayList<UpdateArea>();
         onClickUpdateAreas.add(updateArea);
     }
 
@@ -1519,7 +1517,7 @@ public class ModelFormField {
             List<? extends Object> dataList = UtilGenerics.checkList(this.listAcsr.get(context));
             if (dataList != null && dataList.size() != 0) {
                 for (Object data: dataList) {
-                    Map<String, Object> localContext = FastMap.newInstance();
+                    Map<String, Object> localContext = new HashMap<String, Object>();
                     localContext.putAll(context);
                     if (UtilValidate.isNotEmpty(this.listEntryName)) {
                         localContext.put(this.listEntryName, data);
@@ -2225,7 +2223,7 @@ public class ModelFormField {
         protected FlexibleStringExpander imageTitle;
         protected FlexibleStringExpander targetWindowExdr;
         protected FlexibleMapAccessor<Map<String, String>> parametersMapAcsr;
-        protected List<WidgetWorker.Parameter> parameterList = FastList.newInstance();
+        protected List<WidgetWorker.Parameter> parameterList = new ArrayList<WidgetWorker.Parameter>();
         protected WidgetWorker.AutoServiceParameters autoServiceParameters;
         protected WidgetWorker.AutoEntityParameters autoEntityParameters;
 
@@ -2337,7 +2335,7 @@ public class ModelFormField {
         }
 
         public Map<String, String> getParameterMap(Map<String, Object> context) {
-            Map<String, String> fullParameterMap = FastMap.newInstance();
+            Map<String, String> fullParameterMap = new HashMap<String, String>();
             
             Map<String, String> addlParamMap = this.parametersMapAcsr.get(context);
             if (addlParamMap != null) {
@@ -2433,7 +2431,7 @@ public class ModelFormField {
         protected FlexibleStringExpander target;
         protected FlexibleStringExpander description;
         protected FlexibleStringExpander targetWindowExdr;
-        protected List<WidgetWorker.Parameter> parameterList = FastList.newInstance();
+        protected List<WidgetWorker.Parameter> parameterList = new ArrayList<WidgetWorker.Parameter>();
         protected boolean requestConfirmation = false;
         protected FlexibleStringExpander confirmationMsgExdr;
         protected ModelFormField modelFormField;
@@ -2500,7 +2498,7 @@ public class ModelFormField {
         }
 
         public Map<String, String> getParameterMap(Map<String, Object> context) {
-            Map<String, String> fullParameterMap = FastMap.newInstance();
+            Map<String, String> fullParameterMap = new HashMap<String, String>();
 
             /* leaving this here... may want to add it at some point like the hyperlink element:
             Map<String, String> addlParamMap = this.parametersMapAcsr.get(context);
@@ -3607,7 +3605,7 @@ public class ModelFormField {
         }
 
         public List<String> getTargetParameterList() {
-            List<String> paramList = FastList.newInstance();
+            List<String> paramList = new LinkedList<String>();
             if (UtilValidate.isNotEmpty(this.targetParameter)) {
                 StringTokenizer stk = new StringTokenizer(this.targetParameter, ", ");
                 while (stk.hasMoreTokens()) {

Modified: ofbiz/branches/json-integration-refactoring/framework/widget/src/org/ofbiz/widget/html/HtmlFormRenderer.java
URL: http://svn.apache.org/viewvc/ofbiz/branches/json-integration-refactoring/framework/widget/src/org/ofbiz/widget/html/HtmlFormRenderer.java?rev=1635897&r1=1635896&r2=1635897&view=diff
==============================================================================
--- ofbiz/branches/json-integration-refactoring/framework/widget/src/org/ofbiz/widget/html/HtmlFormRenderer.java (original)
+++ ofbiz/branches/json-integration-refactoring/framework/widget/src/org/ofbiz/widget/html/HtmlFormRenderer.java Sat Nov  1 07:15:09 2014
@@ -20,6 +20,7 @@ package org.ofbiz.widget.html;
 
 import java.io.IOException;
 import java.util.HashSet;
+import java.util.LinkedList;
 import java.util.List;
 import java.util.Locale;
 import java.util.Map;
@@ -29,8 +30,6 @@ import javax.servlet.ServletContext;
 import javax.servlet.http.HttpServletRequest;
 import javax.servlet.http.HttpServletResponse;
 
-import javolution.util.FastList;
-
 import org.ofbiz.base.util.Debug;
 import org.ofbiz.base.util.StringUtil;
 import org.ofbiz.base.util.StringUtil.SimpleEncoder;
@@ -958,7 +957,7 @@ public class HtmlFormRenderer extends Ht
             String backgroundSubmitRefreshTarget = submitField.getBackgroundSubmitRefreshTarget(context);
             if (UtilValidate.isNotEmpty(backgroundSubmitRefreshTarget)) {
                 if (updateAreas == null) {
-                    updateAreas = FastList.newInstance();
+                    updateAreas = new LinkedList<ModelForm.UpdateArea>();
                 }
                 updateAreas.add(new ModelForm.UpdateArea("submit", formId, backgroundSubmitRefreshTarget));
             }

Modified: ofbiz/branches/json-integration-refactoring/framework/widget/src/org/ofbiz/widget/html/HtmlScreenRenderer.java
URL: http://svn.apache.org/viewvc/ofbiz/branches/json-integration-refactoring/framework/widget/src/org/ofbiz/widget/html/HtmlScreenRenderer.java?rev=1635897&r1=1635896&r2=1635897&view=diff
==============================================================================
--- ofbiz/branches/json-integration-refactoring/framework/widget/src/org/ofbiz/widget/html/HtmlScreenRenderer.java (original)
+++ ofbiz/branches/json-integration-refactoring/framework/widget/src/org/ofbiz/widget/html/HtmlScreenRenderer.java Sat Nov  1 07:15:09 2014
@@ -29,8 +29,6 @@ import javax.servlet.ServletContext;
 import javax.servlet.http.HttpServletRequest;
 import javax.servlet.http.HttpServletResponse;
 
-import javolution.util.FastMap;
-
 import org.ofbiz.base.util.Debug;
 import org.ofbiz.base.util.GeneralException;
 import org.ofbiz.base.util.UtilFormatOut;
@@ -52,8 +50,8 @@ import org.ofbiz.widget.form.ModelForm;
 import org.ofbiz.widget.menu.MenuStringRenderer;
 import org.ofbiz.widget.menu.ModelMenu;
 import org.ofbiz.widget.screen.ModelScreenWidget;
-import org.ofbiz.widget.screen.ScreenStringRenderer;
 import org.ofbiz.widget.screen.ModelScreenWidget.ColumnContainer;
+import org.ofbiz.widget.screen.ScreenStringRenderer;
 
 /**
  * Widget Library - HTML Form Renderer implementation
@@ -667,7 +665,7 @@ public class HtmlScreenRenderer extends 
         Delegator delegator = (Delegator) context.get("delegator");
 
         // make a new map for content rendering; so our current map does not get clobbered
-        Map<String, Object> contentContext = FastMap.newInstance();
+        Map<String, Object> contentContext = new HashMap<String, Object>();
         contentContext.putAll(context);
         String dataResourceId = (String)contentContext.get("dataResourceId");
         if (Debug.verboseOn()) Debug.logVerbose("expandedContentId:" + expandedContentId, module);
@@ -800,7 +798,7 @@ public class HtmlScreenRenderer extends 
             Delegator delegator = (Delegator) context.get("delegator");
 
             // create a new map for the content rendering; so our current context does not get overwritten!
-            Map<String, Object> contentContext = FastMap.newInstance();
+            Map<String, Object> contentContext = new HashMap<String, Object>();
             contentContext.putAll(context);
 
             try {

Modified: ofbiz/branches/json-integration-refactoring/framework/widget/src/org/ofbiz/widget/menu/ModelMenuAction.java
URL: http://svn.apache.org/viewvc/ofbiz/branches/json-integration-refactoring/framework/widget/src/org/ofbiz/widget/menu/ModelMenuAction.java?rev=1635897&r1=1635896&r2=1635897&view=diff
==============================================================================
--- ofbiz/branches/json-integration-refactoring/framework/widget/src/org/ofbiz/widget/menu/ModelMenuAction.java (original)
+++ ofbiz/branches/json-integration-refactoring/framework/widget/src/org/ofbiz/widget/menu/ModelMenuAction.java Sat Nov  1 07:15:09 2014
@@ -20,6 +20,8 @@ package org.ofbiz.widget.menu;
 
 import java.text.MessageFormat;
 import java.util.ArrayList;
+import java.util.HashMap;
+import java.util.LinkedList;
 import java.util.List;
 import java.util.Locale;
 import java.util.Map;
@@ -28,9 +30,6 @@ import java.util.TimeZone;
 import javax.servlet.ServletContext;
 import javax.servlet.http.HttpSession;
 
-import javolution.util.FastList;
-import javolution.util.FastMap;
-
 import org.ofbiz.base.util.Debug;
 import org.ofbiz.base.util.GeneralException;
 import org.ofbiz.base.util.ObjectType;
@@ -140,6 +139,7 @@ public abstract class ModelMenuAction {
             }
         }
 
+        @SuppressWarnings("rawtypes")
         @Override
         public void runAction(Map<String, Object> context) {
             String globalStr = this.globalExdr.expandString(context);
@@ -189,9 +189,9 @@ public abstract class ModelMenuAction {
 
             if (UtilValidate.isNotEmpty(this.type)) {
                 if ("NewMap".equals(this.type)) {
-                    newValue = FastMap.newInstance();
+                    newValue = new HashMap();
                 } else if ("NewList".equals(this.type)) {
-                    newValue = FastList.newInstance();
+                    newValue = new LinkedList();
                 } else {
                     try {
                         newValue = ObjectType.simpleTypeConvert(newValue, this.type, null, (TimeZone) context.get("timeZone"), (Locale) context.get("locale"), true);
@@ -379,7 +379,7 @@ public abstract class ModelMenuAction {
 
             List<? extends Element> fieldMapElementList = UtilXml.childElementList(serviceElement, "field-map");
             if (fieldMapElementList.size() > 0) {
-                this.fieldMap = FastMap.newInstance();
+                this.fieldMap = new HashMap<FlexibleMapAccessor<Object>, FlexibleMapAccessor<Object>>();
                 for (Element fieldMapElement: fieldMapElementList) {
                     // set the env-name for each field-name, noting that if no field-name is specified it defaults to the env-name
                     this.fieldMap.put(
@@ -404,7 +404,7 @@ public abstract class ModelMenuAction {
                 if (autoFieldMapBool) {
                     serviceContext = WidgetWorker.getDispatcher(context).getDispatchContext().makeValidContext(serviceNameExpanded, ModelService.IN_PARAM, context);
                 } else {
-                    serviceContext = FastMap.newInstance();
+                    serviceContext = new HashMap<String, Object>();
                 }
 
                 if (this.fieldMap != null) {

Modified: ofbiz/branches/json-integration-refactoring/framework/widget/src/org/ofbiz/widget/menu/ModelMenuCondition.java
URL: http://svn.apache.org/viewvc/ofbiz/branches/json-integration-refactoring/framework/widget/src/org/ofbiz/widget/menu/ModelMenuCondition.java?rev=1635897&r1=1635896&r2=1635897&view=diff
==============================================================================
--- ofbiz/branches/json-integration-refactoring/framework/widget/src/org/ofbiz/widget/menu/ModelMenuCondition.java (original)
+++ ofbiz/branches/json-integration-refactoring/framework/widget/src/org/ofbiz/widget/menu/ModelMenuCondition.java Sat Nov  1 07:15:09 2014
@@ -19,13 +19,12 @@
 package org.ofbiz.widget.menu;
 
 import java.lang.reflect.Method;
+import java.util.LinkedList;
 import java.util.List;
 import java.util.Locale;
 import java.util.Map;
 import java.util.TimeZone;
 
-import javolution.util.FastList;
-
 import org.apache.oro.text.regex.MalformedPatternException;
 import org.apache.oro.text.regex.Pattern;
 import org.apache.oro.text.regex.PatternMatcher;
@@ -101,7 +100,7 @@ public class ModelMenuCondition {
     }
 
     public static List<MenuCondition> readSubConditions(ModelMenuItem modelMenuItem, Element conditionElement) {
-        List<MenuCondition> condList = FastList.newInstance();
+        List<MenuCondition> condList = new LinkedList<MenuCondition>();
         List<? extends Element> subElementList = UtilXml.childElementList(conditionElement);
         for (Element subElement: subElementList) {
             condList.add(readCondition(modelMenuItem, subElement));
@@ -422,7 +421,7 @@ public class ModelMenuCondition {
                 fieldVal = "";
             }
 
-            List<Object> messages = FastList.newInstance();
+            List<Object> messages = new LinkedList<Object>();
             Boolean resultBool = BaseCompare.doRealCompare(fieldVal, value, operator, type, format, messages, null, null, true);
             if (messages.size() > 0) {
                 messages.add(0, "Error with comparison in if-compare between field [" + fieldAcsr.toString() + "] with value [" + fieldVal + "] and value [" + value + "] with operator [" + operator + "] and type [" + type + "]: ");
@@ -473,7 +472,7 @@ public class ModelMenuCondition {
                 fieldVal = "";
             }
 
-            List<Object> messages = FastList.newInstance();
+            List<Object> messages = new LinkedList<Object>();
             Boolean resultBool = BaseCompare.doRealCompare(fieldVal, toFieldVal, operator, type, format, messages, null, null, false);
             if (messages.size() > 0) {
                 messages.add(0, "Error with comparison in if-compare-field between field [" + fieldAcsr.toString() + "] with value [" + fieldVal + "] and to-field [" + toFieldVal.toString() + "] with value [" + toFieldVal + "] with operator [" + operator + "] and type [" + type + "]: ");

Modified: ofbiz/branches/json-integration-refactoring/framework/widget/src/org/ofbiz/widget/menu/ModelMenuItem.java
URL: http://svn.apache.org/viewvc/ofbiz/branches/json-integration-refactoring/framework/widget/src/org/ofbiz/widget/menu/ModelMenuItem.java?rev=1635897&r1=1635896&r2=1635897&view=diff
==============================================================================
--- ofbiz/branches/json-integration-refactoring/framework/widget/src/org/ofbiz/widget/menu/ModelMenuItem.java (original)
+++ ofbiz/branches/json-integration-refactoring/framework/widget/src/org/ofbiz/widget/menu/ModelMenuItem.java Sat Nov  1 07:15:09 2014
@@ -19,6 +19,7 @@
 package org.ofbiz.widget.menu;
 
 import java.io.IOException;
+import java.util.ArrayList;
 import java.util.HashMap;
 import java.util.LinkedList;
 import java.util.List;
@@ -27,9 +28,6 @@ import java.util.Map;
 
 import javax.xml.parsers.ParserConfigurationException;
 
-import javolution.util.FastList;
-import javolution.util.FastMap;
-
 import org.ofbiz.base.util.Debug;
 import org.ofbiz.base.util.StringUtil;
 import org.ofbiz.base.util.UtilFormatOut;
@@ -39,8 +37,8 @@ import org.ofbiz.base.util.UtilXml;
 import org.ofbiz.base.util.collections.FlexibleMapAccessor;
 import org.ofbiz.base.util.string.FlexibleStringExpander;
 import org.ofbiz.entity.GenericValue;
-import org.ofbiz.widget.WidgetWorker;
 import org.ofbiz.widget.PortalPageWorker;
+import org.ofbiz.widget.WidgetWorker;
 import org.w3c.dom.Element;
 import org.xml.sax.SAXException;
 
@@ -569,7 +567,7 @@ public class ModelMenuItem {
         protected WidgetWorker.AutoServiceParameters autoServiceParameters;
         protected WidgetWorker.AutoEntityParameters autoEntityParameters;
         protected FlexibleMapAccessor<Map<String, String>> parametersMapAcsr;
-        protected List<WidgetWorker.Parameter> parameterList = FastList.newInstance();
+        protected List<WidgetWorker.Parameter> parameterList = new ArrayList<WidgetWorker.Parameter>();
         protected boolean requestConfirmation = false;
         protected FlexibleStringExpander confirmationMsgExdr;
 
@@ -702,7 +700,7 @@ public class ModelMenuItem {
             return this.parameterList;
         }
         public Map<String, String> getParameterMap(Map<String, Object> context) {
-            Map<String, String> fullParameterMap = FastMap.newInstance();
+            Map<String, String> fullParameterMap = new HashMap<String, String>();
 
             if (this.parametersMapAcsr != null) {
                 Map<String, String> addlParamMap = this.parametersMapAcsr.get(context);

Modified: ofbiz/branches/json-integration-refactoring/framework/widget/src/org/ofbiz/widget/screen/HtmlWidget.java
URL: http://svn.apache.org/viewvc/ofbiz/branches/json-integration-refactoring/framework/widget/src/org/ofbiz/widget/screen/HtmlWidget.java?rev=1635897&r1=1635896&r2=1635897&view=diff
==============================================================================
--- ofbiz/branches/json-integration-refactoring/framework/widget/src/org/ofbiz/widget/screen/HtmlWidget.java (original)
+++ ofbiz/branches/json-integration-refactoring/framework/widget/src/org/ofbiz/widget/screen/HtmlWidget.java Sat Nov  1 07:15:09 2014
@@ -22,11 +22,10 @@ import java.io.IOException;
 import java.net.MalformedURLException;
 import java.util.ArrayList;
 import java.util.Collection;
+import java.util.HashMap;
 import java.util.List;
 import java.util.Map;
 
-import javolution.util.FastMap;
-
 import org.ofbiz.base.util.Debug;
 import org.ofbiz.base.util.GeneralException;
 import org.ofbiz.base.util.StringUtil;
@@ -220,7 +219,7 @@ public class HtmlWidget extends ModelScr
 
     public static class HtmlTemplateDecorator extends ModelScreenWidget {
         protected FlexibleStringExpander locationExdr;
-        protected Map<String, HtmlTemplateDecoratorSection> sectionMap = FastMap.newInstance();
+        protected Map<String, HtmlTemplateDecoratorSection> sectionMap = new HashMap<String, HtmlTemplateDecoratorSection>();
 
         public HtmlTemplateDecorator(ModelScreen modelScreen, Element htmlTemplateDecoratorElement) {
             super(modelScreen, htmlTemplateDecoratorElement);