LaravelRUS / GitterBot

app/Domains/Bot/Middlewares/KarmaRenderMiddleware.php

Doc Comments +1 added lines, -1 removed lines

@@ -14,7 +14,7 @@
 {
     /**
      * @param Message $message
-     * @return mixed
+     * @return null|Message
      */
     public function handle(Message $message)
     {

Spacing +1 added lines, -1 removed lines

@@ -57,7 +57,7 @@
     {
         $achievements = [];
         foreach ($message->user->achievements as $achieve) {
-            $achievements[] = '"' . $achieve->title . '"';
+            $achievements[] = '"'.$achieve->title.'"';
         }
 
         if (count($achievements)) {

app/Domains/User.php

Doc Comments +1 added lines, -1 removed lines

@@ -118,7 +118,7 @@
     }
 
     /**
-     * @param $roomId
+     * @param string $roomId
      * @return Carbon
      */
     public function getLastKarmaTimeForRoom($roomId)

Spacing +1 added lines, -1 removed lines

@@ -92,7 +92,7 @@
     {
         $karma = $this->karma->count();
 
-        return ($karma > 0 ? '+' : '') . $karma;
+        return ($karma > 0 ? '+' : '').$karma;
     }
 
     /**

app/Core/Providers/RouteServiceProvider.php

Spacing +1 added lines, -1 removed lines

@@ -37,7 +37,7 @@
      */
     public function map(Router $router)
     {
-        $router->group(['namespace' => $this->namespace], function ($router) {
+        $router->group(['namespace' => $this->namespace], function($router) {
             require app_path('Http/routes.php');
         });
     }

app/Domains/Karma.php

Spacing +1 added lines, -1 removed lines

@@ -58,7 +58,7 @@
     {
         parent::boot();
 
-        static::creating(function (Karma $karma) {
+        static::creating(function(Karma $karma) {
             if (!$karma->created_at) {
                 $karma->created_at = $karma->freshTimestamp();
             }

app/Domains/Achieve.php

Spacing +1 added lines, -1 removed lines

@@ -69,7 +69,7 @@
     {
         parent::boot();
 
-        static::creating(function (Achieve $achieve) {
+        static::creating(function(Achieve $achieve) {
             if (!$achieve->created_at) {
                 $achieve->created_at = $achieve->freshTimestamp();
             }

app/Interfaces/Gitter/Http/Stream.php

Spacing +5 added lines, -5 removed lines

@@ -85,7 +85,7 @@ 
 
         $this->headers['Connection'] = 'Keep-Alive';
 
-        $this->buffer->subscribe(function ($message) {
+        $this->buffer->subscribe(function($message) {
             $message = trim($message);
 
             if ($message) {
@@ -117,20 +117,20 @@ 
             ->getHttpClient()
             ->request($this->method, $this->url, $this->headers);
 
-        $request->on('response', function (Response $response) {
-            $response->on('data', function ($data, Response $response) {
+        $request->on('response', function(Response $response) {
+            $response->on('data', function($data, Response $response) {
                 $data = (string)$data;
                 $this->events->fire(static::EVENT_CHUNK, [$this, $data, $response]);
                 $this->buffer->add($data);
             });
         });
 
-        $request->on('end', function () {
+        $request->on('end', function() {
             $this->buffer->clear();
             $this->events->fire(static::EVENT_END, [$this]);
         });
 
-        $request->on('error', function ($exception) {
+        $request->on('error', function($exception) {
             $this->events->fire(static::EVENT_ERROR, [$this, $exception]);
         });
 

app/Interfaces/Http/Controllers/Api/AchievementsController.php

Spacing +3 added lines, -3 removed lines

@@ -27,20 +27,20 @@
      */
     public function index()
     {
-        return \Cache::remember('achievements', 10, function () {
+        return \Cache::remember('achievements', 10, function() {
             $achieveStorage = [];
 
             (new Achieve())
                 ->selectRaw('name, count(user_id) as count')
                 ->groupBy('name')
                 ->get()
-                ->each(function ($item) use (&$achieveStorage) {
+                ->each(function($item) use (&$achieveStorage) {
                     $achieveStorage[$item->name] = $item->count;
                 });
 
             return (new AchieveSubscriber())
                 ->toCollection()
-                ->each(function (AbstractAchieve $achieve) use ($achieveStorage) {
+                ->each(function(AbstractAchieve $achieve) use ($achieveStorage) {
                     $achieve->users = $achieveStorage[$achieve->name] ?? 0;
                 })
                 ->toArray();

app/Interfaces/Http/Controllers/Api/UsersController.php

Spacing +8 added lines, -8 removed lines

@@ -26,7 +26,7 @@ 
      */
     public function index()
     {
-        return \Cache::remember('users', 1, function () {
+        return \Cache::remember('users', 1, function() {
             $karmaStorage = [];
             $thanksStorage = [];
 
@@ -34,7 +34,7 @@ 
                 ->selectRaw('user_target_id, count(user_id) as count')
                 ->groupBy('user_target_id')
                 ->get()
-                ->each(function ($item) use (&$karmaStorage) {
+                ->each(function($item) use (&$karmaStorage) {
                     $karmaStorage[$item->user_target_id] = $item->count;
                 });
 
@@ -42,14 +42,14 @@ 
                 ->selectRaw('user_id, count(user_target_id) as count')
                 ->groupBy('user_id')
                 ->get()
-                ->each(function ($item) use (&$thanksStorage) {
+                ->each(function($item) use (&$thanksStorage) {
                     $thanksStorage[$item->user_id] = $item->count;
                 });
 
 
             return (new User())
                 ->get(['id', 'login', 'name', 'gitter_id', 'avatar', 'url'])
-                ->each(function (User $user) use ($karmaStorage, $thanksStorage) {
+                ->each(function(User $user) use ($karmaStorage, $thanksStorage) {
                     $user->karma_count = $karmaStorage[$user->id] ?? 0;
                     $user->thanks_count = $thanksStorage[$user->id] ?? 0;
                 });
@@ -61,7 +61,7 @@ 
      */
     public function getUsersTop()
     {
-        return \Cache::remember('top.karma', 1, function () {
+        return \Cache::remember('top.karma', 1, function() {
             $karmaStorage = [];
 
             $karma = (new Karma())
@@ -70,7 +70,7 @@ 
                 ->orderBy('count', 'desc')
                 ->take(10)
                 ->get()
-                ->each(function ($item) use (&$karmaStorage) {
+                ->each(function($item) use (&$karmaStorage) {
                     $karmaStorage[$item->user_target_id] = $item->count;
                 });
 
@@ -78,7 +78,7 @@ 
             return (new User())
                 ->whereIn('id', $karma->pluck('user_target_id'))
                 ->get(['id', 'login', 'name', 'gitter_id', 'avatar', 'url'])
-                ->each(function (User $user) use ($karmaStorage) {
+                ->each(function(User $user) use ($karmaStorage) {
                     $user->karma_count = $karmaStorage[$user->id] ?? 0;
                 });
         });
@@ -89,7 +89,7 @@ 
      */
     public function getUser($gitterId)
     {
-        $formatRelations = function (HasMany $query) {
+        $formatRelations = function(HasMany $query) {
             return $query->orderBy('created_at', 'desc');
         };
 

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