Remove useless emptyness checks in HtmlOutputNodeA
authorTim Düsterhus <duesterhus@woltlab.com>
Mon, 18 Sep 2023 09:54:18 +0000 (11:54 +0200)
committerTim Düsterhus <duesterhus@woltlab.com>
Mon, 18 Sep 2023 09:55:30 +0000 (11:55 +0200)
`$value` is guaranteed to be non-empty at this point.

wcfsetup/install/files/lib/system/html/output/node/HtmlOutputNodeA.class.php

index 9dc12b6a9b5ecbaa49e1ed24a00cde736f227cb4..55e7b99d664a5fb8e4644b06ea9fa721bad22b53 100644 (file)
@@ -63,7 +63,7 @@ class HtmlOutputNodeA extends AbstractHtmlOutputNode
             }
 
             if ($this->outputType === 'text/html' || $this->outputType === 'text/simplified-html') {
-                if (!empty($value) && $value === $href->__toString()) {
+                if ($value === $href->__toString()) {
                     while ($element->childNodes->length) {
                         DOMUtil::removeNode($element->childNodes->item(0));
                     }
@@ -96,7 +96,7 @@ class HtmlOutputNodeA extends AbstractHtmlOutputNode
                     );
                 }
             } elseif ($this->outputType === 'text/plain') {
-                if (!empty($value) && $value !== $href->__toString()) {
+                if ($value !== $href->__toString()) {
                     $text = $value . ' [URL:' . $href->__toString() . ']';
                 } else {
                     $text = $href->__toString();