You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@commons.apache.org by dg...@apache.org on 2003/08/28 02:11:23 UTC
cvs commit: jakarta-commons-sandbox/mapper/src/share/org/apache/commons/mapper/jdbc JdbcMapper.java
dgraham 2003/08/27 17:11:23
Modified: mapper/src/test/org/apache/commons/mapper TestMapper.java
mapper/src/share/org/apache/commons/mapper Mapper.java
mapper/src/share/org/apache/commons/mapper/jdbc
JdbcMapper.java
Log:
Changed Mapper.findAllObjects() to return a Collection instead of a List
because the objects aren't necessarily ordered.
Revision Changes Path
1.3 +2 -2 jakarta-commons-sandbox/mapper/src/test/org/apache/commons/mapper/TestMapper.java
Index: TestMapper.java
===================================================================
RCS file: /home/cvs/jakarta-commons-sandbox/mapper/src/test/org/apache/commons/mapper/TestMapper.java,v
retrieving revision 1.2
retrieving revision 1.3
diff -u -r1.2 -r1.3
--- TestMapper.java 21 Jun 2003 02:41:43 -0000 1.2
+++ TestMapper.java 28 Aug 2003 00:11:23 -0000 1.3
@@ -61,7 +61,7 @@
package org.apache.commons.mapper;
-import java.util.List;
+import java.util.Collection;
/**
* A test implementation of the Mapper interface.
@@ -90,7 +90,7 @@
}
- public List findAllObjects() throws MapperException {
+ public Collection findAllObjects() throws MapperException {
return null;
}
1.4 +6 -6 jakarta-commons-sandbox/mapper/src/share/org/apache/commons/mapper/Mapper.java
Index: Mapper.java
===================================================================
RCS file: /home/cvs/jakarta-commons-sandbox/mapper/src/share/org/apache/commons/mapper/Mapper.java,v
retrieving revision 1.3
retrieving revision 1.4
diff -u -r1.3 -r1.4
--- Mapper.java 10 Jun 2003 03:34:41 -0000 1.3
+++ Mapper.java 28 Aug 2003 00:11:23 -0000 1.4
@@ -61,7 +61,7 @@
package org.apache.commons.mapper;
-import java.util.List;
+import java.util.Collection;
/**
* A Mapper is responsible for mapping domain objects to a persistent data store.
@@ -128,11 +128,11 @@
void update(Object object) throws MapperException;
/**
- * Returns a List of all objects for the given mapper.
+ * Returns a Collection of all objects for the given mapper.
* @throws MapperException
* @throws UnsupportedOperationException if this mapper doesn't support this
* method.
*/
- List findAllObjects() throws MapperException;
+ Collection findAllObjects() throws MapperException;
}
1.5 +5 -5 jakarta-commons-sandbox/mapper/src/share/org/apache/commons/mapper/jdbc/JdbcMapper.java
Index: JdbcMapper.java
===================================================================
RCS file: /home/cvs/jakarta-commons-sandbox/mapper/src/share/org/apache/commons/mapper/jdbc/JdbcMapper.java,v
retrieving revision 1.4
retrieving revision 1.5
diff -u -r1.4 -r1.5
--- JdbcMapper.java 21 Jun 2003 01:53:11 -0000 1.4
+++ JdbcMapper.java 28 Aug 2003 00:11:23 -0000 1.5
@@ -61,7 +61,7 @@
package org.apache.commons.mapper.jdbc;
-import java.util.List;
+import java.util.Collection;
import java.util.Map;
import javax.sql.DataSource;
@@ -147,7 +147,7 @@
* @throws UnsupportedOperationException
* @see org.apache.commons.mapper.Mapper#findAllObjects()
*/
- public List findAllObjects() throws MapperException {
+ public Collection findAllObjects() throws MapperException {
throw new UnsupportedOperationException("Find all objects not supported.");
}