Changeset 24135

Timestamp:

2011-06-09T15:42:41+12:00 (13 years ago)

Author:

sjb48

Message:

Saving of the format statement to the collectionConfig is now possible. The bug with select boxes where the selected item was not saved has now been fixed.

File:

• main/trunk/greenstone3/src/java/org/greenstone/gsdl3/collection/Collection.java (modified) (7 diffs)

main/trunk/greenstone3/src/java/org/greenstone/gsdl3/collection/Collection.java

@@ -33 +33 @@
 import java.io.File;
 import java.util.HashMap;
+import java.util.*;
 
 import javax.xml.parsers.DocumentBuilder;
@@ -471 +472 @@
                 int index;
                 Element elem;
+                // Try importing entire tree to this.doc so we can add and remove children at ease
+                //Node current_node = this.doc.importNode(GSXML.getChildByTagName(config, "CollectionConfig"),true);
                 Node current_node = GSXML.getChildByTagName(config, "CollectionConfig");
                 NodeList current_node_list;
-    
+   
+                logger.error("Service is "+service);
+
                 if(service.equals("ClassifierBrowse"))
                 {
@@ -480 +485 @@
                     // default is <browse><format>
 
+                    logger.error("Looking for browse");
                     current_node = GSXML.getChildByTagName(current_node, "browse");
 
@@ -485 +491 @@
                     if(classifier != null)
                     {
+                        logger.error("Classifier is not null");
+                        logger.error("Classifier is "+classifier);
                         current_node_list = GSXML.getChildrenByTagName(current_node, "classifier");
                         index = Integer.parseInt(classifier.substring(2)) - 1;
+                        logger.error("classifier index is "+index);
                         // index should be given by X-1
                         current_node = current_node_list.item(index);
-                        current_node = GSXML.getChildByTagName(current_node, "format");
+                        // what if classifier does not have a format tag?
+                        if(GSXML.getChildByTagName(current_node, "format") == null)
+                        {
+                            logger.error("ERROR: classifier does not have a format child");
+                            // well then create a format tag
+                            Element format_tag = config.createElement("format");
+                            current_node = (Node) current_node.appendChild(format_tag);
+                            //current_node = (Node) format_tag;
+                        }
+                        
+                        else{
+                            current_node = GSXML.getChildByTagName(current_node, "format");
+                        }
                     }
                     else{
+                        logger.error("Classifier is null");
                         current_node = GSXML.getChildByTagName(current_node, "format");
                     }
@@ -498 +520 @@
                 {
                     // look in <format> with no attributes
+                    logger.error("I presume this is search");
             
                     current_node_list = GSXML.getChildrenByTagName(current_node, "search");
@@ -532 +555 @@
                 {
                     //if(transformed.getNodeType() == Node.DOCUMENT_NODE)
-                //transformed = ((Document)transformed).getDocumentElement();
-                    logger.error("Node type: "+current_node_list.item(k).getNodeType());
+                    //transformed = ((Document)transformed).getDocumentElement();
+                    //logger.error("Node type: "+current_node_list.item(k).getNodeType());
                     if(current_node_list.item(k).getNodeType() != Node.PROCESSING_INSTRUCTION_NODE)
-                        current_node = elem.appendChild(this.doc.importNode(current_node_list.item(k),true));
+                        elem.appendChild(config.importNode(current_node_list.item(k),true));
                 }
 
@@ -546 +569 @@
                 // Now convert config document to string for writing to file
                 String new_config = this.converter.getString(config);
+
+                new_config = StringUtils.replace(new_config, "&lt;", "<");
+                new_config = StringUtils.replace(new_config, "&gt;", ">");
+                new_config = StringUtils.replace(new_config, "&quot;", "\"");
     
                 // Write to file (not original! for now)