Merge pull request #117 from snapshotpl/unused-logger

Author: WyriHaximus

Diff for: composer.json

@@ -26,7 +26,6 @@
     ],
     "require": {
         "php": "^7.1 || ^8.0",
-        "psr/log": "~1.0",
         "react/event-loop": "^1.0 || ^0.5 || ^0.4",
         "react/promise": "~2.2"
     },

Diff for: src/Bunny/AbstractClient.php

@@ -14,7 +14,6 @@
 use Bunny\Protocol\ProtocolReader;
 use Bunny\Protocol\ProtocolWriter;
 use InvalidArgumentException;
-use Psr\Log\LoggerInterface;
 use React\Promise;
 
 use function is_array;
@@ -38,9 +37,6 @@ abstract class AbstractClient
     /** @var array */
     protected $options;
 
-    /** @var LoggerInterface */
-    protected $log;
-
     /** @var resource|null */
     protected $stream;
 
@@ -87,9 +83,8 @@ abstract class AbstractClient
      * Constructor.
      *
      * @param array $options
-     * @param LoggerInterface $log
      */
-    public function __construct(array $options = [], LoggerInterface $log = null)
+    public function __construct(array $options = [])
     {
         if (!isset($options["host"])) {
             $options["host"] = "127.0.0.1";
@@ -144,7 +139,6 @@ public function __construct(array $options = [], LoggerInterface $log = null)
         }
 
         $this->options = $options;
-        $this->log = $log;
 
         $this->init();
     }

Diff for: src/Bunny/Async/Client.php

@@ -7,7 +7,6 @@
 use Bunny\Protocol\HeartbeatFrame;
 use Bunny\Protocol\MethodConnectionStartFrame;
 use Bunny\Protocol\MethodConnectionTuneFrame;
-use Psr\Log\LoggerInterface;
 use React\EventLoop\Factory;
 use React\EventLoop\LoopInterface;
 use React\EventLoop\Timer\Timer;
@@ -64,12 +63,11 @@ class Client extends AbstractClient
      *
      * @param LoopInterface $eventLoop
      * @param array $options see {@link AbstractClient} for available options
-     * @param LoggerInterface $log if argument is passed, AMQP communication will be recorded in debug level
      */
-    public function __construct(LoopInterface $eventLoop = null, array $options = [], LoggerInterface $log = null)
+    public function __construct(LoopInterface $eventLoop = null, array $options = [])
     {
         $options["async"] = true;
-        parent::__construct($options, $log);
+        parent::__construct($options);
 
         if ($eventLoop === null) {
             $eventLoop = Factory::create();