Fix incorrect double-linking of standard CommonMark autolinks (fixes #12)

Author: colinodell

CHANGELOG.md

@@ -7,6 +7,12 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased][unreleased]
 
+## [0.3.1] - 2019-06-17
+
+### Fixed
+
+ - Fixed extension incorrectly double-linking standard CommonMark autolinks (#12)
+
 ## [0.3.0] - 2019-04-10
 
 ### Changed
@@ -41,7 +47,8 @@ This release brings the email and URL autolink processors into alignment with th
 
 Initial release!
 
-[unreleased]: https://github.com/thephpleague/commonmark-ext-autolink/compare/v0.3.0...HEAD
+[unreleased]: https://github.com/thephpleague/commonmark-ext-autolink/compare/v0.3.1...HEAD
+[0.3.1]: https://github.com/thephpleague/commonmark-ext-autolink/compare/v0.3.0...v0.3.1
 [0.3.0]: https://github.com/thephpleague/commonmark-ext-autolink/compare/v0.2.1...v0.3.0
 [0.2.1]: https://github.com/thephpleague/commonmark-ext-autolink/compare/v0.2.0...v0.2.1
 [0.2.0]: https://github.com/thephpleague/commonmark-ext-autolink/compare/v0.1.0...v0.2.0

src/EmailAutolinkProcessor.php

@@ -30,8 +30,9 @@ public function processDocument(Document $document)
         $walker = $document->walker();
 
         while ($event = $walker->next()) {
-            if ($event->getNode() instanceof Text) {
-                self::processAutolinks($event->getNode());
+            $node = $event->getNode();
+            if ($node instanceof Text && !($node->parent() instanceof Link)) {
+                self::processAutolinks($node);
             }
         }
     }

src/UrlAutolinkProcessor.php

@@ -59,8 +59,9 @@ public function processDocument(Document $document)
         $walker = $document->walker();
 
         while ($event = $walker->next()) {
-            if ($event->getNode() instanceof Text) {
-                self::processAutolinks($event->getNode(), $this->finalRegex);
+            $node = $event->getNode();
+            if ($node instanceof Text && !($node->parent() instanceof Link)) {
+                self::processAutolinks($node, $this->finalRegex);
             }
         }
     }

tests/EmailAutolinkProcessorTest.php

@@ -47,5 +47,8 @@ public function dataProviderForEmailAutolinks()
         yield ['a.b-c_d@a.b.', '<p><a href="mailto:a.b-c_d@a.b">a.b-c_d@a.b</a>.</p>'];
         yield ['a.b-c_d@a.b-', '<p>a.b-c_d@a.b-</p>'];
         yield ['a.b-c_d@a.b_', '<p>a.b-c_d@a.b_</p>'];
+
+        // Regression: CommonMark autolinks should not be double-linked
+        yield ['<foo@example.com>', '<p><a href="mailto:foo@example.com">foo@example.com</a></p>'];
     }
 }

tests/UrlAutolinkProcessorTest.php

@@ -72,5 +72,8 @@ public function dataProviderForAutolinkTests()
 
         // Regression: two links with one underscore each
         yield ["https://eventum.example.net/history.php?iss_id=107092\nhttps://gitlab.example.net/group/project/merge_requests/39#note_150630", "<p><a href=\"https://eventum.example.net/history.php?iss_id=107092\">https://eventum.example.net/history.php?iss_id=107092</a>\n<a href=\"https://gitlab.example.net/group/project/merge_requests/39#note_150630\">https://gitlab.example.net/group/project/merge_requests/39#note_150630</a></p>"];
+
+        // Regression: CommonMark autolinks should not be double-linked
+        yield ['<https://www.google.com>', '<p><a href="https://www.google.com">https://www.google.com</a></p>'];
     }
 }