Changeset 4521

Timestamp:

2003-06-09T13:03:43+12:00 (21 years ago)

Author:

mdewsnip

Message:

Changed away from using SynchronizedTreeModelTools, for the moment.

File:

• trunk/gli/src/org/greenstone/gatherer/valuetree/GValueModel.java (modified) (1 diff)

trunk/gli/src/org/greenstone/gatherer/valuetree/GValueModel.java

@@ -111 +111 @@
         } 
         value_node = new GValueNode(new_subject);
+        System.err.println("(GValueModel) addValue()... " + value_node);
+        
+        // Figure out where this node will be inserted in subjects 
+        // children.
+        int position = -1;
+        for(int i = 0; position == -1 && i < subject.getChildCount(); i++) {
+        Object sibling = subject.getChildAt(i);
+        int rel_pos = value.compareTo(sibling.toString());
+        ///ystem.err.println("'"+value+"'.compareTo('"+sibling+"') = " + rel_pos);
+        if(rel_pos <= 0) {
+            position = i;
+        }
+        }
+        if(position == -1) {
+        position = subject.getChildCount();
+        }
         // Insert it. If position is still -1, append it to the end of subjects children.
-        //SynchronizedTreeModelTools.insertNodeInto(this, subject, value_node);
-        insertNodeInto(value_node, subject, 0);
+        ///ystem.err.println("Inserting '" + value + "' at position " + position);
+        insertNodeInto(value_node, subject, position);
+        // System.err.println("(GValueModel) Done insert node into...");
+        // SynchronizedTreeModelTools.insertNodeInto(this, subject, value_node);
         // Inform listeners that we've changed.
         Gatherer.c_man.getCollection().msm.fireValueChanged(element, null, this);