You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@avalon.apache.org by ha...@apache.org on 2001/11/14 03:47:22 UTC
cvs commit: jakarta-avalon-cornerstone/apps/db/src/java/org/apache/avalon/db/basic/actions BasicCreateQueryable.java BasicInsert.java
hammant 01/11/13 18:47:22
Modified: apps/db/src/java/org/apache/avalon/db/basic/actions
BasicCreateQueryable.java BasicInsert.java
Log:
updated type handling
Revision Changes Path
1.6 +4 -6 jakarta-avalon-cornerstone/apps/db/src/java/org/apache/avalon/db/basic/actions/BasicCreateQueryable.java
Index: BasicCreateQueryable.java
===================================================================
RCS file: /home/cvs/jakarta-avalon-cornerstone/apps/db/src/java/org/apache/avalon/db/basic/actions/BasicCreateQueryable.java,v
retrieving revision 1.5
retrieving revision 1.6
diff -u -r1.5 -r1.6
--- BasicCreateQueryable.java 2001/11/14 02:40:29 1.5
+++ BasicCreateQueryable.java 2001/11/14 02:47:22 1.6
@@ -37,13 +37,13 @@
*
*
* @author Paul Hammant <a href="mailto:Paul_Hammant@yahoo.com">Paul_Hammant@yahoo.com</a>
- * @version $Revision: 1.5 $
+ * @version $Revision: 1.6 $
*/
public abstract class BasicCreateQueryable extends AbstractAction implements CreateTable {
/**
- * Method execute
+ * Method getColumns
*
*
*/
@@ -60,13 +60,11 @@
if (subColumnsNodes.item(x).getNodeName().equals("column")) {
columns.add(createColumn(subColumnsNodes.item(x)));
} else {
- // TODO There should be no other element types inside <columns>
+ throw new ActionException("Only 'column' allowed as a node under 'columns' node");
}
-
}
-
} else {
- //TODO Only <columns> is supported in <create-table> so far
+ throw new ActionException("Only 'columns' allowed as a node under 'create-table' node");
}
}
1.7 +2 -1 jakarta-avalon-cornerstone/apps/db/src/java/org/apache/avalon/db/basic/actions/BasicInsert.java
Index: BasicInsert.java
===================================================================
RCS file: /home/cvs/jakarta-avalon-cornerstone/apps/db/src/java/org/apache/avalon/db/basic/actions/BasicInsert.java,v
retrieving revision 1.6
retrieving revision 1.7
diff -u -r1.6 -r1.7
--- BasicInsert.java 2001/11/13 16:04:34 1.6
+++ BasicInsert.java 2001/11/14 02:47:22 1.7
@@ -99,8 +99,9 @@
String colName = rowNode.getAttribute("col");
String valueStr = rowNode.getFirstChild().getNodeValue();
int ix = mTable.getColumnIndex(colName);
- // TODO perhaps coluns could know their own index?
+ // TODO perhaps columns could know their own index?
Column col = mTable.getColumn(colName);
+ col.test(valueStr);
values[ix] = col.convertFromString(valueStr);
}
}
--
To unsubscribe, e-mail: <ma...@jakarta.apache.org>
For additional commands, e-mail: <ma...@jakarta.apache.org>