Merge pull request #59 from clue-labs/phpstan

Minor code style adjustments to make phpstan happy

Author: clue

src/ActionSender.php

@@ -133,7 +133,7 @@ public function sendText($channel, $message)
      */
     public function hangup($channel, $cause)
     {
-        return $this->request('Hangup', array('Channel' => $channel, 'Cause' => $cause));
+        return $this->request('Hangup', array('Channel' => $channel, 'Cause' => (string) $cause));
     }
 
     /**
@@ -201,7 +201,7 @@ private function boolParam($value)
 
     /**
      * @param string $name
-     * @param array $args
+     * @param array<string,string|string[]|null> $args
      * @return \React\Promise\PromiseInterface<Response,\Exception>
      */
     private function request($name, array $args = array())

src/Client.php

@@ -7,6 +7,7 @@
 use Clue\React\Ami\Protocol\Event;
 use Clue\React\Ami\Protocol\Message;
 use Clue\React\Ami\Protocol\Parser;
+use Clue\React\Ami\Protocol\Response;
 use Clue\React\Ami\Protocol\UnexpectedMessageException;
 use Evenement\EventEmitter;
 use React\Promise\Deferred;
@@ -102,6 +103,8 @@ public function handleMessage(Message $message)
             $this->emit('event', array($message));
             return;
         }
+
+        assert($message instanceof Response);
         $id = $message->getActionId();
         if (!isset($this->pending[$id])) {
             $this->emit('error', array(new UnexpectedMessageException($message), $this));
@@ -187,7 +190,7 @@ public function isBusy()
      * ```
      *
      * @param string $name
-     * @param array $args
+     * @param array<string,string|string[]|null> $args
      * @return Action
      */
     public function createAction($name, array $args = array())

src/Protocol/Parser.php

@@ -16,6 +16,10 @@ class Parser
     private $buffer = '';
     private $gotInitial = false;
 
+    /**
+     * @param string $chunk
+     * @return array<Event|Response>
+     */
     public function push($chunk)
     {
         $this->buffer .= $chunk;