Merge pull request #1561 from hydephp/improve-markdown-to-plaintext-c…

…onverter

Update Markdown to plaintext formatter to trim whitespace hydephp/develop@296556d

src/Framework/Actions/ConvertsMarkdownToPlainText.php

@@ -93,6 +93,7 @@ protected function applyStringTransformations(string $markdown): string
         foreach ($lines as $line => $contents) {
             $contents = $this->removeTables($contents);
             $contents = $this->removeBlockquotes($contents);
+            $contents = $this->trimWhitespace($contents);
 
             $lines[$line] = $contents;
         }
@@ -127,4 +128,16 @@ protected function removeBlockquotes(string $contents): string
 
         return $contents;
     }
+
+    protected function trimWhitespace(string $contents): string
+    {
+        // If it is a list, don't trim the whitespace
+        $firstCharacter = substr(trim($contents), 0, 1);
+
+        if ($firstCharacter === '-' || $firstCharacter === '*' || $firstCharacter === '+' || is_numeric($firstCharacter)) {
+            return $contents;
+        }
+
+        return trim($contents);
+    }
 }

tests/Feature/Actions/ConvertsMarkdownToPlainTextTest.php

@@ -259,11 +259,11 @@ public function testItRemovesCode()
     public function testItRemovesCodeBlocks()
     {
         $markdown = <<<'MD'
-            <p>Hello World</p>
+        <p>Hello World</p>
         MD;
 
         $text = <<<'TXT'
-            Hello World
+        Hello World
         TXT;
 
         $this->assertSame($text, $this->convert($markdown));
@@ -367,9 +367,9 @@ public function testItRemovesHtml()
         $text = <<<'TXT'
         This word is bold. This word is italic.
 
-            
-                &copy; My Company
-            
+        
+        &copy; My Company
+        
 
         Hello World
         TXT;
@@ -487,6 +487,23 @@ public function testItRemovesTables()
         $this->assertSame($text, $this->convert($markdown));
     }
 
+    public function testItTrimsIndentation()
+    {
+        $markdown = <<<'MD'
+        foo
+            bar
+                baz
+        MD;
+
+        $text = <<<'TXT'
+        foo
+        bar
+        baz
+        TXT;
+
+        $this->assertSame($text, $this->convert($markdown));
+    }
+
     public function testWithEmptyString()
     {
         $this->assertSame('', $this->convert(''));