PHP5 Compliance

Author: allflame

src/Bridge/DebugListener.php

@@ -1,6 +1,4 @@
 <?php
-declare(strict_types=1);
-
 namespace Jaeger\Symfony\Bridge;
 
 use Jaeger\Symfony\Debug\Extractor\DebugExtractorInterface;
@@ -37,13 +35,19 @@ public static function getSubscribedEvents()
         ];
     }
 
+    /**
+     * @return DebugListener
+     */
     public function onTerminate()
     {
         $this->debuggable->disable();
 
         return $this;
     }
 
+    /**
+     * @return DebugListener
+     */
     public function onStart()
     {
         if ('' === ($debugId = $this->extractor->getDebug())) {

src/Debug/Extractor/CookieDebugExtractor.php

@@ -1,6 +1,4 @@
 <?php
-declare(strict_types=1);
-
 namespace Jaeger\Symfony\Debug\Extractor;
 
 use Symfony\Component\EventDispatcher\EventSubscriberInterface;
@@ -14,9 +12,14 @@ class CookieDebugExtractor implements DebugExtractorInterface, EventSubscriberIn
 
     private $cookieName;
 
-    public function __construct(string $cookieName)
+    /**
+     * CookieDebugExtractor constructor.
+     *
+     * @param string $cookieName
+     */
+    public function __construct($cookieName)
     {
-        $this->cookieName = $cookieName;
+        $this->cookieName = (string)$cookieName;
     }
 
     public function onRequest(GetResponseEvent $event)
@@ -42,9 +45,12 @@ public function onTerminate()
         return $this;
     }
 
-    public function getDebug(): string
+    /**
+     * @return string
+     */
+    public function getDebug()
     {
-        return $this->debugId;
+        return (string)$this->debugId;
     }
 
     public static function getSubscribedEvents()

src/Debug/Extractor/DebugExtractorChain.php

@@ -1,25 +1,37 @@
 <?php
-declare(strict_types=1);
-
 namespace Jaeger\Symfony\Debug\Extractor;
 
 class DebugExtractorChain implements DebugExtractorInterface
 {
     private $queue;
 
+    /**
+     * DebugExtractorChain constructor.
+     *
+     * @param \SplPriorityQueue $queue
+     */
     public function __construct(\SplPriorityQueue $queue)
     {
         $this->queue = $queue;
     }
 
-    public function add(DebugExtractorInterface $extractor, int $priority = 0): DebugExtractorChain
+    /**
+     * @param DebugExtractorInterface $extractor
+     * @param int                     $priority
+     *
+     * @return DebugExtractorChain
+     */
+    public function add(DebugExtractorInterface $extractor, $priority = 0)
     {
-        $this->queue->insert($extractor, $priority);
+        $this->queue->insert($extractor, (int)$priority);
 
         return $this;
     }
 
-    public function getDebug(): string
+    /**
+     * @return string
+     */
+    public function getDebug()
     {
         $queue = clone $this->queue;
         while (false === $queue->isEmpty()) {

src/Debug/Extractor/DebugExtractorInterface.php

@@ -1,12 +1,10 @@
 <?php
-declare(strict_types=1);
-
 namespace Jaeger\Symfony\Debug\Extractor;
 
 interface DebugExtractorInterface
 {
     /**
      * @return string
      */
-    public function getDebug() : string;
+    public function getDebug();
 }

src/Debug/Extractor/EnvDebugExtractor.php

@@ -1,5 +1,4 @@
 <?php
-declare(strict_types=1);
 
 namespace Jaeger\Symfony\Debug\Extractor;
 
@@ -12,7 +11,12 @@ class EnvDebugExtractor implements DebugExtractorInterface, EventSubscriberInter
 
     private $debugId = '';
 
-    public function __construct(string $envName)
+    /**
+     * EnvDebugExtractor constructor.
+     *
+     * @param string $envName
+     */
+    public function __construct($envName)
     {
         $this->envName = $envName;
     }
@@ -32,14 +36,17 @@ public function onTerminate()
         return $this;
     }
 
-    public function getDebug(): string
+    /**
+     * @return string
+     */
+    public function getDebug()
     {
-        return $this->debugId;
+        return (string)$this->debugId;
     }
 
     public function onCommand()
     {
-        if (null === ($data = $_ENV[$this->envName] ?? null)) {
+        if (false === array_key_exists($this->envName, $_ENV)) {
             return $this;
         }
         $this->debugId = (string)$_ENV[$this->envName];

src/Resources/DependencyInjection/DebugExtractorChainCompilerPass.php

@@ -1,5 +1,4 @@
 <?php
-declare(strict_types=1);
 
 namespace Jaeger\Symfony\Resources\DependencyInjection;
 
@@ -20,7 +19,7 @@ public function process(ContainerBuilder $container)
         $definition = $container->getDefinition('jaeger.debug.extractor.chain');
         foreach ($container->findTaggedServiceIds('jaeger.debug.extractor') as $id => $tags) {
             foreach ($tags as $tag) {
-                $priority = $tag['priority'] ?? 0;
+                $priority = array_key_exists('priority', $tag) ? $tag['priority'] : 0;
                 $definition->addMethodCall('add', [new Reference($id), $priority]);
             }
         }