Fixed issue of adding an extra <br/> tag in breaks.

Author: ishaileshmishra

Changelog.md

@@ -2,7 +2,13 @@
 
 A brief description of what changes project contains
 
-## Aug 8, 2023
+## Oct 5, 2023
+
+#### v1.2.5
+
+- The issue of adding an extra <br/> tag in breaks has been fixed
+
+## Sep 26, 2023
 
 #### v1.2.4
 

pom.xml

@@ -4,7 +4,7 @@
     <modelVersion>4.0.0</modelVersion>
     <groupId>com.contentstack.sdk</groupId>
     <artifactId>utils</artifactId>
-    <version>1.2.4</version>
+    <version>1.2.5</version>
     <packaging>jar</packaging>
     <name>Contentstack-utils</name>
     <description>Java Utils SDK for Contentstack Content Delivery API, Contentstack is a headless CMS</description>

src/main/java/com/contentstack/utils/node/NodeToHTML.java

@@ -29,8 +29,6 @@ private NodeToHTML() {
      */
     public static String textNodeToHTML(JSONObject nodeText, Option renderOption) {
         String text = nodeText.optString("text");
-
-        // compare with the nodeText options
         if (nodeText.has("superscript")) {
             text = renderOption.renderMark(MarkType.SUPERSCRIPT, text);
         }

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

@@ -54,27 +54,26 @@ public String renderOptions(JSONObject embeddedObject, Metadata metadata) {
      */
     @Override
     public String renderMark(MarkType markType, String text) {
-        // Replace "\n" with "<br/>" tags
-        text = renderHtmlWithLineBreaks(text);
+        String textContainsBreak = renderHtmlWithLineBreaks(text); // Replace "\n" with "<br/>" tags
         switch (markType) {
             case SUPERSCRIPT:
-                return "<sup>" + text + "</sup>";
+                return "<sup>" + textContainsBreak + "</sup>";
             case SUBSCRIPT:
-                return "<sub>" + text + "</sub>";
+                return "<sub>" + textContainsBreak + "</sub>";
             case INLINECODE:
-                return "<span>" + text + "</span>";
+                return "<span>" + textContainsBreak + "</span>";
             case STRIKETHROUGH:
-                return "<strike>" + text + "</strike>";
+                return "<strike>" + textContainsBreak + "</strike>";
             case UNDERLINE:
-                return "<u>" + text + "</u>";
+                return "<u>" + textContainsBreak + "</u>";
             case ITALIC:
-                return "<em>" + text + "</em>";
+                return "<em>" + textContainsBreak + "</em>";
             case BOLD:
-                return "<strong>" + text + "</strong>";
+                return "<strong>" + textContainsBreak + "</strong>";
             case BREAK:
-                return "<br />" + text;
+                return "<br />"+text;
             default:
-                return text;
+                return textContainsBreak;
         }
     }
 
@@ -101,11 +100,8 @@ private String escapeInjectHtml(JSONObject nodeObj, String nodeType) {
      */
     @Override
     public String renderNode(String nodeType, JSONObject nodeObject, NodeCallback callback) {
-
         String strAttrs = strAttrs(nodeObject);
-
         String children = callback.renderChildren(nodeObject.optJSONArray("children"));
-
         switch (nodeType) {
             case "p":
                 return "<p" + strAttrs + ">" + children + "</p>";
@@ -175,15 +171,12 @@ public String renderNode(String nodeType, JSONObject nodeObject, NodeCallback ca
      */
     private String renderHtmlWithLineBreaks(String content) {
         // Replace "\n" with "<br/>" tags
-        String htmlContent = content.replaceAll("\\n", "<br />");
-
+        return content.replaceAll("\\n", "<br />");
         // Now, you can render the HTML content
         // (You can use your rendering method here, e.g., send it to a WebView or display it in a GUI component)
-
         // For demonstration purposes, let's just print it
-        System.out.println(htmlContent);
-
-        return htmlContent;
+        // System.out.println(htmlContent);
+        // return htmlContent;
     }
 
 

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

@@ -8,6 +8,8 @@
 import org.junit.Test;
 import org.junit.runners.MethodSorters;
 
+import java.io.IOException;
+
 import static com.contentstack.utils.RTEResult.*;
 import static com.contentstack.utils.RTEString.*;
 
@@ -94,4 +96,12 @@ public void testAvailableEntryItemTypes() {
         String result = Utils.jsonToHTML(rteObject, new DefaultOption(), null);
         Assert.assertEquals(kParagraphHtml, result);
     }
+
+    @Test
+    public void testHERFID() throws IOException {
+        final String rte = "src/test/resources/wallmart/lessthanequalto.json";
+        JSONObject theRTE = new ReadResource().readJson(rte);
+        String result = Utils.jsonToHTML(theRTE, new DefaultOption(), null);
+        System.out.println(result);
+    }
 }

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

@@ -158,8 +158,6 @@ public void testRenderFunction() {
 
     @Test
     public void testCustomJSONRTE() {
-
-
         JSONObject rteObject = new JSONObject();
         String[] keyPath = {
                 "rich_text_editor", "global_rich_multiple.group.rich_text_editor"