Author: tv
Date: Sun Jan 31 20:28:09 2016
New Revision: 1727853

URL: http://svn.apache.org/viewvc?rev=1727853&view=rev
Log:
Modify several signatures to comply with the types used in Torque 4

Modified:
    
db/torque/torque4/trunk/torque-runtime/src/main/java/org/apache/torque/manager/AbstractBaseManager.java
    
db/torque/torque4/trunk/torque-runtime/src/main/java/org/apache/torque/manager/CacheListener.java
    
db/torque/torque4/trunk/torque-runtime/src/test/java/org/apache/torque/manager/AbstractBaseManagerTest.java

Modified: 
db/torque/torque4/trunk/torque-runtime/src/main/java/org/apache/torque/manager/AbstractBaseManager.java
URL: 
http://svn.apache.org/viewvc/db/torque/torque4/trunk/torque-runtime/src/main/java/org/apache/torque/manager/AbstractBaseManager.java?rev=1727853&r1=1727852&r2=1727853&view=diff
==============================================================================
--- 
db/torque/torque4/trunk/torque-runtime/src/main/java/org/apache/torque/manager/AbstractBaseManager.java
 (original)
+++ 
db/torque/torque4/trunk/torque-runtime/src/main/java/org/apache/torque/manager/AbstractBaseManager.java
 Sun Jan 31 20:28:09 2016
@@ -37,6 +37,7 @@ import org.apache.commons.jcs.access.Gro
 import org.apache.commons.jcs.access.exception.CacheException;
 import org.apache.commons.logging.Log;
 import org.apache.commons.logging.LogFactory;
+import org.apache.torque.Column;
 import org.apache.torque.Torque;
 import org.apache.torque.TorqueException;
 import org.apache.torque.om.ObjectKey;
@@ -60,7 +61,7 @@ public abstract class AbstractBaseManage
     protected static final Log log = 
LogFactory.getLog(AbstractBaseManager.class);
 
     /** used to cache the om objects. cache is set by the region property */
-    protected transient CacheAccess<Serializable, T> cache;
+    protected transient CacheAccess<ObjectKey, T> cache;
 
     /** used to cache the method result objects. cache is set by the region 
property */
     protected transient GroupCacheAccess<MethodCacheKey, Object> groupCache;
@@ -81,11 +82,11 @@ public abstract class AbstractBaseManage
     private boolean isNew = true;
 
     /** The fields which are valid fields of interest for a listener. */
-    protected Map<String, ?> validFields;
+    protected Map<Column, ?> validFields;
 
     /** The listeners for this manager. */
-    protected Map<String, FastArrayList> listenersMap
-        = new HashMap<String, FastArrayList>();
+    protected Map<Column, FastArrayList> listenersMap
+        = new HashMap<Column, FastArrayList>();
 
     /**
      * Get the Class instance
@@ -224,7 +225,7 @@ public abstract class AbstractBaseManage
      * @param key the primary key of the object
      * @return the object from cache
      */
-    protected T cacheGet(final Serializable key)
+    protected T cacheGet(final ObjectKey key)
     {
         T om = null;
         if (cache != null)
@@ -285,7 +286,7 @@ public abstract class AbstractBaseManage
      * @throws TorqueException Any exceptions caught during processing will be
      *         rethrown wrapped into a TorqueException.
      */
-    protected T removeInstanceImpl(final Serializable key)
+    protected T removeInstanceImpl(final ObjectKey key)
         throws TorqueException
     {
         T oldOm = null;
@@ -323,7 +324,7 @@ public abstract class AbstractBaseManage
      * @throws TorqueException Any exceptions caught during processing will be
      *         rethrown wrapped into a TorqueException.
      */
-    protected T putInstanceImpl(final Serializable key, final T om)
+    protected T putInstanceImpl(final ObjectKey key, final T om)
         throws TorqueException
     {
         if (getOMClass() != null && !getOMClass().isInstance(om))
@@ -535,8 +536,8 @@ public abstract class AbstractBaseManage
     public void addCacheListenerImpl(
             final CacheListener<? extends Persistent> listener)
     {
-        List<String> keys = listener.getInterestedFields();
-        for (String key : keys)
+        List<Column> keys = listener.getInterestedFields();
+        for (Column key : keys)
         {
             // Peer.column names are the fields
             if (validFields != null && validFields.containsKey(key))
@@ -580,7 +581,7 @@ public abstract class AbstractBaseManage
      * @param key
      * @return A subset of the list identified by <code>key</code>.
      */
-    private synchronized FastArrayList createSubsetList(final String key)
+    private synchronized FastArrayList createSubsetList(final Column key)
     {
         FastArrayList list = null;
         if (listenersMap.containsKey(key))

Modified: 
db/torque/torque4/trunk/torque-runtime/src/main/java/org/apache/torque/manager/CacheListener.java
URL: 
http://svn.apache.org/viewvc/db/torque/torque4/trunk/torque-runtime/src/main/java/org/apache/torque/manager/CacheListener.java?rev=1727853&r1=1727852&r2=1727853&view=diff
==============================================================================
--- 
db/torque/torque4/trunk/torque-runtime/src/main/java/org/apache/torque/manager/CacheListener.java
 (original)
+++ 
db/torque/torque4/trunk/torque-runtime/src/main/java/org/apache/torque/manager/CacheListener.java
 Sun Jan 31 20:28:09 2016
@@ -21,6 +21,7 @@ package org.apache.torque.manager;
 
 import java.util.List;
 
+import org.apache.torque.Column;
 import org.apache.torque.om.Persistent;
 
 /**
@@ -49,5 +50,5 @@ public interface CacheListener<T extends
     /**
      *
      */
-    List<String> getInterestedFields();
+    List<Column> getInterestedFields();
 }

Modified: 
db/torque/torque4/trunk/torque-runtime/src/test/java/org/apache/torque/manager/AbstractBaseManagerTest.java
URL: 
http://svn.apache.org/viewvc/db/torque/torque4/trunk/torque-runtime/src/test/java/org/apache/torque/manager/AbstractBaseManagerTest.java?rev=1727853&r1=1727852&r2=1727853&view=diff
==============================================================================
--- 
db/torque/torque4/trunk/torque-runtime/src/test/java/org/apache/torque/manager/AbstractBaseManagerTest.java
 (original)
+++ 
db/torque/torque4/trunk/torque-runtime/src/test/java/org/apache/torque/manager/AbstractBaseManagerTest.java
 Sun Jan 31 20:28:09 2016
@@ -19,13 +19,15 @@ package org.apache.torque.manager;
  * under the License.
  */
 
-import java.util.Arrays;
+import java.util.ArrayList;
 import java.util.HashMap;
 import java.util.List;
 
 import junit.framework.TestCase;
 
 import org.apache.torque.BaseTestCase;
+import org.apache.torque.Column;
+import org.apache.torque.ColumnImpl;
 import org.apache.torque.Torque;
 import org.apache.torque.TorqueException;
 import org.apache.torque.om.ObjectKey;
@@ -55,19 +57,25 @@ public class AbstractBaseManagerTest ext
        }
 
        /* CacheListener methods */
-       public void addedObject(TestPersistent om)
+       @Override
+    public void addedObject(TestPersistent om)
        {
                this.addedObjectCalled = true;
        }
 
-       public void refreshedObject(TestPersistent om)
+       @Override
+    public void refreshedObject(TestPersistent om)
        {
                this.refreshedObjectCalled = true;
        }
 
-       public List<String> getInterestedFields()
+       @Override
+    public List<Column> getInterestedFields()
        {
-               return Arrays.asList("test1", "test2");
+           List<Column> columnList = new ArrayList<Column>(2);
+               columnList.add(new ColumnImpl("test1"));
+               columnList.add(new ColumnImpl("test2"));
+               return columnList;
        }
 
        /**
@@ -167,20 +175,20 @@ public class AbstractBaseManagerTest ext
        @SuppressWarnings("unchecked")
        public void testListeners()
        {
-               manager.validFields = new HashMap<String, Object>();
-               manager.validFields.put("test1", null);
-               manager.validFields.put("test2", null);
+               manager.validFields = new HashMap<Column, Object>();
+               manager.validFields.put(new ColumnImpl("test1"), null);
+               manager.validFields.put(new ColumnImpl("test2"), null);
                manager.addCacheListenerImpl(this);
                this.addedObjectCalled = false;
                this.refreshedObjectCalled = false;
                TestPersistent test1 = new TestPersistent();
                TestPersistent test2 = new TestPersistent();
-               manager.notifyListeners(manager.listenersMap.get("test1"), 
(TestPersistent)null, test1);
+               manager.notifyListeners(manager.listenersMap.get(new 
ColumnImpl("test1")), (TestPersistent)null, test1);
                assertTrue("Should call addedObject", addedObjectCalled);
                assertFalse("Should not call refreshedObject", 
refreshedObjectCalled);
                this.addedObjectCalled = false;
                this.refreshedObjectCalled = false;
-               manager.notifyListeners(manager.listenersMap.get("test2"), 
test2, test1);
+               manager.notifyListeners(manager.listenersMap.get(new 
ColumnImpl("test2")), test2, test1);
                assertFalse("Should not call addedObject", addedObjectCalled);
                assertTrue("Should call refreshedObject", 
refreshedObjectCalled);
        }



---------------------------------------------------------------------
To unsubscribe, e-mail: torque-dev-unsubscr...@db.apache.org
For additional commands, e-mail: torque-dev-h...@db.apache.org

Reply via email to