Merge pull request #72 from contentstack/fix/DX-707

fix: ignore td/th in case of atttrs had void:true

Author: nadeem-cs

src/main/java/com/contentstack/utils/render/DefaultOption.java

@@ -14,6 +14,7 @@
 
 public class DefaultOption implements Option {
 
+
     /**
      * The function `renderOptions` takes in a JSON object and metadata and returns a string based on the
      * style type of the metadata.
@@ -153,10 +154,19 @@ public String renderNode(String nodeType, JSONObject nodeObject, NodeCallback ca
                 return "<tfoot" + strAttrs + ">" + cleanChildren + "</tfoot>";
             case "tr":
                 return "<tr" + strAttrs + ">" + cleanChildren + "</tr>";
-            case "th":
-                return "<th" + strAttrs + ">" + cleanChildren + "</th>";
-            case "td":
-                return "<td" + strAttrs + ">" + cleanChildren + "</td>";
+            case "th":{
+                if (nodeObject.has("attrs") && nodeObject.optJSONObject("attrs").has("void") && 
+                    nodeObject.optJSONObject("attrs").optBoolean("void")) {
+                  return "";
+                }else{
+                return "<th" + strAttrs + ">" + cleanChildren + "</th>";}}
+            case "td":{
+                if (nodeObject.has("attrs") && nodeObject.optJSONObject("attrs").has("void") && 
+                    nodeObject.optJSONObject("attrs").optBoolean("void")) {
+                  return "";
+                }else{
+                return "<td" + strAttrs + ">" + cleanChildren + "</td>";}}
+              
             case "blockquote":
                 return "<blockquote" + strAttrs + ">" + cleanChildren + "</blockquote>";
             case "code":

src/test/java/com/contentstack/utils/DefaultOptionTests.java

@@ -1,15 +1,21 @@
 package com.contentstack.utils;
 
 import com.contentstack.utils.helper.Metadata;
+import com.contentstack.utils.interfaces.NodeCallback;
 import com.contentstack.utils.render.DefaultOption;
+
+
 import org.json.JSONObject;
+import org.json.JSONArray;
 import org.jsoup.nodes.Attributes;
 import org.junit.Assert;
 import org.junit.BeforeClass;
 import org.junit.FixMethodOrder;
 import org.junit.Test;
 import org.junit.runners.MethodSorters;
 
+import static org.junit.Assert.assertEquals;
+
 import java.io.IOException;
 import java.util.logging.Level;
 import java.util.logging.Logger;
@@ -82,5 +88,99 @@ public void testEmbeddedDefaultDisplayable() {
         String result = defaultOptions.renderOptions(localJsonObj.optJSONObject("_embedded_items"), metadata);
         Assert.assertEquals("<img src=\"\" alt=\"\" />", result);
     }
+     @Test
+    public void testRenderNodeWithVoidTd() {
+        DefaultOption defaultOptions = new DefaultOption();
+        JSONObject nodeObject = new JSONObject();
+        JSONObject attrs = new JSONObject();
+        attrs.put("void", true);
+        nodeObject.put("attrs", attrs);
+        nodeObject.put("children", new JSONArray());
+
+        NodeCallback callback = children -> {
+            // Simple callback implementation for testing purposes
+            StringBuilder sb = new StringBuilder();
+            for (int i = 0; i < children.length(); i++) {
+                sb.append(children.getJSONObject(i).getString("type"));
+            }
+            return sb.toString();
+        };
+
+        String result = defaultOptions.renderNode("td", nodeObject, callback);
+        Assert.assertEquals("", result);
+    }
+    @Test
+    public void testRenderNodeWithVoidTh() {
+        DefaultOption defaultOptions = new DefaultOption();
+        JSONObject nodeObject = new JSONObject();
+        JSONObject attrs = new JSONObject();
+        attrs.put("void", true);
+        nodeObject.put("attrs", attrs);
+        nodeObject.put("children", new JSONArray());
+
+        NodeCallback callback = children -> {
+            // Simple callback implementation for testing purposes
+            StringBuilder sb = new StringBuilder();
+            for (int i = 0; i < children.length(); i++) {
+                sb.append(children.getJSONObject(i).getString("type"));
+            }
+            return sb.toString();
+        };
+
+        String result = defaultOptions.renderNode("th", nodeObject, callback);
+        Assert.assertEquals("", result);
+    }
+
+    @Test
+    public void testRenderNodeWithoutVoidTd() {
+        DefaultOption defaultOptions = new DefaultOption();
+        JSONObject nodeObject = new JSONObject();
+        JSONObject attrs = new JSONObject();
+        attrs.put("class", "example");
+        nodeObject.put("attrs", attrs);
+        JSONArray children = new JSONArray();
+        JSONObject child = new JSONObject();
+        child.put("type", "text");
+        child.put("content", "example content");
+        children.put(child);
+        nodeObject.put("children", children);
+
+        NodeCallback callback = childrenArray -> {
+            StringBuilder sb = new StringBuilder();
+            for (int i = 0; i < childrenArray.length(); i++) {
+                sb.append(childrenArray.getJSONObject(i).getString("content"));
+            }
+            return sb.toString();
+        };
+
+        String result = defaultOptions.renderNode("td", nodeObject, callback);
+        Assert.assertEquals("<td class=\"example\">example content</td>", result);
+    }
+
+    @Test
+    public void testRenderNodeWithoutVoidTh() {
+        DefaultOption defaultOptions = new DefaultOption();
+        JSONObject nodeObject = new JSONObject();
+        JSONObject attrs = new JSONObject();
+        attrs.put("class", "example");
+        nodeObject.put("attrs", attrs);
+        JSONArray children = new JSONArray();
+        JSONObject child = new JSONObject();
+        child.put("type", "text");
+        child.put("content", "example content");
+        children.put(child);
+        nodeObject.put("children", children);
+
+        NodeCallback callback = childrenArray -> {
+            StringBuilder sb = new StringBuilder();
+            for (int i = 0; i < childrenArray.length(); i++) {
+                sb.append(childrenArray.getJSONObject(i).getString("content"));
+            }
+            return sb.toString();
+        };
+
+        String result = defaultOptions.renderNode("th", nodeObject, callback);
+        Assert.assertEquals("<th class=\"example\">example content</th>", result);
+    }
 
 }