Changeset 34241 for main/trunk/gli/src/org/greenstone/gatherer/cdm/CollectionConfigXMLReadWrite.java

Timestamp:

2020-07-03T12:18:35+12:00 (4 years ago)

Author:

ak19

Message:

The previous commit fixed the issue where HTML in collection descriptions was not being preserved when GLI was not involved. This commit fixes the remaining problems with preserving HTML in coll descriptions when GLI is involved.

File:

• main/trunk/gli/src/org/greenstone/gatherer/cdm/CollectionConfigXMLReadWrite.java (modified) (2 diffs)

main/trunk/gli/src/org/greenstone/gatherer/cdm/CollectionConfigXMLReadWrite.java

@@ -94 +94 @@
         {
             Element item = (Element) item_list.get(i);
-            String text = XMLTools.getNodeText(item);           
-
+            boolean isCollDescr = name_value.equals(StaticStrings.COLLECTIONMETADATA_COLLECTIONEXTRA_STR);
+
+            // Sadly, XMLTools.getNodeText(item) returns empty for any HTML in displayItem
+            // so we have to do it the hard way
+            String text = isCollDescr ? preserveHTMLInDescriptionDisplayItem(item)
+                : XMLTools.getNodeText(item); // else as before for all other displayItems
+
+            
             //If there is nothing to display, don't bother creating the element
             // Either lang or key should be set. If key set, no textnode.
@@ -2800 +2806 @@
     }
 
+    /**
+     * For now it only makes sends to allow a collection's about page description to contain HTML.
+     * Collection titles may go into a pre tag or text only context or into HTML headings,
+     * where preserving any HTML int coll title would become meaningless or even a hindrance.
+     * TODO: if users used HTML pre tags, they'll have wanted whitespace preserved.
+     * TODO There's probably a better way to do this, but I'm unable to think of it. Messy but works.
+     */
+    static private String preserveHTMLInDescriptionDisplayItem(Element collDescription) {
+    // First get the <displayItem> as text
+    String text = XMLTools.elementToString(collDescription, true);
+
+    // We only want the contents, not the xml processing instruction or the
+    // <displayItem></displayItem> or self-closing(!) <displayItem/> when no text for colldescr
+    // Remove xml processing instr too by removing <displayItem until first > after that
+    // this will also handle self-closing tag case
+    String lookFor = "<displayItem";
+    int start = text.indexOf(lookFor);
+    if(start != -1) {
+        start += lookFor.length();
+        start = text.indexOf(">", start);
+        text = text.substring(start+1).trim(); // trim to ensure no empty lines to deceive us
+        // especially after a self-closing tag
+    }
+
+    // If we have a positive number of lines remaining, it means it wasn't a self-closing
+    // tag. Remove the closing tag
+    String[] lines = text.split("\\r?\\n");
+    text = "";              
+    if(lines.length > 1) {
+        for (int j = 0; j < lines.length-1; j++) { // skip last line: closing tag
+        text += lines[j].trim() + "\n"; // Easiest solution:
+                                   // trim white space introduced by the one extra level of
+                                   // indentation intoduced when enclosing <dispItem> tags removed
+        }
+    }
+
+    text = text.replaceAll("&amp;", "&");// feels stupid, when we're going to change it back again soon
+               // but it's the last bit needed to get GLI to preserve html coll descriptions
+               // while displaying the html as html (without char entities) in the config file editor
+    
+    return text;//.trim(); // don't want trailing whitespace within displayItem
+    }
+    
     static public String generateStringVersion(Document doc)
     {