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

do contains() check for group-by nodes, constraint check for database nodes

Browse source 
in getNode().
This commit is contained in:
hns 2001-08-22 00:12:20 +00:00
parent af41c07c8b
commit 50fa6c0e73
1 changed files with 7 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

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

@ -235,12 +235,17 @@ public final class NodeManager {
// check if we can use the cached node without further checks. // check if we can use the cached node without further checks.
// we need further checks for subnodes fetched by name if the subnodes were changed. // we need further checks for subnodes fetched by name if the subnodes were changed.
if (!rel.virtual && !rel.usesPrimaryKey () && node != null && node.getState() != Node.INVALID) { if (node != null && node.getState() != Node.INVALID && !rel.virtual && !rel.usesPrimaryKey ()) {
// check if node is null node (cached null) // check if node is null node (cached null)
if (node.isNullNode ()) { if (node.isNullNode ()) {
if (node.created() < rel.otherType.getLastDataChange ()) if (node.created() < rel.otherType.getLastDataChange ())
node = null; // cached null not valid anymore node = null; // cached null not valid anymore
// } else if (false && app.doesSubnodeChecking () && home.contains (node) < 0) { // apply different consistency checks for groupby nodes and database nodes:
// for group nodes, check if they're contained
} else if (rel.groupby != null) {
if (home.contains (node) < 0)
node = null;
// for database nodes, check if constraints are fulfilled
} else if (!rel.checkConstraints (home, node)) { } else if (!rel.checkConstraints (home, node)) {
node = null; node = null;
} }