OmarElGabry / lumen-api-oauth

app/Http/routes.php

Indentation +1 added lines, -1 removed lines

@@ -13,7 +13,7 @@
 
 // Home Page
 $app->get('/', function () use ($app){
-	return $app->version();
+    return $app->version();
 });
 
 // Posts

Spacing +4 added lines, -4 removed lines

@@ -12,14 +12,14 @@
 */
 
 // Home Page
-$app->get('/', function () use ($app){
+$app->get('/', function() use ($app){
 	return $app->version();
 });
 
 // Posts
-$app->get('/posts','PostController@index');
-$app->post('/posts','PostController@store');
-$app->get('/posts/{post_id}','PostController@show');
+$app->get('/posts', 'PostController@index');
+$app->post('/posts', 'PostController@store');
+$app->get('/posts/{post_id}', 'PostController@show');
 $app->put('/posts/{post_id}', 'PostController@update');
 $app->patch('/posts/{post_id}', 'PostController@update');
 $app->delete('/posts/{post_id}', 'PostController@destroy');

app/User.php

Spacing +2 added lines, -2 removed lines

@@ -40,11 +40,11 @@
      * @return int|boolean
      * @see    https://github.com/lucadegasperi/oauth2-server-laravel/blob/master/docs/authorization-server/password.md
      */
-    public function verify($email, $password){
+    public function verify($email, $password) {
 
         $user = User::where('email', $email)->first();
 
-        if($user && Hash::check($password, $user->password)){
+        if ($user && Hash::check($password, $user->password)) {
             return $user->id;
         }
 

Unused Use Statements -1 removed lines

@@ -7,7 +7,6 @@
 use Illuminate\Database\Eloquent\Model;
 use Illuminate\Contracts\Auth\Authenticatable as AuthenticatableContract;
 use Illuminate\Contracts\Auth\Access\Authorizable as AuthorizableContract;
-
 use Illuminate\Support\Facades\Hash;
 
 class User extends Model implements AuthenticatableContract, AuthorizableContract

app/Http/Controllers/Controller.php

Doc Comments +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
      * Check if the user is authorized to perform a given action on a resource.
      *
      * @param  \Illuminate\Http\Request  $request
-     * @param  array $resource
+     * @param  string $resource
      * @param  mixed|array $arguments
      * @return boolean
      * @see    https://lumen.laravel.com/docs/authorization 

Indentation +11 added lines, -11 removed lines

@@ -16,9 +16,9 @@ 
      * @param  string $code
      * @return \Illuminate\Http\JsonResponse
      */
-	public function success($data, $code){
-		return response()->json(['data' => $data], $code);
-	}
+    public function success($data, $code){
+        return response()->json(['data' => $data], $code);
+    }
 
     /**
      * Return a JSON response for error.
@@ -27,9 +27,9 @@ 
      * @param  string $code
      * @return \Illuminate\Http\JsonResponse
      */
-	public function error($message, $code){
-		return response()->json(['message' => $message], $code);
-	}
+    public function error($message, $code){
+        return response()->json(['message' => $message], $code);
+    }
 
     /**
      * Check if the user is authorized to perform a given action on a resource.
@@ -42,14 +42,14 @@ 
      */
     protected function authorizeUser(Request $request, $resource, $arguments = []){
     	
-    	$user 	 = User::find($this->getUserId());
-    	$action	 = $this->getAction($request); 
+        $user 	 = User::find($this->getUserId());
+        $action	 = $this->getAction($request); 
 
         // The ability string must match the string defined in App\Providers\AuthServiceProvider\ability()
         $ability = "{$action}-{$resource}";
 
-    	// return $this->authorizeForUser($user, "{$action}-{$resource}", $data);
-    	return Gate::forUser($user)->allows($ability, $arguments);
+        // return $this->authorizeForUser($user, "{$action}-{$resource}", $data);
+        return Gate::forUser($user)->allows($ability, $arguments);
     }
 
     /**
@@ -72,7 +72,7 @@ 
      * @return boolean
      */
     protected function getUserId(){
-    	return \LucaDegasperi\OAuth2Server\Facades\Authorizer::getResourceOwnerId();
+        return \LucaDegasperi\OAuth2Server\Facades\Authorizer::getResourceOwnerId();
     }
 
     /**

Spacing +10 added lines, -10 removed lines

@@ -7,7 +7,7 @@ 
 use App\User;
 use Gate;
 
-class Controller extends BaseController{
+class Controller extends BaseController {
 
     /**
      * Return a JSON response for success.
@@ -16,7 +16,7 @@ 
      * @param  string $code
      * @return \Illuminate\Http\JsonResponse
      */
-	public function success($data, $code){
+	public function success($data, $code) {
 		return response()->json(['data' => $data], $code);
 	}
 
@@ -27,7 +27,7 @@ 
      * @param  string $code
      * @return \Illuminate\Http\JsonResponse
      */
-	public function error($message, $code){
+	public function error($message, $code) {
 		return response()->json(['message' => $message], $code);
 	}
 
@@ -40,10 +40,10 @@ 
      * @return boolean
      * @see    https://lumen.laravel.com/docs/authorization 
      */
-    protected function authorizeUser(Request $request, $resource, $arguments = []){
+    protected function authorizeUser(Request $request, $resource, $arguments = []) {
     	
-    	$user 	 = User::find($this->getUserId());
-    	$action	 = $this->getAction($request); 
+    	$user = User::find($this->getUserId());
+    	$action = $this->getAction($request); 
 
         // The ability string must match the string defined in App\Providers\AuthServiceProvider\ability()
         $ability = "{$action}-{$resource}";
@@ -61,7 +61,7 @@ 
      * @param  \Illuminate\Http\Request  $request
      * @return bool
      */
-    public function isAuthorized(Request $request){
+    public function isAuthorized(Request $request) {
         return false;
     }
 
@@ -71,7 +71,7 @@ 
      *
      * @return boolean
      */
-    protected function getUserId(){
+    protected function getUserId() {
     	return \LucaDegasperi\OAuth2Server\Facades\Authorizer::getResourceOwnerId();
     }
 
@@ -81,7 +81,7 @@ 
      * @param  \Illuminate\Http\Request  $request
      * @return string
      */
-    protected function getAction(Request $request){
+    protected function getAction(Request $request) {
         return explode('@', $request->route()[1]["uses"], 2)[1];
     }
 
@@ -91,7 +91,7 @@ 
      * @param  \Illuminate\Http\Request  $request
      * @return array
      */
-    protected function getArgs(Request $request){
+    protected function getArgs(Request $request) {
         return $request->route()[2];
     }
 }