Revert "fixing display issues when using <> or \" characters in scena…

…rio outl…" This reverts commit 38f02ca.
serenity-bdd · Sep 5, 2024 · 6ae8e10 · 6ae8e10
1 parent 38f02ca
commit 6ae8e10
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 11 deletions.
diff --git a/serenity-report-resources/src/main/resources/freemarker/default.ftl b/serenity-report-resources/src/main/resources/freemarker/default.ftl
@@ -38,14 +38,11 @@
         <#list breadcrumbs as breadcrumb>
             <#assign breadcrumbReport = absoluteReportName.forRequirement(breadcrumb) />
             <#assign breadcrumbTitle = breadcrumb.displayName > <!-- inflection.of(breadcrumb.displayName).asATitle()-->
-            > <a href="${breadcrumbReport}">
-            ${formatter.escapeHtmlTags(formatter.htmlCompatibleStoryTitle(breadcrumbTitle))}
-        </a>
+            > <a href="${breadcrumbReport}">${formatter.htmlCompatibleStoryTitle(breadcrumbTitle)}</a>
         </#list>
 <#--        > ${formatter.htmlCompatibleTestTitle(formatter.renderTitle(testOutcome.title))}-->
 <#--        > ${formatter.htmlCompatibleTestTitle(testOutcome.title)}-->
-        >
-            ${formatter.escapeHtmlTags(formatter.htmlCompatibleTestTitle(formatter.humanReadableFormOf(testOutcome.title)))}
+        > ${formatter.htmlCompatibleTestTitle(formatter.humanReadableFormOf(testOutcome.title))}
         </span>
         </div>
         <div class="rightbg"></div>
@@ -349,7 +346,7 @@
                                 <#assign roeResult = row.result/>
                             </#if>
                             <td class="test-${roeResult}"><a
-                                        href="#${rowIndex}"><#outputformat 'HTML'>${formatter.plainHtmlCompatible(value)}</#outputformat></a>
+                                        href="#${rowIndex}">${formatter.plainHtmlCompatible(value)}</a>
                             </td>
                         </#list>
                     </tr>

diff --git a/serenity-report-resources/src/main/resources/freemarker/junit5/default.ftl b/serenity-report-resources/src/main/resources/freemarker/junit5/default.ftl
@@ -338,7 +338,7 @@
                                     <#assign roeResult = row.result/>
                                 </#if>
                                 <td class="test-${roeResult}"><a
-                                            href="#${rowIndex}"><#outputformat 'HTML'>${formatter.plainHtmlCompatible(value)}</#outputformat></a>
+                                            href="#${rowIndex}">${formatter.plainHtmlCompatible(value)}</a>
                                 </td>
                             </#list>
                         </tr>

diff --git a/serenity-reports/src/main/java/net/thucydides/core/reports/html/Formatter.java b/serenity-reports/src/main/java/net/thucydides/core/reports/html/Formatter.java
@@ -481,10 +481,6 @@ public String htmlCompatibleStoryTitle(Object fieldValue) {
                 (htmlCompatible(renderMarkdownWithoutTags(firstLine))) : htmlCompatible(firstLine);
     }
 
-    public String escapeHtmlTags(String fieldValue) {
-        return fieldValue.replace("<", "&lt;").replace(">", "&gt;");
-    }
-
     public String htmlCompatibleTestTitle(Object fieldValue) {
         String firstLine = fieldValue.toString().split("\\n")[0];