[DomCrawler] Don't pass null to trim()

derrabus · derrabus · commit 51eda098edff · 2021-08-28T17:40:01.000+02:00
Signed-off-by: Alexander M. Turek &lt;me@derrabus.de&gt;
diff --git a/src/Symfony/Component/DomCrawler/Test/Constraint/CrawlerSelectorAttributeValueSame.php b/src/Symfony/Component/DomCrawler/Test/Constraint/CrawlerSelectorAttributeValueSame.php
@@ -47,7 +47,7 @@ protected function matches($crawler): bool
             return false;
         }
 
-        return $this->expectedText === trim($crawler->attr($this->attribute));
+        return $this->expectedText === trim($crawler->attr($this->attribute) ?? '');
     }
 
     /**
diff --git a/src/Symfony/Component/DomCrawler/Tests/Test/Constraint/CrawlerSelectorAttributeValueSameTest.php b/src/Symfony/Component/DomCrawler/Tests/Test/Constraint/CrawlerSelectorAttributeValueSameTest.php
@@ -23,6 +23,7 @@ public function testConstraint()
     {
         $constraint = new CrawlerSelectorAttributeValueSame('input[name="username"]', 'value', 'Fabien');
         $this->assertTrue($constraint->evaluate(new Crawler('<html><body><form><input type="text" name="username" value="Fabien">'), '', true));
+        $this->assertFalse($constraint->evaluate(new Crawler('<html><body><form><input type="text" name="username">'), '', true));
         $this->assertFalse($constraint->evaluate(new Crawler('<html><head><title>Bar'), '', true));
 
         try {

Original file line number	Diff line number	Diff line change
`@@ -47,7 +47,7 @@ protected function matches($crawler): bool`
`47`	`47`	`return false;`
`48`	`48`	`}`
`49`	`49`
`50`		`- return $this->expectedText === trim($crawler->attr($this->attribute));`
	`50`	`+ return $this->expectedText === trim($crawler->attr($this->attribute) ?? '');`
`51`	`51`	`}`
`52`	`52`
`53`	`53`	`/**`
Original file line number	Diff line number	Diff line change
`@@ -23,6 +23,7 @@ public function testConstraint()`
`23`	`23`	`{`
`24`	`24`	`$constraint = new CrawlerSelectorAttributeValueSame('input[name="username"]', 'value', 'Fabien');`
`25`	`25`	`$this->assertTrue($constraint->evaluate(new Crawler('<html><body><form><input type="text" name="username" value="Fabien">'), '', true));`
	`26`	`+ $this->assertFalse($constraint->evaluate(new Crawler('<html><body><form><input type="text" name="username">'), '', true));`
`26`	`27`	`$this->assertFalse($constraint->evaluate(new Crawler('<html><head><title>Bar'), '', true));`
`27`	`28`
`28`	`29`	`try {`