Add button to ban user and delete its threads

driesvints · driesvints · commit c9b1f32c5c5a · 2014-11-03T22:40:14.000+01:00
diff --git a/app/Lio/Forum/Threads/ThreadRepository.php b/app/Lio/Forum/Threads/ThreadRepository.php
@@ -69,4 +69,13 @@ public function getBySlug($slug)
     {
         return $this->model->with('author')->where('slug', '=', $slug)->first();
     }
+
+    /**
+     * @param int $userId
+     * @return mixed
+     */
+    public function deleteByAuthorId($userId)
+    {
+        return $this->model->where('author_id', $userId)->delete();
+    }
 }
diff --git a/app/controllers/Admin/UsersController.php b/app/controllers/Admin/UsersController.php
@@ -1,19 +1,26 @@
 <?php namespace Admin;
 
 use BaseController;
+use Input;
 use Lio\Accounts\UserRepository;
 use Lio\Accounts\RoleRepository;
-use Input;
+use Lio\Forum\Threads\ThreadRepository;
 
 class UsersController extends BaseController
 {
     private $users;
     private $roles;
 
-    public function __construct(UserRepository $users, RoleRepository $roles)
+    /**
+     * @var \Lio\Forum\Threads\ThreadRepository
+     */
+    private $threads;
+
+    public function __construct(UserRepository $users, RoleRepository $roles, ThreadRepository $threads)
     {
         $this->users = $users;
         $this->roles = $roles;
+        $this->threads = $threads;
     }
 
     public function getIndex()
@@ -49,4 +56,18 @@ public function postEdit($userId)
 
         return $this->redirectAction('Admin\UsersController@getIndex', ['success' => 'The user has been saved.']);
     }
+
+    public function putBanAndDeleteThreads($userId)
+    {
+        // Ban the user
+        $user = $this->users->requireById($userId);
+        $user->is_banned = 1;
+
+        $this->users->save($user);
+
+        // Remove all threads by the user
+        $this->threads->deleteByAuthorId($userId);
+
+        return $this->redirectAction('Admin\UsersController@getIndex', ['success' => 'The user has been banned and its threads have been removed.']);
+    }
 }
diff --git a/app/routes.php b/app/routes.php
@@ -110,5 +110,6 @@
         Route::get('users', 'UsersController@getIndex');
         Route::get('edit/{user}', 'UsersController@getEdit');
         Route::post('edit/{user}', 'UsersController@postEdit');
+        Route::put('ban-and-delete-threads/{user}', 'UsersController@putBanAndDeleteThreads');
     });
 });
diff --git a/app/views/admin/users/edit.blade.php b/app/views/admin/users/edit.blade.php
@@ -50,7 +50,16 @@
                     {{ Form::button('Save', ['type' => 'submit']) }}
                 </div>
             </div>
-
         {{ Form::close() }}
+
+        @if (! $user->is_banned)
+            {{ Form::open(['action' => ['Admin\UsersController@putBanAndDeleteThreads', $user->id], 'method' => 'put']) }}
+                <div class="row">
+                    <div class="large-12 columns">
+                        {{ Form::button('Ban and delete threads', ['type' => 'submit']) }}
+                    </div>
+                </div>
+            {{ Form::close() }}
+        @endif
     </div>
 </div>

Original file line number	Diff line number	Diff line change
`@@ -69,4 +69,13 @@ public function getBySlug($slug)`
`69`	`69`	`{`
`70`	`70`	`return $this->model->with('author')->where('slug', '=', $slug)->first();`
`71`	`71`	`}`
	`72`	`+`
	`73`	`+ /**`
	`74`	`+ * @param int $userId`
	`75`	`+ * @return mixed`
	`76`	`+ */`
	`77`	`+ public function deleteByAuthorId($userId)`
	`78`	`+ {`
	`79`	`+ return $this->model->where('author_id', $userId)->delete();`
	`80`	`+ }`
`72`	`81`	`}`