Bug 12077: in help, attribute bgcolor was not handled in tables. 86/9686/2
Calixte DENIZET [Wed, 14 Nov 2012 10:00:03 +0000 (11:00 +0100)]
Change-Id: I2105773b10359d16634be3c1d3d5772ddd67cddf

scilab/CHANGES_5.4.X
scilab/modules/helptools/src/java/org/scilab/modules/helptools/HTMLDocbookTagConverter.java

index ad2d6be..6d14c75 100644 (file)
@@ -96,6 +96,8 @@ Bug fixes
 
 * Bug #12050 fixed - Fix a typo: dimensionnal => dimensional.
 
+* Bug #12077 fixed - In help, attribute bgcolor was not handled in tables.
+
 
                     Changes between version 5.3.3 and 5.4.0
                     =======================================
index 26d6872..18c1c3c 100644 (file)
@@ -209,11 +209,13 @@ public class HTMLDocbookTagConverter extends DocbookTagConverter implements Temp
         buffer.append(tag);
         if (attrs != null) {
             for (int i = 0; i < attrs.length; i += 2) {
-                buffer.append(" ");
-                buffer.append(attrs[i]);
-                buffer.append("=\"");
-                buffer.append(attrs[i + 1]);
-                buffer.append("\"");
+                if (attrs[i + 1] != null && !attrs[i + 1].isEmpty()) {
+                    buffer.append(" ");
+                    buffer.append(attrs[i]);
+                    buffer.append("=\"");
+                    buffer.append(attrs[i + 1]);
+                    buffer.append("\"");
+                }
             }
         }
 
@@ -1198,7 +1200,8 @@ public class HTMLDocbookTagConverter extends DocbookTagConverter implements Temp
      * @throws SAXEception if an error is encountered
      */
     public String handleTr(final Map<String, String> attributes, final String contents) throws SAXException {
-        return encloseContents("tr", contents);
+        String bgcolor = attributes.get("bgcolor");
+        return encloseContents("tr", new String[] {"bgcolor", bgcolor}, contents);
     }
 
     /**
@@ -1210,10 +1213,8 @@ public class HTMLDocbookTagConverter extends DocbookTagConverter implements Temp
      */
     public String handleTd(final Map<String, String> attributes, final String contents) throws SAXException {
         String align = attributes.get("align");
-        if (align == null) {
-            return encloseContents("td", new String[] {"align", align}, contents);
-        }
-        return encloseContents("td", contents);
+        String bgcolor = attributes.get("bgcolor");
+        return encloseContents("td", new String[] {"align", align, "bgcolor", bgcolor}, contents);
     }
 
     /**
@@ -1225,18 +1226,10 @@ public class HTMLDocbookTagConverter extends DocbookTagConverter implements Temp
      */
     public String handleInformaltable(final Map<String, String> attributes, final String contents) throws SAXException {
         String id = attributes.get("id");
+        String bgcolor = attributes.get("bgcolor");
         String border = attributes.get("border");
-        if (border == null) {
-            border = "";
-        }
         String cellpadding = attributes.get("cellpadding");
-        if (cellpadding == null) {
-            cellpadding = "";
-        }
         String width = attributes.get("width");
-        if (width == null) {
-            width = "";
-        }
         if (id != null) {
             return "<a name=\"" + id + "\"></a>" + encloseContents("table", new String[] {"class", "informaltable", "border", border, "cellpadding", cellpadding, "width", width}, contents);
         } else {
@@ -1485,10 +1478,14 @@ public class HTMLDocbookTagConverter extends DocbookTagConverter implements Temp
      */
     public String handleTable(final Map<String, String> attributes, final String contents) throws SAXException {
         String id = attributes.get("id");
+        String bgcolor = attributes.get("bgcolor");
+        String border = attributes.get("border");
+        String cellpadding = attributes.get("cellpadding");
+
         if (id != null) {
-            return "<a name=\"" + id + "\"></a>" + encloseContents("table", "doctable", contents);
+            return "<a name=\"" + id + "\"></a>" + encloseContents("table", new String[] {"class", "doctable", "bgcolor", bgcolor, "border", border, "cellpadding", cellpadding}, contents);
         } else {
-            return encloseContents("table", "doctable", contents);
+            return encloseContents("table", new String[] {"class", "doctable", "bgcolor", bgcolor, "border", border, "cellpadding", cellpadding}, contents);
         }
     }