Passed
Push — master ( 7d362e...62822e )
by Omar El
02:41
created
app/Http/Controllers/PostController.php 1 patch
Spacing   +14 added lines, -14 removed lines patch added patch discarded remove patch
@@ -6,21 +6,21 @@  discard block
 block discarded – undo
6 6
 
7 7
 use Illuminate\Http\Request;
8 8
 
9
-class PostController extends Controller{
9
+class PostController extends Controller {
10 10
 
11
-	public function __construct(){
11
+	public function __construct() {
12 12
 
13 13
 		$this->middleware('oauth', ['except' => ['index', 'show']]);
14
-		$this->middleware('authorize:' . __CLASS__, ['except' => ['index', 'show', 'store']]);
14
+		$this->middleware('authorize:'.__CLASS__, ['except' => ['index', 'show', 'store']]);
15 15
 	}
16 16
 
17
-	public function index(){
17
+	public function index() {
18 18
 
19 19
 		$posts = Post::all();
20 20
 		return $this->success($posts, 200);
21 21
 	}
22 22
 
23
-	public function store(Request $request){
23
+	public function store(Request $request) {
24 24
 
25 25
 		$this->validateRequest($request);
26 26
 
@@ -33,28 +33,28 @@  discard block
 block discarded – undo
33 33
 		return $this->success("The post with with id {$post->id} has been created", 201);
34 34
 	}
35 35
 
36
-	public function show($id){
36
+	public function show($id) {
37 37
 
38 38
 		$post = Post::find($id);
39 39
 
40
-		if(!$post){
40
+		if (!$post) {
41 41
 			return $this->error("The post with {$id} doesn't exist", 404);
42 42
 		}
43 43
 
44 44
 		return $this->success($post, 200);
45 45
 	}
46 46
 
47
-	public function update(Request $request, $id){
47
+	public function update(Request $request, $id) {
48 48
 
49 49
 		$post = Post::find($id);
50 50
 
51
-		if(!$post){
51
+		if (!$post) {
52 52
 			return $this->error("The post with {$id} doesn't exist", 404);
53 53
 		}
54 54
 
55 55
 		$this->validateRequest($request);
56 56
 
57
-		$post->title 		= $request->get('title');
57
+		$post->title = $request->get('title');
58 58
 		$post->content 		= $request->get('content');
59 59
 		$post->user_id 		= $this->getUserId();
60 60
 
@@ -63,11 +63,11 @@  discard block
 block discarded – undo
63 63
 		return $this->success("The post with with id {$post->id} has been updated", 200);
64 64
 	}
65 65
 
66
-	public function destroy($id){
66
+	public function destroy($id) {
67 67
 
68 68
 		$post = Post::find($id);
69 69
 
70
-		if(!$post){
70
+		if (!$post) {
71 71
 			return $this->error("The post with {$id} doesn't exist", 404);
72 72
 		}
73 73
 
@@ -79,7 +79,7 @@  discard block
 block discarded – undo
79 79
 		return $this->success("The post with with id {$id} has been deleted along with it's comments", 200);
80 80
 	}
81 81
 
82
-	public function validateRequest(Request $request){
82
+	public function validateRequest(Request $request) {
83 83
 
84 84
 		$rules = [
85 85
 			'title' => 'required', 
@@ -89,7 +89,7 @@  discard block
 block discarded – undo
89 89
 		$this->validate($request, $rules);
90 90
 	}
91 91
 
92
-	public function isAuthorized(Request $request){
92
+	public function isAuthorized(Request $request) {
93 93
 
94 94
 		$resource = "posts";
95 95
 		$post     = Post::find($this->getArgs($request)["post_id"]);
Please login to merge, or discard this patch.
app/Http/Controllers/Controller.php 1 patch
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -7,7 +7,7 @@  discard block
 block discarded – undo
7 7
 use App\User;
8 8
 use Gate;
9 9
 
10
-class Controller extends BaseController{
10
+class Controller extends BaseController {
11 11
 
12 12
     /**
13 13
      * Return a JSON response for success.
@@ -16,7 +16,7 @@  discard block
 block discarded – undo
16 16
      * @param  string $code
17 17
      * @return \Illuminate\Http\JsonResponse
18 18
      */
19
-	public function success($data, $code){
19
+	public function success($data, $code) {
20 20
 		return response()->json(['data' => $data], $code);
21 21
 	}
22 22
 
@@ -27,7 +27,7 @@  discard block
 block discarded – undo
27 27
      * @param  string $code
28 28
      * @return \Illuminate\Http\JsonResponse
29 29
      */
30
-	public function error($message, $code){
30
+	public function error($message, $code) {
31 31
 		return response()->json(['message' => $message], $code);
32 32
 	}
33 33
 
@@ -40,10 +40,10 @@  discard block
 block discarded – undo
40 40
      * @return boolean
41 41
      * @see    https://lumen.laravel.com/docs/authorization 
42 42
      */
43
-    protected function authorizeUser(Request $request, $resource, $arguments = []){
43
+    protected function authorizeUser(Request $request, $resource, $arguments = []) {
44 44
     	
45
-    	$user 	 = User::find($this->getUserId());
46
-    	$action	 = $this->getAction($request); 
45
+    	$user = User::find($this->getUserId());
46
+    	$action = $this->getAction($request); 
47 47
 
48 48
         // The ability string must match the string defined in App\Providers\AuthServiceProvider\ability()
49 49
         $ability = "{$action}-{$resource}";
@@ -61,7 +61,7 @@  discard block
 block discarded – undo
61 61
      * @param  \Illuminate\Http\Request  $request
62 62
      * @return bool
63 63
      */
64
-    public function isAuthorized(Request $request){
64
+    public function isAuthorized(Request $request) {
65 65
         return false;
66 66
     }
67 67
 
@@ -71,7 +71,7 @@  discard block
 block discarded – undo
71 71
      *
72 72
      * @return boolean
73 73
      */
74
-    protected function getUserId(){
74
+    protected function getUserId() {
75 75
     	return \LucaDegasperi\OAuth2Server\Facades\Authorizer::getResourceOwnerId();
76 76
     }
77 77
 
@@ -81,7 +81,7 @@  discard block
 block discarded – undo
81 81
      * @param  \Illuminate\Http\Request  $request
82 82
      * @return string
83 83
      */
84
-    protected function getAction(Request $request){
84
+    protected function getAction(Request $request) {
85 85
         return explode('@', $request->route()[1]["uses"], 2)[1];
86 86
     }
87 87
 
@@ -91,7 +91,7 @@  discard block
 block discarded – undo
91 91
      * @param  \Illuminate\Http\Request  $request
92 92
      * @return array
93 93
      */
94
-    protected function getArgs(Request $request){
94
+    protected function getArgs(Request $request) {
95 95
         return $request->route()[2];
96 96
     }
97 97
 }
Please login to merge, or discard this patch.