taskforcedev / laravel-forum

src/Events/PostReply.php

Spacing +1 added lines, -1 removed lines

@@ -24,6 +24,6 @@
      */
     public function broadcastOn()
     {
-        return ['laravel-forum'];
+        return [ 'laravel-forum' ];
     }
 }

src/Helpers/Sanitizer.php

Spacing +2 added lines, -2 removed lines

@@ -19,9 +19,9 @@
 
         $script = $dom->getElementsByTagName('script');
 
-        $remove = [];
+        $remove = [ ];
         foreach ($script as $item) {
-            $remove[] = $item;
+            $remove[ ] = $item;
         }
 
         foreach ($remove as $item) {

src/Http/Controllers/ApiController.php

Spacing +3 added lines, -3 removed lines

@@ -82,7 +82,7 @@ 
         $post = ForumPost::create($data);
 
         event(new PostCreated($post, $user));
-        return redirect()->route('laravel-forum.view.post', [$forum_id , $post->id]);
+        return redirect()->route('laravel-forum.view.post', [ $forum_id, $post->id ]);
     }
 
     public function forumReplyStore()
@@ -103,13 +103,13 @@ 
         ];
 
         if (!ForumReply::valid($data)) {
-            return redirect()->route('laravel-forum.view.post', [$forum_id, $post_id]);
+            return redirect()->route('laravel-forum.view.post', [ $forum_id, $post_id ]);
         }
 
         $reply = ForumReply::create($data);
 
         event(new PostReply($reply, $user));
-        return redirect()->route('laravel-forum.view.post', [$forum_id, $post_id]);
+        return redirect()->route('laravel-forum.view.post', [ $forum_id, $post_id ]);
     }
 
     private function adminCheck()

Unused Use Statements -1 removed lines

@@ -6,7 +6,6 @@
 use \Schema;
 
 use Illuminate\Http\Request;
-
 use Taskforcedev\LaravelForum\Forum;
 use Taskforcedev\LaravelForum\ForumCategory;
 use Taskforcedev\LaravelForum\ForumPost;

src/Http/Controllers/ForumController.php

Spacing +3 added lines, -3 removed lines

@@ -25,7 +25,7 @@ 
         }
 
         $data = $this->buildData();
-        $data['forum'] = $forum;
+        $data[ 'forum' ] = $forum;
 
         return view('laravel-forum::forum.forum', $data);
     }
@@ -39,7 +39,7 @@ 
         }
 
         $data = $this->buildData();
-        $data['forum'] = $forum;
+        $data[ 'forum' ] = $forum;
 
         return view('laravel-forum::forum.createPost', $data);
     }
@@ -50,7 +50,7 @@ 
             $post = ForumPost::with('replies')->where('id', $post_id)->firstOrFail();
 
             $data = $this->buildData();
-            $data['post'] = $post;
+            $data[ 'post' ] = $post;
 
             return view('laravel-forum::forum.thread', $data);
         } catch (Exception $e) {

src/Events/PostCreated.php

Spacing +1 added lines, -1 removed lines

@@ -24,6 +24,6 @@
      */
     public function broadcastOn()
     {
-        return ['laravel-forum'];
+        return [ 'laravel-forum' ];
     }
 }

src/Http/Controllers/AdminController.php

Spacing +5 added lines, -5 removed lines

@@ -25,7 +25,7 @@ 
 
         $data = $this->buildData();
 
-        $data['counts'] = [
+        $data[ 'counts' ] = [
             'forum' => (new Forum)->getCount(),
             'category' => (new ForumCategory)->getCount(),
             'post' => (new ForumPost)->getCount(),
@@ -45,8 +45,8 @@ 
         }
 
         $data = $this->buildData();
-        $data['categories'] = ForumCategory::all();
-        $data['forums'] = Forum::orderBy('category_id')->get();
+        $data[ 'categories' ] = ForumCategory::all();
+        $data[ 'forums' ] = Forum::orderBy('category_id')->get();
         return view('laravel-forum::admin/forums', $data);
     }
 
@@ -60,7 +60,7 @@ 
         }
 
         $data = $this->buildData();
-        $data['categories'] = ForumCategory::all();
+        $data[ 'categories' ] = ForumCategory::all();
         return view('laravel-forum::admin/categories', $data);
     }
 
@@ -71,7 +71,7 @@ 
         }
 
         $data = $this->buildData();
-        $data['categories'] = ForumCategory::all();
+        $data[ 'categories' ] = ForumCategory::all();
         return view('laravel-forum::admin.forum.create', $data);
     }
 }

Unused Use Statements -1 removed lines

@@ -1,7 +1,6 @@
 <?php namespace Taskforcedev\LaravelForum\Http\Controllers;
 
 use \Schema;
-use Illuminate\Http\Request;
 use Taskforcedev\LaravelForum\Models\Forum;
 use Taskforcedev\LaravelForum\Models\ForumPost;
 use Taskforcedev\LaravelForum\Models\ForumReply;

src/Http/routes.php

Indentation +23 added lines, -23 removed lines

@@ -29,28 +29,28 @@
 
 Route::group(['namespace' => 'Taskforcedev\LaravelForum\Http\Controllers', 'middleware' => 'api'], function() {
 
-  Route::group(['prefix' => 'api'], function() {
-      /* Forum Category */
-      Route::post('forum_category',   [ 'as' => 'laravel-forum.api.store.category',  'uses' => 'ApiController@forumCategoryStore'  ]);
-      Route::delete('forum_category', [ 'as' => 'laravel-forum.api.delete.forum.category', 'uses' => 'ApiController@forumCategoryDelete' ]);
-
-      /* Forum */
-      Route::post('forum',   [ 'as' => 'laravel-forum.api.store.forum',  'uses' => 'ApiController@forumStore'  ]);
-      Route::delete('forum', [ 'as' => 'laravel-forum.api.delete.forum', 'uses' => 'ApiController@forumDelete' ]);
-
-      /* Forum Post */
-      Route::post('forum_post',              [ 'as' => 'laravel-forum.api.store.forum.post',  'uses' => 'ApiController@forumPostStore' ]);
-      Route::delete('forum/{id}/post/{pid}', [ 'as' => 'laravel-forum.api.delete.forum.post', 'uses' => 'ApiController@postDelete'     ]);
-      /* Forum Post: Reply */
-      Route::post('forum_reply', [ 'as' => 'laravel-forum.api.store.forum.reply', 'uses' => 'ApiController@forumReplyStore' ]);
-
-      /* Lock Forum Post */
-      Route::post('post/unlock/{id}', [ 'as' => 'laravel-forum.api.post.unlock', 'uses' => 'ApiController@unlockPost' ]);
-      Route::post('post/lock/{id}',   [ 'as' => 'laravel-forum.api.post.lock',   'uses' => 'ApiController@lockPost'   ]);
-
-      /* Sticky Forum Post */
-      Route::post('post/sticky/{id}',   [ 'as' => 'laravel-forum.api.post.sticky',   'uses' => 'ApiController@stickyPost'   ]);
-      Route::post('post/unsticky/{id}', [ 'as' => 'laravel-forum.api.post.unsticky', 'uses' => 'ApiController@unstickyPost' ]);
-  });
+    Route::group(['prefix' => 'api'], function() {
+        /* Forum Category */
+        Route::post('forum_category',   [ 'as' => 'laravel-forum.api.store.category',  'uses' => 'ApiController@forumCategoryStore'  ]);
+        Route::delete('forum_category', [ 'as' => 'laravel-forum.api.delete.forum.category', 'uses' => 'ApiController@forumCategoryDelete' ]);
+
+        /* Forum */
+        Route::post('forum',   [ 'as' => 'laravel-forum.api.store.forum',  'uses' => 'ApiController@forumStore'  ]);
+        Route::delete('forum', [ 'as' => 'laravel-forum.api.delete.forum', 'uses' => 'ApiController@forumDelete' ]);
+
+        /* Forum Post */
+        Route::post('forum_post',              [ 'as' => 'laravel-forum.api.store.forum.post',  'uses' => 'ApiController@forumPostStore' ]);
+        Route::delete('forum/{id}/post/{pid}', [ 'as' => 'laravel-forum.api.delete.forum.post', 'uses' => 'ApiController@postDelete'     ]);
+        /* Forum Post: Reply */
+        Route::post('forum_reply', [ 'as' => 'laravel-forum.api.store.forum.reply', 'uses' => 'ApiController@forumReplyStore' ]);
+
+        /* Lock Forum Post */
+        Route::post('post/unlock/{id}', [ 'as' => 'laravel-forum.api.post.unlock', 'uses' => 'ApiController@unlockPost' ]);
+        Route::post('post/lock/{id}',   [ 'as' => 'laravel-forum.api.post.lock',   'uses' => 'ApiController@lockPost'   ]);
+
+        /* Sticky Forum Post */
+        Route::post('post/sticky/{id}',   [ 'as' => 'laravel-forum.api.post.sticky',   'uses' => 'ApiController@stickyPost'   ]);
+        Route::post('post/unsticky/{id}', [ 'as' => 'laravel-forum.api.post.unsticky', 'uses' => 'ApiController@unstickyPost' ]);
+    });
 
 });

Spacing +20 added lines, -20 removed lines

@@ -1,25 +1,25 @@ 
 <?php
 
-Route::group(['namespace' => 'Taskforcedev\LaravelForum\Http\Controllers', 'middleware' => 'web'], function() {
+Route::group([ 'namespace' => 'Taskforcedev\LaravelForum\Http\Controllers', 'middleware' => 'web' ], function() {
     // Public Routes
-    Route::get('forum/{id}/post/{pid}', [ 'as' => 'laravel-forum.view.post',  'uses' => 'ForumController@viewPost'   ]);
+    Route::get('forum/{id}/post/{pid}', [ 'as' => 'laravel-forum.view.post', 'uses' => 'ForumController@viewPost' ]);
 
-    Route::get('forum/{id}/post',       [ 'as' => 'laravel-forum.write.post', 'uses' => 'ForumController@createPost' ]);
-    Route::get('forum/{id}',            [ 'as' => 'laravel-forum.view',       'uses' => 'ForumController@view'       ]);
-    Route::get('forum',                 [ 'as' => 'laravel-forum.index',      'uses' => 'ForumController@index'      ]);
+    Route::get('forum/{id}/post', [ 'as' => 'laravel-forum.write.post', 'uses' => 'ForumController@createPost' ]);
+    Route::get('forum/{id}', [ 'as' => 'laravel-forum.view', 'uses' => 'ForumController@view' ]);
+    Route::get('forum', [ 'as' => 'laravel-forum.index', 'uses' => 'ForumController@index' ]);
 
     // Admin Routes.
-    Route::group(['prefix' => 'admin'], function() {
-        Route::get('forums',            [ 'as' => 'laravel-forum.admin.index',       'uses' => 'AdminController@index'      ]);
+    Route::group([ 'prefix' => 'admin' ], function() {
+        Route::get('forums', [ 'as' => 'laravel-forum.admin.index', 'uses' => 'AdminController@index' ]);
 
-        Route::group(['prefix' => 'forums'], function() {
+        Route::group([ 'prefix' => 'forums' ], function() {
             /* Forums */
-            Route::get('forums',        [ 'as' => 'laravel-forum.admin.forums',      'uses' => 'AdminController@forums'    ]);
-            Route::get('forums/create', ['as' => 'laravel-forum.admin.forum.create', 'uses' => 'AdminController@forumForm' ]);
+            Route::get('forums', [ 'as' => 'laravel-forum.admin.forums', 'uses' => 'AdminController@forums' ]);
+            Route::get('forums/create', [ 'as' => 'laravel-forum.admin.forum.create', 'uses' => 'AdminController@forumForm' ]);
 
         /* Categories */
-            Route::get('categories/create', ['as' => 'laravel-forum.admin.category.create', 'uses' => 'AdminController@categoryForm' ]);
-            Route::get('categories',        ['as' => 'laravel-forum.admin.categories',   'uses' => 'AdminController@categories']);
+            Route::get('categories/create', [ 'as' => 'laravel-forum.admin.category.create', 'uses' => 'AdminController@categoryForm' ]);
+            Route::get('categories', [ 'as' => 'laravel-forum.admin.categories', 'uses' => 'AdminController@categories' ]);
         });
 
         
@@ -27,29 +27,29 @@ 
 
 });
 
-Route::group(['namespace' => 'Taskforcedev\LaravelForum\Http\Controllers', 'middleware' => ['web', 'api']], function() {
+Route::group([ 'namespace' => 'Taskforcedev\LaravelForum\Http\Controllers', 'middleware' => [ 'web', 'api' ] ], function() {
 
-  Route::group(['prefix' => 'api'], function() {
+  Route::group([ 'prefix' => 'api' ], function() {
       /* Forum Category */
-      Route::post('forum_category',   [ 'as' => 'laravel-forum.api.store.category',  'uses' => 'ApiController@forumCategoryStore'  ]);
+      Route::post('forum_category', [ 'as' => 'laravel-forum.api.store.category', 'uses' => 'ApiController@forumCategoryStore' ]);
       Route::delete('forum_category', [ 'as' => 'laravel-forum.api.delete.forum.category', 'uses' => 'ApiController@forumCategoryDelete' ]);
 
       /* Forum */
-      Route::post('forum',   [ 'as' => 'laravel-forum.api.store.forum',  'uses' => 'ApiController@forumStore'  ]);
+      Route::post('forum', [ 'as' => 'laravel-forum.api.store.forum', 'uses' => 'ApiController@forumStore' ]);
       Route::delete('forum', [ 'as' => 'laravel-forum.api.delete.forum', 'uses' => 'ApiController@forumDelete' ]);
 
       /* Forum Post */
-      Route::post('forum_post',              [ 'as' => 'laravel-forum.api.store.forum.post',  'uses' => 'ApiController@forumPostStore' ]);
-      Route::delete('forum/{id}/post/{pid}', [ 'as' => 'laravel-forum.api.delete.forum.post', 'uses' => 'ApiController@postDelete'     ]);
+      Route::post('forum_post', [ 'as' => 'laravel-forum.api.store.forum.post', 'uses' => 'ApiController@forumPostStore' ]);
+      Route::delete('forum/{id}/post/{pid}', [ 'as' => 'laravel-forum.api.delete.forum.post', 'uses' => 'ApiController@postDelete' ]);
       /* Forum Post: Reply */
       Route::post('forum_reply', [ 'as' => 'laravel-forum.api.store.forum.reply', 'uses' => 'ApiController@forumReplyStore' ]);
 
       /* Lock Forum Post */
       Route::post('post/unlock/{id}', [ 'as' => 'laravel-forum.api.post.unlock', 'uses' => 'ApiController@unlockPost' ]);
-      Route::post('post/lock/{id}',   [ 'as' => 'laravel-forum.api.post.lock',   'uses' => 'ApiController@lockPost'   ]);
+      Route::post('post/lock/{id}', [ 'as' => 'laravel-forum.api.post.lock', 'uses' => 'ApiController@lockPost' ]);
 
       /* Sticky Forum Post */
-      Route::post('post/sticky/{id}',   [ 'as' => 'laravel-forum.api.post.sticky',   'uses' => 'ApiController@stickyPost'   ]);
+      Route::post('post/sticky/{id}', [ 'as' => 'laravel-forum.api.post.sticky', 'uses' => 'ApiController@stickyPost' ]);
       Route::post('post/unsticky/{id}', [ 'as' => 'laravel-forum.api.post.unsticky', 'uses' => 'ApiController@unstickyPost' ]);
   });
 

src/Models/ForumReply.php

Doc Comments +1 added lines, -1 removed lines

@@ -22,7 +22,7 @@
 
     /**
      * Eloquent Relation.
-     * @return mixed
+     * @return \Illuminate\Database\Eloquent\Relations\BelongsTo
      */
     public function author()
     {

Spacing +3 added lines, -3 removed lines

@@ -17,8 +17,8 @@ 
 {
     protected $table = 'forum_post_replies';
 
-    protected $hidden = [];
-    protected $fillable = ['body', 'post_id', 'author_id'];
+    protected $hidden = [ ];
+    protected $fillable = [ 'body', 'post_id', 'author_id' ];
 
     /**
      * Eloquent Relation.
@@ -59,6 +59,6 @@ 
     public static function getReplyCount()
     {
         $results = DB::select('select COUNT(*) as count from forum_post_replies');
-        return $results[0]->count;
+        return $results[ 0 ]->count;
     }
 }

database/migrations/2016_08_24_133554_posts_add_sticky_and_locked.php

Spacing +3 added lines, -3 removed lines

@@ -8,7 +8,7 @@ 
     public function up()
     {
         if (!Schema::hasColumn('forum_posts', 'sticky') && !Schema::hasColumn('forum_posts', 'locked')) {
-            Schema::table('forum_posts', function ($table) {
+            Schema::table('forum_posts', function($table) {
                 $table->boolean('sticky')->default(0);
                 $table->boolean('locked')->default(0);
             });
@@ -17,8 +17,8 @@ 
 
     public function down()
     {
-        Schema::table('forum_posts', function ($table) {
-            $table->dropColumn(['sticky', 'locked']);
+        Schema::table('forum_posts', function($table) {
+            $table->dropColumn([ 'sticky', 'locked' ]);
         });
     }
 }