LaravelRUS / GitterBot

app/Domains/RoomManager.php

Unused Use Statements -1 removed lines

@@ -13,7 +13,6 @@
 
 use Domains\Room\RoomInterface;
 use Illuminate\Support\Collection;
-use Interfaces\Gitter\StandartGitterRoom;
 
 /**
  * Class RoomManager

app/Interfaces/Gitter/Karma/Validator.php

Doc Comments +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
 
     /**
      * @param Message $message
-     * @return Status[]|Collection
+     * @return Collection
      */
     public function validate(Message $message)
     {

app/Interfaces/Gitter/Support/PriorityList.php

Doc Comments +1 added lines, -1 removed lines

@@ -31,7 +31,7 @@
     }
 
     /**
-     * @param $value
+     * @param \Domains\Middleware\MiddlewareInterface $value
      * @param int $priority
      * @return $this
      */

app/Interfaces/Slack/UserMapper.php

Doc Comments -1 removed lines

@@ -20,7 +20,6 @@
 class UserMapper
 {
     /**
-     * @param array|\StdClass $attributes
      * @return User
      * @throws InvalidArgumentException
      */

Spacing +2 added lines, -2 removed lines

@@ -26,7 +26,7 @@ 
      */
     public static function fromSlackObject(\Slack\User $user)
     {
-        $values = (new AttributeMapper((array) $user->data))
+        $values = (new AttributeMapper((array)$user->data))
             ->rename('id', 'gitter_id')
             ->rename('name', 'login')
             ->rename('real_name', 'name')
@@ -38,7 +38,7 @@ 
 
         $user = User::where('gitter_id', $values['gitter_id'])->first();
         if (!$user) {
-            $user = User::unguarded(function () use ($values) {
+            $user = User::unguarded(function() use ($values) {
                 return User::create($values);
             });
         }

app/Interfaces/Slack/TextParser.php

Spacing +1 added lines, -1 removed lines

@@ -21,7 +21,7 @@
 
         $this->text = preg_replace_callback('%\[url\s*=\s*("(?:[^"]*")|\A[^\']*\Z|(?:[^\'">\]\s]+))\s*(?:[^]\s]*)\]([\W\D\w\s]*?)\[/url\]%iu',
 
-            function ($matches) {
+            function($matches) {
                 if (isset($matches[1]) && isset($matches[2]))
                     return "<".$matches[1]."|".$matches[2].">";
                 else

Braces +5 added lines, -4 removed lines

@@ -22,10 +22,11 @@
         $this->text = preg_replace_callback('%\[url\s*=\s*("(?:[^"]*")|\A[^\']*\Z|(?:[^\'">\]\s]+))\s*(?:[^]\s]*)\]([\W\D\w\s]*?)\[/url\]%iu',
 
             function ($matches) {
-                if (isset($matches[1]) && isset($matches[2]))
-                    return "<".$matches[1]."|".$matches[2].">";
-                else
-                    throw new \RuntimeException(sprintf("Text identified by '%d' has malformed BBCode urls", $this->id));
+                if (isset($matches[1]) && isset($matches[2])) {
+                                    return "<".$matches[1]."|".$matches[2].">";
+                } else {
+                                    throw new \RuntimeException(sprintf("Text identified by '%d' has malformed BBCode urls", $this->id));
+                }
             },
 
             $this->text

app/Interfaces/Slack/Client.php

Spacing +7 added lines, -7 removed lines

@@ -67,9 +67,9 @@ 
      */
     public function sendMessage(RoomInterface $room, $message)
     {
-        $this->client->getChannelById($room->id())->then(function (\Slack\Channel $channel) use($message) {
+        $this->client->getChannelById($room->id())->then(function(\Slack\Channel $channel) use($message) {
             $this->client->apiCall('chat.postMessage', [
-                'text' => (string) $this->parser->parse($message),
+                'text' => (string)$this->parser->parse($message),
                 'channel' => $channel->getId(),
                 'as_user' => true,
             ]);
@@ -83,7 +83,7 @@ 
      */
     public function listen(RoomInterface $room)
     {
-        $this->client->on('message', function (\Slack\Payload $msg) use($room) {
+        $this->client->on('message', function(\Slack\Payload $msg) use($room) {
             if ($msg->getData()['channel'] != $room->id()) {
                 return;
             }
@@ -99,7 +99,7 @@ 
             );
         });
 
-        $this->client->on('reaction_added', function (\Slack\Payload $payload) use($room) {
+        $this->client->on('reaction_added', function(\Slack\Payload $payload) use($room) {
             if ($payload->getData()['item']['type'] != 'message' and $payload->getData()['item']['channel'] != $room->id()) {
                 return;
             }
@@ -172,9 +172,9 @@ 
     protected function logException(\Exception $e)
     {
         \Log::error(
-            $e->getMessage() . ' in ' . $e->getFile() . ':' . $e->getLine() . "\n" .
-            $e->getTraceAsString() . "\n" .
-            str_repeat('=', 80) . "\n"
+            $e->getMessage().' in '.$e->getFile().':'.$e->getLine()."\n".
+            $e->getTraceAsString()."\n".
+            str_repeat('=', 80)."\n"
         );
     }
 

app/Interfaces/Slack/MessageMapper.php

Spacing +3 added lines, -3 removed lines

@@ -35,13 +35,13 @@
 
         $this->attributes = (new AttributeMapper($attributes))
             ->rename('channel', 'room_id')
-            ->value('user', function ($user) use($room) {
+            ->value('user', function($user) use($room) {
                 return $room->client()->getUserById($user);
             })
-            ->value('ts', function ($date) {
+            ->value('ts', function($date) {
                 return Carbon::createFromTimestamp($date)->setTimezone('Europe/Moscow');
             }, 'created_at')
-            ->value('mentions', function ($ids) use($room) {
+            ->value('mentions', function($ids) use($room) {
                 $users = [];
                 if (is_array($ids)) {
                     foreach ($ids as $userId) {

app/Interfaces/Google/GoogleSearch.php

Spacing +1 added lines, -1 removed lines

@@ -94,7 +94,7 @@
             $result[] = sprintf('[*] [url=%s]%s[/url]', $link->link, $link->title);
         }
 
-        if (! empty($result)) {
+        if (!empty($result)) {
             $result = '[list]'.implode(PHP_EOL, $result).'[/list]';
         }
 

app/Interfaces/Gitter/TextParser.php

Spacing +4 added lines, -4 removed lines

@@ -26,7 +26,7 @@ 
 
         $this->text = preg_replace_callback('%\[i\]([\W\D\w\s]*?)\[/i\]%iu',
 
-            function ($matches) {
+            function($matches) {
                 return "_".trim($matches[1], " ")."_";
             },
 
@@ -42,7 +42,7 @@ 
 
         $this->text = preg_replace_callback('%\[user\]([\W\D\w\s]*?)\[/user\]%iu',
 
-            function ($matches) {
+            function($matches) {
                 $username = trim($matches[1]);
 
                 return empty($username) ? "" : "@{$username}";
@@ -59,7 +59,7 @@ 
 
         $this->text = preg_replace_callback('%\[pre\]([\W\D\w\s]*?)\[/pre\]%iu',
 
-            function ($matches) {
+            function($matches) {
                 return "`{$matches[1]}``";
             },
 
@@ -74,7 +74,7 @@ 
 
         $this->text = preg_replace_callback('%\[h([0-6]{1})\]([\W\D\w\s]*?)\[/h[0-6]?\]%iu',
 
-            function ($matches) {
+            function($matches) {
                 $size = $matches[1];
 
                 return str_repeat('#', $size).' '.$matches[2].PHP_EOL;