LaravelRUS / GitterBot

app/Interfaces/Http/routes.php

Spacing +3 added lines, -3 removed lines

@@ -8,7 +8,7 @@ 
 Route::get('/achievements', 'HomeController@index');
 
 
-Route::group(['prefix' => 'api', 'namespace' => 'Api'], function () {
+Route::group(['prefix' => 'api', 'namespace' => 'Api'], function() {
     // Users
     Route::get('users.json', 'UsersController@index');
     Route::get('users/top.json', 'UsersController@getUsersTop');
@@ -18,12 +18,12 @@ 
     Route::get('achievements.json', 'AchievementsController@index');
 
 
-    Route::any('/{any}', function () {
+    Route::any('/{any}', function() {
         return ['error' => 'Not found'];
     })->where('any', '.*?');
 });
 
 
-Route::any('/{any}', function () {
+Route::any('/{any}', function() {
     return Redirect::to('/');
 })->where('any', '.*?');
\ No newline at end of file

app/Core/Doctrine/CacheBridge.php

Doc Comments +1 added lines, -1 removed lines

@@ -69,7 +69,7 @@
      * @param int $lifeTime The cache lifetime.
      *                         If != 0, sets a specific lifetime for this cache entry (0 => infinite lifeTime).
      *
-     * @return boolean TRUE if the entry was successfully stored in the cache, FALSE otherwise.
+     * @return boolean|null TRUE if the entry was successfully stored in the cache, FALSE otherwise.
      */
     public function save($id, $data, $lifeTime = 0)
     {

app/Domains/Message.php

Doc Comments +5 added lines, -5 removed lines

@@ -74,7 +74,7 @@ 
     }
 
     /**
-     * @return mixed|string
+     * @return string
      */
     public function getEscapedTextAttribute()
     {
@@ -87,7 +87,7 @@ 
     }
 
     /**
-     * @return mixed|string
+     * @return string
      */
     public function getTextWithoutSpecialCharsAttribute()
     {
@@ -118,7 +118,7 @@ 
     }
 
     /**
-     * @param $text
+     * @param string $text
      * @return Model
      */
     public function pre($text)
@@ -127,7 +127,7 @@ 
     }
 
     /**
-     * @param $code
+     * @param string $code
      * @param string $lang
      * @return Model
      */
@@ -157,7 +157,7 @@ 
     }
 
     /**
-     * @param $symbol
+     * @param string $symbol
      * @param $text
      * @return string
      */

Unused Use Statements -1 removed lines

@@ -11,7 +11,6 @@
 namespace Domains;
 
 use Carbon\Carbon;
-use LogicException;
 use Interfaces\Gitter\Client;
 use Core\Mappers\MessageMapperTrait;
 use Illuminate\Database\Eloquent\Model;

Spacing +3 added lines, -3 removed lines

@@ -134,7 +134,7 @@ 
     public function code($code, $lang = '')
     {
         return $this->answer(
-            '```' . $lang . "\n" . $code . "\n" . '```'
+            '```'.$lang."\n".$code."\n".'```'
         );
     }
 
@@ -167,8 +167,8 @@ 
         $strings = explode("\n", $text);
         foreach ($strings as $string) {
             $result[] =
-                $symbol .
-                str_replace($symbol, '\\' . $symbol, trim($string)) .
+                $symbol.
+                str_replace($symbol, '\\'.$symbol, trim($string)).
                 $symbol;
         }
 

app/Interfaces/Console/Commands/GitterSync.php

Unused Use Statements -1 removed lines

@@ -13,7 +13,6 @@
 use Carbon\Carbon;
 use Core\Mappers\UserMapper;
 use Domains\Message;
-use Domains\User;
 use Gitter\Client;
 use Gitter\Support\ApiIterator;
 use Illuminate\Console\Command;

Spacing +21 added lines, -21 removed lines

@@ -59,10 +59,10 @@ 
 
 
         // Sync users
-        $db->transaction(function () use ($container) {
-            $this->comment('Start users synchronize at ' . ($start = Carbon::now()));
+        $db->transaction(function() use ($container) {
+            $this->comment('Start users synchronize at '.($start = Carbon::now()));
             $container->call([$this, 'importUsers']);
-            $this->comment('Ends ' . Carbon::now()->diffForHumans($start));
+            $this->comment('Ends '.Carbon::now()->diffForHumans($start));
         });
 
 
@@ -70,10 +70,10 @@ 
 
 
         // Sync messages
-        $db->transaction(function () use ($container) {
-            $this->comment('Start messages synchronize at ' . ($start = Carbon::now()));
+        $db->transaction(function() use ($container) {
+            $this->comment('Start messages synchronize at '.($start = Carbon::now()));
             $container->call([$this, 'importMessages']);
-            $this->comment('Ends ' . Carbon::now()->diffForHumans($start));
+            $this->comment('Ends '.Carbon::now()->diffForHumans($start));
         });
 
 
@@ -93,14 +93,14 @@ 
         $rootTimeZone   = new \DateTimeZone('UTC');
 
 
-        $messages = new ApiIterator(function ($page) use ($client, $room, $limit, &$lastMessageId) {
+        $messages = new ApiIterator(function($page) use ($client, $room, $limit, &$lastMessageId) {
             $query = ['limit' => $limit];
 
             if ($lastMessageId !== null) {
                 $query['beforeId'] = $lastMessageId;
             }
 
-            $result = $this->rescue(function () use ($room, $query, $client) {
+            $result = $this->rescue(function() use ($room, $query, $client) {
                 return $client->http->getMessages($room->id, $query)->wait();
             });
 
@@ -136,7 +136,7 @@ 
              */
             try {
 
-                $db->transaction(function () use ($message, $data, $db) {
+                $db->transaction(function() use ($message, $data, $db) {
                     $db->table('messages')->where('gitter_id', $data['gitter_id'])->delete();
                     $db->table('messages')->insert($data);
 
@@ -153,15 +153,15 @@ 
                 });
 
                 $this->info(
-                    '#' . $i . ' ' .
-                    $data['created_at'] . ' ' .
-                    mb_substr(str_replace("\n", '', $data['text']), 0, 32) . '... '
+                    '#'.$i.' '.
+                    $data['created_at'].' '.
+                    mb_substr(str_replace("\n", '', $data['text']), 0, 32).'... '
 
                 );
 
             } catch (\Throwable $e) {
 
-                $this->error($e->getMessage() . "\n" . $e->getTraceAsString());
+                $this->error($e->getMessage()."\n".$e->getTraceAsString());
 
             }
         }
@@ -176,7 +176,7 @@ 
      */
     private function getRoom(Client $client)
     {
-        return $this->rescue(function () use ($client) {
+        return $this->rescue(function() use ($client) {
             return $client->http->getRoomById($this->argument('room'))->wait();
         });
     }
@@ -192,8 +192,8 @@ 
     {
         return (new Evacuator($request))
             ->retry(Evacuator::INFINITY_RETRIES)
-            ->catch(function (\Throwable $e) {
-                $this->error($e->getMessage() . "\n" . '// retry again');
+            ->catch(function(\Throwable $e) {
+                $this->error($e->getMessage()."\n".'// retry again');
                 sleep(1);
             })
             ->invoke();
@@ -210,8 +210,8 @@ 
         $room = $this->getRoom($client);
 
 
-        $users = new ApiIterator(function ($page) use ($client, $room) {
-            return $this->rescue(function () use ($room, $page, $client) {
+        $users = new ApiIterator(function($page) use ($client, $room) {
+            return $this->rescue(function() use ($room, $page, $client) {
 
                 return $client->http->getRoomUsers($room->id, ['limit' => 30, 'skip' => 30 * $page])->wait();
 
@@ -221,7 +221,7 @@ 
 
         foreach ($users as $i => $user) {
             $user = UserMapper::fromGitterObject($user);
-            $this->comment('#' . $i . ' @' . $user->login);
+            $this->comment('#'.$i.' @'.$user->login);
         }
     }
 
@@ -241,8 +241,8 @@ 
             }
 
             if ($state->isIncrement() || $state->isTimeout() || $state->isSelf()) {
-                echo "\r" . '[' . $message->created_at . '] ' .
-                    $state->getTranslation($user->karma_text) . "\n";
+                echo "\r".'['.$message->created_at.'] '.
+                    $state->getTranslation($user->karma_text)."\n";
             }
         }
     }

app/Interfaces/Gitter/Client.php

Doc Comments +2 added lines, -2 removed lines

@@ -169,7 +169,7 @@ 
     }
 
     /**
-     * @param $route
+     * @param string $route
      * @param array $args
      * @param string $method
      * @return Stream
@@ -181,7 +181,7 @@ 
     }
 
     /**
-     * @param $route
+     * @param string $route
      * @param array $args
      * @param null $content
      * @param string $method

Spacing +3 added lines, -3 removed lines

@@ -41,14 +41,14 @@ 
     public static function make($token, $roomId)
     {
         $client = new Client($token);
-        App::singleton(Client::class, function () use ($client) {
+        App::singleton(Client::class, function() use ($client) {
             return $client;
         });
         App::alias(Client::class, 'gitter');
 
 
         $room = new Room($roomId);
-        App::singleton(Room::class, function () use ($room) {
+        App::singleton(Room::class, function() use ($room) {
             return $room;
         });
         App::alias(Room::class, 'room');
@@ -140,7 +140,7 @@ 
         return [
             'Content-Type'  => 'application/json',
             'Accept'        => 'application/json',
-            'Authorization' => 'Bearer ' . $this->token,
+            'Authorization' => 'Bearer '.$this->token,
         ];
     }
 

app/Domains/Message/Message.php

Spacing +1 added lines, -1 removed lines

@@ -48,7 +48,7 @@
 
         preg_match_all('/\w+/iu', $text, $words, PREG_PATTERN_ORDER);
 
-        return array_map(function ($word) {
+        return array_map(function($word) {
             return mb_strtolower($word);
         }, $words[0]);
     }

app/Domains/Analyser/MessageRelations.php

Spacing +1 added lines, -1 removed lines

@@ -88,7 +88,7 @@
 
         /** @var User $mention */
         foreach ($message->mentions as $mention) {
-            $question = $this->buffer->last(function ($i, Message $question) use ($mention) {
+            $question = $this->buffer->last(function($i, Message $question) use ($mention) {
                 if ($question->user) {
                     return $question->user->getId() === $mention->getId();
                 }

app/Core/Providers/GitterClientServiceProvider.php

Spacing +2 added lines, -2 removed lines

@@ -30,12 +30,12 @@
         $config = $this->app->make(Repository::class);
 
 
-        $this->app->singleton(Client::class, function (Container $app) use ($config) {
+        $this->app->singleton(Client::class, function(Container $app) use ($config) {
             return new Client($config->get('gitter.token'));
         });
 
 
-        $this->app->singleton('bot', function (Container $app) {
+        $this->app->singleton('bot', function(Container $app) {
             /** @var Client $client */
             $client = $app->make(Client::class);
             return $client->http->getCurrentUser()->wait();

app/Core/Mappers/UserMapper.php

Spacing +1 added lines, -1 removed lines

@@ -37,7 +37,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);
             });
         }