[web-doc] master: Ensure blank lines are copied as line breaks

[email protected] (Nilgün Belma Bugüner) Sun, 26 Jun 2022 05:00:51 +0000
Newsgroups php.doc.web
Message-ID <[email protected]>
Author: Nilgün Belma Bugüner (nilgun)
Date: 2022-06-26T07:54:06+03:00

Commit: https://github.com/php/web-doc/commit/088e06dd7e68f28b878c5d4a4a29f293b3b0fbd8
Raw diff: https://github.com/php/web-doc/commit/088e06dd7e68f28b878c5d4a4a29f293b3b0fbd8.diff

Ensure blank lines are copied as line breaks
Thank you, Lucas Azevedo.
Closes GH-35.

Changed paths:
  M  include/lib_revcheck.inc.php


Diff:

diff --git a/include/lib_revcheck.inc.php b/include/lib_revcheck.inc.php
index 6e8852c..0ff14fc 100644
--- a/include/lib_revcheck.inc.php
+++ b/include/lib_revcheck.inc.php
@@ -286,7 +286,12 @@ function showdiff ()
         $diffStartLine = substr_count($raw, "\n", 0, strpos($raw, " @@"));
 
         foreach (array_slice($lines, $diffStartLine) as $line) {
-            $fc = substr( $line , 0 , 1 );
+            $fc = substr($line, 0, 1);
+
+            $code = substr($line, 1);
+            if ($code === '') {
+                $code = "<br>";
+            }
 
             echo "<div style='display: flex;'>";
 
@@ -294,15 +299,14 @@ function showdiff ()
                 echo "<div style='$lineNumberStyles $addAccentBg'></div>";
                 echo "<div style='$lineNumberStyles $addAccentBg'>$newLineNumber</div>";
                 echo "<div style='$addAccentBg flex: 0 0 20px; text-align: center; user-select: none;'>$fc</div>";
-                if ($line == "+") $line .= " ";
-                echo "<div style='$codeStyles $addBg'>" . substr($line, 1) . "</div>\n";
+                echo "<div style='$codeStyles $addBg'>" .  $code . "</div>\n";
 
                 $newLineNumber++;
             } else if ($fc == "-") {
                 echo "<div style='$lineNumberStyles $delAccentBg'>$oldLineNumber</div>";
                 echo "<div style='$lineNumberStyles $delAccentBg'></div>";
                 echo "<div style='$delAccentBg flex: 0 0 20px; text-align: center; user-select: none;'>$fc</div>";
-                echo "<div style='$codeStyles $delBg'>" . substr($line, 1) . "</div>\n";
+                echo "<div style='$codeStyles $delBg'>" .  $code . "</div>\n";
 
                 $oldLineNumber++;
             } else if ($fc == "@") {
@@ -318,7 +322,7 @@ function showdiff ()
                 echo "<div style='$lineNumberStyles color: gray;'>$oldLineNumber</div>";
                 echo "<div style='$lineNumberStyles color: gray;'>$newLineNumber</div>";
                 echo "<div style='flex: 0 0 20px; text-align: center; user-select: none;'></div>";
-                echo "<div style='$codeStyles color: gray;'>" . substr($line, 1) . "</div>\n";
+                echo "<div style='$codeStyles color: gray;'>" .  $code . "</div>\n";
 
                 $oldLineNumber++;
                 $newLineNumber++;