Fixed issue paquettg#97

Author: paquettg

CHANGELOG.md

@@ -7,6 +7,11 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+### Changed
+- Fixed issue with \ causing an infite loop.
+
+## 2.2.0
+
 ### Added
 - Added support for php 7.4.
 - Added custom header support for curl request.

src/PHPHtmlParser/Dom.php

@@ -708,7 +708,7 @@ protected function parseTag(): array
                     case '"':
                         $attr['doubleQuote'] = true;
                         $this->content->fastForward(1);
-                        $string = $this->content->copyUntil('"', true, true);
+                        $string = $this->content->copyUntil('"', true);
                         do {
                             $moreString = $this->content->copyUntilUnless('"', '=>');
                             $string .= $moreString;
@@ -720,7 +720,7 @@ protected function parseTag(): array
                     case "'":
                         $attr['doubleQuote'] = false;
                         $this->content->fastForward(1);
-                        $string = $this->content->copyUntil("'", true, true);
+                        $string = $this->content->copyUntil("'", true);
                         do {
                             $moreString = $this->content->copyUntilUnless("'", '=>');
                             $string .= $moreString;

tests/DomTest.php

@@ -142,6 +142,13 @@ public function testLoadNoValueAttribute()
         $this->assertEquals('<div class="content"><div class="grid-container" ui-view>Main content here</div></div>', $dom->innerHtml);
     }
 
+    public function testLoadBackslashAttributeValue()
+    {
+        $dom = new Dom;
+        $dom->load('<div class="content"><div id="\" class="grid-container" ui-view>Main content here</div></div>');
+        $this->assertEquals('<div class="content"><div id="\" class="grid-container" ui-view>Main content here</div></div>', $dom->innerHtml);
+    }
+
     public function testLoadNoValueAttributeBefore()
     {
         $dom = new Dom;
@@ -504,4 +511,13 @@ public function testMultipleSquareSelector()
         $items = $dom->find('input[type=text][name=foo][baz=fig]');
         $this->assertEquals(1, count($items));
     }
+
+    public function testLoadGetAttributeWithBackslash()
+    {
+        $dom = new Dom();
+        $dom->load('<div><a href="/test/"><img alt="\" src="/img/test.png" /><br /></a><a href="/demo/"><img alt="demo" src="/img/demo.png" /></a></div>');
+        $imgs = $dom->find('img', 0);
+        $this->assertEquals("/img/test.png", $imgs->getAttribute('src'));
+
+    }
 }