rector and codesniffer fixes

Author: splitbrain

inc/TreeBuilder/AbstractBuilder.php

@@ -2,7 +2,6 @@
 
 namespace dokuwiki\TreeBuilder;
 
-
 use dokuwiki\test\mock\Doku_Renderer;
 use dokuwiki\TreeBuilder\Node\AbstractNode;
 use dokuwiki\TreeBuilder\Node\ExternalLink;
@@ -145,7 +144,7 @@ public function getTop(): Top
     public function getAll(): array
     {
         if (!$this->generated) throw new \RuntimeException('need to call generate() first');
-        if (empty($this->nodes)) {
+        if ($this->nodes === []) {
             $this->nodes = [];
             foreach ($this->top->getDescendants() as $node) {
                 $this->nodes[$node->getId()] = $node;
@@ -163,9 +162,7 @@ public function getAll(): array
     public function getLeaves(): array
     {
         if (!$this->generated) throw new \RuntimeException('need to call generate() first');
-        return array_filter($this->getAll(), function ($page) {
-            return !$page->getChildren();
-        });
+        return array_filter($this->getAll(), fn($page) => !$page->getChildren());
     }
 
     /**
@@ -176,9 +173,7 @@ public function getLeaves(): array
     public function getBranches(): array
     {
         if (!$this->generated) throw new \RuntimeException('need to call generate() first');
-        return array_filter($this->getAll(), function ($page) {
-            return !!$page->getChildren();
-        });
+        return array_filter($this->getAll(), fn($page) => (bool) $page->getChildren());
     }
 
     /**
@@ -218,7 +213,7 @@ public function render(Doku_Renderer $R, $top = null, $level = 1): void
         foreach ($top->getChildren() as $node) {
             $R->listitem_open(1, $node->hasChildren());
             $R->listcontent_open();
-            if (is_a($node, ExternalLink::class)) {
+            if ($node instanceof ExternalLink) {
                 $R->externallink($node->getId(), $node->getTitle());
             } else {
                 $R->internallink($node->getId(), $node->getTitle());
@@ -261,6 +256,6 @@ protected function applyRecursionDecision(AbstractNode $node, int $depth): bool
      */
     public function __toString(): string
     {
-        return join("\n", $this->getAll());
+        return implode("\n", $this->getAll());
     }
 }

inc/TreeBuilder/ControlPageBuilder.php

@@ -17,9 +17,9 @@
 class ControlPageBuilder extends AbstractBuilder
 {
     /** @var int do not include internal links */
-    const FLAG_NOINTERNAL = 1;
+    public const FLAG_NOINTERNAL = 1;
     /** @var int do not include external links */
-    const FLAG_NOEXTERNAL = 2;
+    public const FLAG_NOEXTERNAL = 2;
 
     /** @var string */
     protected string $controlPage;
@@ -66,7 +66,7 @@ public function generate(): void
                     break;
                 case 'listu_close':
                     // if we had a node on this level, remove it from the parents
-                    if(isset($parents[$level])) {
+                    if (isset($parents[$level])) {
                         unset($parents[$level]);
                     }
                     $level--; // close list level
@@ -89,7 +89,7 @@ public function generate(): void
                         );
                     }
 
-                    if($level) {
+                    if ($level) {
                         // remember this page as the parent for this level
                         $parents[$level] = $newpage;
                         // parent is the last page on the previous level
@@ -102,7 +102,7 @@ public function generate(): void
 
                     $newpage->setParent($parent);
                     $newpage = $this->applyNodeProcessor($newpage);
-                    if($newpage instanceof AbstractNode) {
+                    if ($newpage instanceof AbstractNode) {
                         $parent->addChild($newpage);
                     }
                     break;

inc/TreeBuilder/Node/AbstractNode.php

@@ -91,7 +91,7 @@ public function getChildren(): array
      */
     public function hasChildren(): bool
     {
-        return !empty($this->children);
+        return $this->children !== [];
     }
 
     /**
@@ -178,10 +178,7 @@ public function setProperty(string $name, $value): void
      */
     public function getProperty(string $name, $default = null)
     {
-        if (isset($this->properties[$name])) {
-            return $this->properties[$name];
-        }
-        return $default;
+        return $this->properties[$name] ?? $default;
     }
 
     /**

inc/TreeBuilder/Node/ExternalLink.php

@@ -7,5 +7,4 @@
  */
 class ExternalLink extends AbstractNode
 {
-
 }

inc/TreeBuilder/Node/WikiNamespace.php

@@ -7,5 +7,4 @@
  */
 class WikiNamespace extends AbstractNode
 {
-
 }

inc/TreeBuilder/Node/WikiPage.php

@@ -7,5 +7,4 @@
  */
 class WikiPage extends AbstractNode
 {
-
 }

inc/TreeBuilder/Node/WikiStartpage.php

@@ -34,5 +34,4 @@ public function getNs(): string
     {
         return $this->originalNamespace;
     }
-
 }

inc/TreeBuilder/PageTreeBuilder.php

@@ -232,7 +232,7 @@ protected function namespacePath(string $namespace): string
         $dir = wikiFN($namespace . ':xxx');
         $dir = substr($dir, strlen($base));
         $dir = dirname($dir); // remove the 'xxx' part
-        if($dir === '.') $dir = ''; // dirname returns '.' for root namespace
+        if ($dir === '.') $dir = ''; // dirname returns '.' for root namespace
         return $dir;
     }
 
@@ -255,6 +255,4 @@ protected function applyNodeProcessor(AbstractNode $node): ?AbstractNode
         }
         return parent::applyNodeProcessor($node);
     }
-
-
 }