botman / botman

src/Storages/Drivers/FileStorage.php

Doc Comments +1 added lines, -1 removed lines

@@ -16,7 +16,7 @@
     }
 
     /**
-     * @param $key
+     * @param string $key
      * @return string
      */
     protected function getFilename($key)

Spacing +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
 
         $saved = $this->get($key)->merge($data);
 
-        if (! is_dir(dirname($file))) {
+        if ( ! is_dir(dirname($file))) {
             mkdir(dirname($file), 0777, true);
         }
         file_put_contents($file, json_encode($saved->all()));

src/Storages/Drivers/RedisStorage.php

Doc Comments +1 added lines, -1 removed lines

@@ -82,7 +82,7 @@
     /**
      * Namespace botman keys in redis.
      *
-     * @param $key
+     * @param string $key
      * @return string
      */
     private function decorateKey($key)

Spacing +1 added lines, -1 removed lines

@@ -27,7 +27,7 @@
      */
     public function __construct($host = '127.0.0.1', $port = 6379, $auth = null)
     {
-        if (! class_exists(Redis::class)) {
+        if ( ! class_exists(Redis::class)) {
             throw new RuntimeException('phpredis extension is required for RedisStorage');
         }
         $this->host = $host;

src/Middleware/Wit.php

Spacing +1 added lines, -1 removed lines

@@ -109,7 +109,7 @@
     {
         $entities = Collection::make($message->getExtras())->get('entities', []);
 
-        if (! empty($entities)) {
+        if ( ! empty($entities)) {
             foreach ($entities as $name => $entity) {
                 if ($name === 'intent') {
                     foreach ($entity as $item) {

src/Middleware/MiddlewareManager.php

Spacing +1 added lines, -1 removed lines

@@ -74,7 +74,7 @@
      */
     public function applyMiddleware($method, $payload, array $additionalMiddleware = [], Closure $destination = null)
     {
-        $destination = is_null($destination) ? function ($payload) {
+        $destination = is_null($destination) ? function($payload) {
             return $payload;
         }
         : $destination;

Doc Comments +1 added lines, -1 removed lines

@@ -62,7 +62,7 @@
 
     /**
      * @param Matching[] ...$middleware
-     * @return Matching[]|$this
+     * @return Matching[]
      */
     public function matching(Matching ...$middleware)
     {

src/Drivers/DriverManager.php

Doc Comments +1 added lines, -1 removed lines

@@ -94,7 +94,7 @@
      * @param string $name
      * @param array $config
      * @param Request|null $request
-     * @return mixed|HttpDriver|NullDriver
+     * @return HttpDriver
      */
     public static function loadFromName($name, array $config, Request $request = null)
     {

Spacing +1 added lines, -1 removed lines

@@ -47,7 +47,7 @@
      */
     public static function getAvailableHttpDrivers()
     {
-        return Collection::make(self::$drivers)->filter(function ($driver) {
+        return Collection::make(self::$drivers)->filter(function($driver) {
             return is_subclass_of($driver, HttpDriver::class);
         })->toArray();
     }

src/Traits/HandlesConversations.php

Doc Comments +1 added lines, -1 removed lines

@@ -49,7 +49,7 @@
      * Get a stored conversation array from the cache for a given message.
      *
      * @param null|IncomingMessage $message
-     * @return array
+     * @return callable
      */
     public function getStoredConversation($message = null)
     {

Spacing +6 added lines, -6 removed lines

@@ -20,7 +20,7 @@ 
      */
     public function startConversation(Conversation $instance, $recipient = null, $driver = null)
     {
-        if (! is_null($recipient) && ! is_null($driver)) {
+        if ( ! is_null($recipient) && ! is_null($driver)) {
             $this->message = new IncomingMessage('', $recipient, '');
             $this->driver = DriverManager::loadFromName($driver, $this->config);
         }
@@ -74,7 +74,7 @@ 
      */
     public function touchCurrentConversation()
     {
-        if (! is_null($this->currentConversationData)) {
+        if ( ! is_null($this->currentConversationData)) {
             $touched = $this->currentConversationData;
             $touched['time'] = microtime();
 
@@ -167,11 +167,11 @@ 
     {
         $this->loadedConversation = false;
 
-        Collection::make($this->getMessages())->reject(function (IncomingMessage $message) {
+        Collection::make($this->getMessages())->reject(function(IncomingMessage $message) {
             return $message->isFromBot();
-        })->filter(function (IncomingMessage $message) {
+        })->filter(function(IncomingMessage $message) {
             return $this->cache->has($message->getConversationIdentifier()) || $this->cache->has($message->getOriginatedConversationIdentifier());
-        })->each(function ($message) {
+        })->each(function($message) {
             $message = $this->middleware->applyMiddleware('received', $message);
             $message = $this->middleware->applyMiddleware('captured', $message);
 
@@ -245,7 +245,7 @@ 
     {
         /** @var \BotMan\BotMan\Messages\Conversations\Conversation $conversation */
         $conversation = $convo['conversation'];
-        if (! $conversation instanceof ShouldQueue) {
+        if ( ! $conversation instanceof ShouldQueue) {
             $conversation->setBot($this);
         }
         /*

src/Messages/Conversations/Conversation.php

Spacing +3 added lines, -3 removed lines

@@ -119,7 +119,7 @@ 
     {
         $conversation = $this->bot->getStoredConversation();
 
-        if (! $question instanceof Question && ! $question) {
+        if ( ! $question instanceof Question && ! $question) {
             $question = unserialize($conversation['question']);
         }
 
@@ -129,7 +129,7 @@ 
         if (is_string($next)) {
             $next = unserialize($next)->getClosure();
         } elseif (is_array($next)) {
-            $next = Collection::make($next)->map(function ($callback) {
+            $next = Collection::make($next)->map(function($callback) {
                 if ($this->bot->getDriver()->serializesCallbacks() && ! $this->bot->runsOnSocket()) {
                     $callback['callback'] = unserialize($callback['callback'])->getClosure();
                 }
@@ -183,7 +183,7 @@ 
     public function __sleep()
     {
         $properties = get_object_vars($this);
-        if (! $this instanceof ShouldQueue) {
+        if ( ! $this instanceof ShouldQueue) {
             unset($properties['bot']);
         }
 

Doc Comments +3 added lines, -3 removed lines

@@ -167,7 +167,7 @@ 
     /**
      * Should the conversation be skipped (temporarily).
      * @param  IncomingMessage $message
-     * @return bool
+     * @return boolean|null
      */
     public function skipsConversation(IncomingMessage $message)
     {
@@ -177,7 +177,7 @@ 
     /**
      * Should the conversation be removed and stopped (permanently).
      * @param  IncomingMessage $message
-     * @return bool
+     * @return boolean|null
      */
     public function stopsConversation(IncomingMessage $message)
     {
@@ -186,7 +186,7 @@ 
 
     /**
      * Override default conversation cache time (only for this conversation).
-     * @return mixed
+     * @return integer|null
      */
     public function getConversationCacheTime()
     {

src/Messages/Incoming/IncomingMessage.php

Spacing +1 added lines, -1 removed lines

@@ -87,7 +87,7 @@
      */
     public function getExtras($key = null)
     {
-        if (! is_null($key)) {
+        if ( ! is_null($key)) {
             return Collection::make($this->extras)->get($key);
         }
 

src/Commands/ConversationManager.php

Doc Comments +1 added lines, -1 removed lines

@@ -34,7 +34,7 @@
      *
      * @param IncomingMessage $message
      * @param array $parameters
-     * @return array
+     * @return callable
      */
     public function addDataParameters(IncomingMessage $message, array $parameters)
     {

Spacing +1 added lines, -1 removed lines

@@ -66,7 +66,7 @@
     public function getMatchingMessages($messages, MiddlewareManager $middleware, Answer $answer, DriverInterface $driver, $withReceivedMiddleware = true) : array
     {
         $matcher = new Matcher();
-        $messages = Collection::make($messages)->reject(function (IncomingMessage $message) {
+        $messages = Collection::make($messages)->reject(function(IncomingMessage $message) {
             return $message->isFromBot();
         });