Author: michiel
Date: 2010-03-23 18:53:54 +0100 (Tue, 23 Mar 2010)
New Revision: 41578

Modified:
   
mmbase/branches/MMBase-1_9/core/src/main/java/org/mmbase/storage/search/implementation/BasicStep.java
Log:
Removed the restricotn on nodenumber should be >0. It's annoying and made me 
make horrible hacks, which I'm goging to remove soon

Modified: 
mmbase/branches/MMBase-1_9/core/src/main/java/org/mmbase/storage/search/implementation/BasicStep.java
===================================================================
--- 
mmbase/branches/MMBase-1_9/core/src/main/java/org/mmbase/storage/search/implementation/BasicStep.java
       2010-03-23 17:52:25 UTC (rev 41577)
+++ 
mmbase/branches/MMBase-1_9/core/src/main/java/org/mmbase/storage/search/implementation/BasicStep.java
       2010-03-23 17:53:54 UTC (rev 41578)
@@ -76,10 +76,9 @@
      */
     public Step addNode(int nodeNumber) {
         if (! modifiable) throw new IllegalStateException();
-        if (nodeNumber < 0) {
-            throw new IllegalArgumentException("Invalid nodeNumber value: " + 
nodeNumber);
+        if (nodes == null) {
+            nodes =  new TreeSet<Integer>();
         }
-        if (nodes == null) nodes =  new TreeSet<Integer>();
         nodes.add(nodeNumber);
         return this;
     }
@@ -105,7 +104,7 @@
 
     // javadoc is inherited
     public SortedSet<Integer> getNodes() {
-        return nodes == null ? null : Collections.unmodifiableSortedSet(nodes);
+        return nodes == null ? null : (modifiable ? nodes : 
Collections.unmodifiableSortedSet(nodes));
     }
 
     // javadoc is inherited

_______________________________________________
Cvs mailing list
Cvs@lists.mmbase.org
http://lists.mmbase.org/mailman/listinfo/cvs

Reply via email to