Revert "Add return type hints to resolve Symfony deprecation notices (#…

…22)" (#23)

This reverts commit 1eafc1b.

src/Build/BuildContext.php

@@ -25,16 +25,20 @@ public function __construct(array $params)
 
     /**
      * @param array<string, mixed> $params
+     *
+     * @return BuildContext
      */
-    public function change(array $params): self
+    public function change(array $params)
     {
         return new self(array_merge($this->params, $params));
     }
 
     /**
+     * @param string $name
+     *
      * @return mixed|null
      */
-    public function get(string $name)
+    public function get($name)
     {
         return isset($this->params[$name]) ? $this->params[$name] : null;
     }

src/Build/BuildDirector.php

@@ -16,8 +16,5 @@
  */
 interface BuildDirector
 {
-    /**
-     * @return void
-     */
     public function build(BuildContext $c, Tree $t, BuildDispatcher $d);
 }

src/Build/TreeFactory.php

@@ -41,7 +41,7 @@ class TreeFactory implements ServiceSubscriberInterface
     /** @var ContainerInterface */
     protected $container;
 
-    public static function getSubscribedServices(): array
+    public static function getSubscribedServices()
     {
         return [
             BuildDispatcher::class,
@@ -64,14 +64,19 @@ public function __construct(
         $this->stopwatch = $stopwatch;
     }
 
-    public function debug(string $msg): void
+    public function debug($msg)
     {
         if ($this->logger) {
             $this->logger->debug("$msg (PID ".getmypid().', microtime '.microtime().')');
         }
     }
 
-    protected function startTiming(string $sectionName): ?StopwatchEvent
+    /**
+     * @param $sectionName
+     *
+     * @return StopwatchEvent|null
+     */
+    protected function startTiming($sectionName)
     {
         if ($this->stopwatch) {
             return $this->stopwatch->start('webfactory/navigation-bundle: '.$sectionName);
@@ -80,14 +85,17 @@ protected function startTiming(string $sectionName): ?StopwatchEvent
         return null;
     }
 
-    protected function stopTiming(StopwatchEvent $watch = null): void
+    protected function stopTiming(StopwatchEvent $watch = null)
     {
         if ($watch) {
             $watch->stop();
         }
     }
 
-    public function getTree(): Tree
+    /**
+     * @return Tree
+     */
+    public function getTree()
     {
         if (!$this->_tree) {
             $self = $this;
@@ -116,7 +124,7 @@ public function getTree(): Tree
         return $this->_tree;
     }
 
-    public function buildTreeCache(ConfigCacheInterface $cache): void
+    public function buildTreeCache(ConfigCacheInterface $cache)
     {
         $this->_tree = new Tree();
         // Dynamic (runtime) lookup:

src/Command/DumpTreeCommand.php

@@ -32,7 +32,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
         return 0;
     }
 
-    private function dumpNode(Node $n, OutputInterface $output, $depth = 0): void
+    private function dumpNode(Node $n, OutputInterface $output, $depth = 0)
     {
         $first = true;
 

src/Twig/NavigationExtension.php

@@ -25,7 +25,7 @@ public function __construct(ContainerInterface $container)
         $this->container = $container;
     }
 
-    public function getFunctions(): array
+    public function getFunctions()
     {
         return [
             new TwigFunction('navigation_tree', [$this, 'renderTree'], ['needs_environment' => true, 'is_safe' => ['html']]),

src/WebfactoryNavigationBundle.php

@@ -14,7 +14,7 @@
 
 class WebfactoryNavigationBundle extends Bundle
 {
-    public function build(ContainerBuilder $container): void
+    public function build(ContainerBuilder $container)
     {
         $container->addCompilerPass(new BuildDirectorPass());
     }