antville/helma
antville/helma
1
1
Fork 0
Code Issues 5 Pull requests 5 Projects Releases 5 Activity Actions

Generate multiple valid IDs in generateMaxID() when the

Browse source 
prototype's table is empty. Previously, new IDs for empty
tables always returned 0.

This fixes bug 59.

http://helma.org/bugs/show_bug.cgi?id=59
This commit is contained in:
hns 2002-02-01 15:20:22 +00:00
parent 896e5e1854
commit ced9315833
1 changed files with 12 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

18
src/helma/objectmodel/db/NodeManager.java
View file

@ -579,11 +579,14 @@ public final class NodeManager {
qds = new QueryDataSet (con, q);
qds.fetchRecords ();
// check for empty table
if (qds.size () == 0)
return "0";
long currMax = qds.getRecord (0).getValue (1).asLong ();
currMax = map.getNewID (currMax);
retval = Long.toString (currMax);
if (qds.size () == 0) {
long currMax = map.getNewID (0);
retval = Long.toString (currMax);
} else {
long currMax = qds.getRecord (0).getValue (1).asLong ();
currMax = map.getNewID (currMax);
retval = Long.toString (currMax);
}
} finally {
// tx.timer.endEvent ("generateID "+map);
if (qds != null) {
@ -594,6 +597,9 @@ public final class NodeManager {
}
/**
* Generate a new ID from an Oracle sequence.
*/
public String generateID (DbMapping map) throws Exception {
Transactor tx = (Transactor) Thread.currentThread ();
@ -606,7 +612,7 @@ public final class NodeManager {
String q = "SELECT "+map.getIDgen()+".nextval FROM dual";
qds = new QueryDataSet (con, q);
qds.fetchRecords ();
retval = qds.getRecord (0).getValue (1).asString ();
retval = qds.getRecord (0).getValue (1).asString ();
} finally {
// tx.timer.endEvent ("generateID "+map);
if (qds != null) {