feat: Added a Method to Fix Bare Line Feed

WebFiori · Jul 3, 2024 · 23a0dc0 · 23a0dc0
1 parent c8b8a52
commit 23a0dc0
Show file tree

Hide file tree

Showing 2 changed files with 35 additions and 2 deletions.
diff --git a/tests/webfiori/test/ui/LoadTemplateTest.php b/tests/webfiori/test/ui/LoadTemplateTest.php
@@ -146,7 +146,6 @@ public function test09() {
         $compiler = new TemplateCompiler('template.html');
         $this->assertEquals("<div v-if=\"someVar <= 6 || someVar >= 8 || someVar === 6\">\r\n"
                 . "    <script>\r\n"
-                . "        \r\n"
                 . "        function allIsGood() {\r\n"
                 . "            if (a > 6) {\r\n"
                 . "                alert(\"Oh. A is > 6 but probably < 100.\");\r\n"

diff --git a/webfiori/ui/HTMLNode.php b/webfiori/ui/HTMLNode.php
@@ -756,10 +756,44 @@ public static function createComment(string $text) : HTMLNode {
      */
     public static function createTextNode(string $nodeText, bool $escHtmlEntities = true) : HTMLNode {
         $text = new HTMLNode(self::TEXT_NODE);
-        $text->setText($nodeText, $escHtmlEntities);
+        $text->setText(self::fixBareLineFeed($nodeText), $escHtmlEntities);
 
         return $text;
     }
+    /**
+     * Removes bare line feed characters (LF) and replaces them with CRLF.
+     * 
+     * A bare line feed is LF which was not preceded by a carriage return (CR).
+     * 
+     * @param string $str The string to be fixed.
+     * 
+     * @return string The method will return a string with all bare line feed
+     * characters replaced with CRLF.
+     */
+    public static function fixBareLineFeed(string $str) : string {
+        $finalStr = '';
+        $index = 0;
+        $len = strlen($str);
+
+        for ($index = 0 ; $index < $len ; $index++) {
+            $char = $str[$index];
+
+            if ($char == "\n") {
+
+                if ($index != 0 && $str[$index - 1] != "\r") {
+                    //Bare line feed found. Replace with \r\n
+                    $finalStr = trim($finalStr).HTMLDoc::NL;
+                } else {
+                    $finalStr .= $char;
+                }
+            } else {
+                $finalStr .= $char;
+            }
+        }
+
+        return $finalStr;
+    }
+
     #[ReturnTypeWillChange]
     /**
      * Returns the element that the iterator is currently is pointing to.