OmarElGabry / lumen-api-oauth

app/Comment.php

Spacing +3 added lines, -3 removed lines

@@ -2,7 +2,7 @@ 
 
 use Illuminate\Database\Eloquent\Model;
 
-class Comment extends Model{
+class Comment extends Model {
 
     /**
      * The attributes that are mass assignable.
@@ -16,14 +16,14 @@ 
      *
      * @var array
      */
-	protected $hidden   = ['created_at', 'updated_at'];
+	protected $hidden = ['created_at', 'updated_at'];
 
     /**
      * Define an inverse one-to-many relationship with App\Post.
      *
      * @return \Illuminate\Database\Eloquent\Relations\BelongsTo
      */
-	public function post(){
+	public function post() {
 		return $this->belongsTo('App\Post');
 	}
 

app/Exceptions/Handler.php

Spacing +3 added lines, -3 removed lines

@@ -48,15 +48,15 @@
      */
     public function render($request, Exception $e)
     {
-        if(env('APP_DEBUG')){
+        if (env('APP_DEBUG')) {
             return parent::render($request, $e);
         }
 
-        if($e instanceof NotFoundHttpException){
+        if ($e instanceof NotFoundHttpException) {
             return response()->json(['message' => 'Bad Request', 'code' => 400], 400);
         }
 
-        if($e instanceof MethodNotAllowedHttpException){
+        if ($e instanceof MethodNotAllowedHttpException) {
             return response()->json(['message' => 'Not Found', 'code' => 404], 404);
         }
 

app/Post.php

Spacing +3 added lines, -3 removed lines

@@ -2,7 +2,7 @@ 
 
 use Illuminate\Database\Eloquent\Model;
 
-class Post extends Model{
+class Post extends Model {
 
     /**
      * The attributes that are mass assignable.
@@ -16,14 +16,14 @@ 
      *
      * @var array
      */
-	protected $hidden   = ['created_at', 'updated_at'];
+	protected $hidden = ['created_at', 'updated_at'];
 
     /**
      * Define a one-to-many relationship with App\Comment
      *
      * @return \Illuminate\Database\Eloquent\Relations\HasMany
      */
-	public function comments(){
+	public function comments() {
 		return $this->hasMany('App\Comment');
 	}
 

app/Providers/AuthServiceProvider.php

Spacing +6 added lines, -6 removed lines

@@ -56,7 +56,7 @@ 
      * @param  array  $arguments
      * @return boolean
      */
-    private function isOwner($arguments = []){
+    private function isOwner($arguments = []) {
 
         foreach ($arguments as $resource => $actions) {
             foreach ($actions as $action) {
@@ -67,9 +67,9 @@ 
                 //     }
                 // });
 
-                Gate::define($this->ability($action, $resource), function ($user, $arg) {
+                Gate::define($this->ability($action, $resource), function($user, $arg) {
                    
-                    if(is_null($arg))  { return false; }
+                    if (is_null($arg)) { return false; }
 
                     return $arg->user_id === $user->id || $user->is_admin;
                 });            
@@ -83,11 +83,11 @@ 
      * @param  array  $arguments
      * @return boolean
      */
-    private function isAdmin($arguments){
+    private function isAdmin($arguments) {
 
         foreach ($arguments as $resource => $actions) {
             foreach ($actions as $action) {
-                Gate::define($this->ability($action, $resource), function ($user) {
+                Gate::define($this->ability($action, $resource), function($user) {
                     return $user->is_admin;
                 });
             }
@@ -101,7 +101,7 @@ 
      * @param  string  $resource
      * @return string
      */
-    private function ability($action, $resource){
+    private function ability($action, $resource) {
         return "{$action}-{$resource}";
     }
     

app/Http/Middleware/Authorize.php

Spacing +2 added lines, -2 removed lines

@@ -11,11 +11,11 @@
      * @param  \Closure  $next
      * @return mixed
      */
-    public function handle($request, Closure $next, $controller){
+    public function handle($request, Closure $next, $controller) {
 
         $controller = new $controller();
 
-        if(!$controller->isAuthorized($request)){
+        if (!$controller->isAuthorized($request)) {
             return $controller->error("You aren't allowed to perform the requested action", 403);
         }
 

app/Http/Controllers/CommentController.php

Spacing +4 added lines, -4 removed lines

@@ -2,19 +2,19 @@
 
 use App\Comment;
 
-class CommentController extends Controller{
+class CommentController extends Controller {
 
-	public function index(){
+	public function index() {
 		
 		$comments = Comment::all();
 		return $this->success($comments, 200);
 	}
 
-	public function show($id){
+	public function show($id) {
 
 		$comment = Comment::find($id);
 
-		if(!$comment){
+		if (!$comment) {
 			return $this->error("The comment with {$id} doesn't exist", 404);
 		}
 

app/Http/Controllers/Controller.php

Spacing +10 added lines, -10 removed lines

@@ -5,7 +5,7 @@ 
 use App\User;
 use Gate;
 
-class Controller extends BaseController{
+class Controller extends BaseController {
 
     /**
      * Return a JSON response for success.
@@ -14,7 +14,7 @@ 
      * @param  string $code
      * @return \Illuminate\Http\JsonResponse
      */
-	public function success($data, $code){
+	public function success($data, $code) {
 		return response()->json(['data' => $data], $code);
 	}
 
@@ -25,7 +25,7 @@ 
      * @param  string $code
      * @return \Illuminate\Http\JsonResponse
      */
-	public function error($message, $code){
+	public function error($message, $code) {
 		return response()->json(['message' => $message], $code);
 	}
 
@@ -38,10 +38,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}";
@@ -59,7 +59,7 @@ 
      * @param  \Illuminate\Http\Request  $request
      * @return bool
      */
-    public function isAuthorized(Request $request){
+    public function isAuthorized(Request $request) {
         return false;
     }
 
@@ -74,7 +74,7 @@ 
      * 
      * @throws LucaDegasperi\OAuth2Server\Exceptions\NoActiveAccessTokenException
      */
-    protected function getUserId(){
+    protected function getUserId() {
     	return \LucaDegasperi\OAuth2Server\Facades\Authorizer::getResourceOwnerId();
     }
 
@@ -84,7 +84,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];
     }
 
@@ -94,7 +94,7 @@ 
      * @param  \Illuminate\Http\Request  $request
      * @return array
      */
-    protected function getArgs(Request $request){
+    protected function getArgs(Request $request) {
         return $request->route()[2];
     }
 }

app/Http/routes.php

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;
         }