SherifElfadaly / Laravel-Api-Skeleton

src/Modules/Core/Utl/ApiConsumer.php

Doc Comments -16 removed lines

@@ -28,10 +28,6 @@ 
     }
 
     /**
-     * @param  string $uri
-     * @param  array  $data
-     * @param  array  $headers
-     * @param  string $content
      * @return \Illuminate\Http\Response
      */
     public function get()
@@ -40,10 +36,6 @@ 
     }
 
     /**
-     * @param  string $uri
-     * @param  array  $data
-     * @param  array  $headers
-     * @param  string $content
      * @return \Illuminate\Http\Response
      */
     public function post()
@@ -52,10 +44,6 @@ 
     }
 
     /**
-     * @param  string $uri
-     * @param  array  $data
-     * @param  array  $headers
-     * @param  string $content
      * @return \Illuminate\Http\Response
      */
     public function put()
@@ -64,10 +52,6 @@ 
     }
 
     /**
-     * @param  string $uri
-     * @param  array  $data
-     * @param  array  $headers
-     * @param  string $content
      * @return \Illuminate\Http\Response
      */
     public function delete()

Indentation +214 added lines, -214 removed lines

@@ -7,218 +7,218 @@
 class ApiConsumer
 {
 
-    private $app;
-
-    private $router;
-
-    private $request;
-
-    private $disableMiddleware = false;
-
-    /**
-     * @param \Illuminate\Foundation\Application $app
-     * @param \Illuminate\Http\Request $request,
-     * @param \Illuminate\Routing\Router $router
-     */
-    public function __construct(Application $app, Request $request, LaravelRouter $router)
-    {
-        $this->app = $app;
-        $this->request = $request;
-        $this->router = $router;
-    }
-
-    /**
-     * @param  string $uri
-     * @param  array  $data
-     * @param  array  $headers
-     * @param  string $content
-     * @return \Illuminate\Http\Response
-     */
-    public function get()
-    {
-        return $this->quickCall('GET', func_get_args());
-    }
-
-    /**
-     * @param  string $uri
-     * @param  array  $data
-     * @param  array  $headers
-     * @param  string $content
-     * @return \Illuminate\Http\Response
-     */
-    public function post()
-    {
-        return $this->quickCall('POST', func_get_args());
-    }
-
-    /**
-     * @param  string $uri
-     * @param  array  $data
-     * @param  array  $headers
-     * @param  string $content
-     * @return \Illuminate\Http\Response
-     */
-    public function put()
-    {
-        return $this->quickCall('PUT', func_get_args());
-    }
-
-    /**
-     * @param  string $uri
-     * @param  array  $data
-     * @param  array  $headers
-     * @param  string $content
-     * @return \Illuminate\Http\Response
-     */
-    public function delete()
-    {
-        return $this->quickCall('DELETE', func_get_args());
-    }
-
-    /**
-     * @param  array $requests An array of requests
-     * @return array
-     */
-    public function batchRequest(array $requests)
-    {
-        foreach ($requests as $i => $request) {
-            $requests[$i] = call_user_func_array([$this, 'singleRequest'], $request);
-        }
-
-        return $requests;
-    }
-
-    /**
-     * @param  string $method
-     * @param  array  $args
-     * @return \Illuminate\Http\Response
-     */
-    public function quickCall($method, array $args)
-    {
-        array_unshift($args, $method);
-        return call_user_func_array([$this, "singleRequest"], $args);
-    }
-
-    /**
-     * @param  string $method
-     * @param  string $uri
-     * @param  array  $data
-     * @param  array  $headers
-     * @param  string $content
-     * @return \Illuminate\Http\Response
-     */
-    public function singleRequest($method, $uri, array $data = [], array $headers = [], $content = null)
-    {
-        // Save the current request so we can reset the router back to it
-        // after we've completed our internal request.
-        $currentRequest = $this->request->instance()->duplicate();
-
-        $headers = $this->overrideHeaders($currentRequest->server->getHeaders(), $headers);
-
-        if ($this->disableMiddleware) {
-            $this->app->instance('middleware.disable', true);
-        }
-
-        $response = $this->request($method, $uri, $data, $headers, $content);
-
-        if ($this->disableMiddleware) {
-            $this->app->instance('middleware.disable', false);
-        }
-
-        // Once the request has completed we reset the currentRequest of the router
-        // to match the original request.
-        $this->request->instance()->initialize(
-            $currentRequest->query->all(),
-            $currentRequest->request->all(),
-            $currentRequest->attributes->all(),
-            $currentRequest->cookies->all(),
-            $currentRequest->files->all(),
-            $currentRequest->server->all(),
-            $currentRequest->content
-        );
-
-        return $response;
-    }
-
-    private function overrideHeaders(array $default, array $headers)
-    {
-        $headers = $this->transformHeadersToUppercaseUnderscoreType($headers);
-        return array_merge($default, $headers);
-    }
-
-    public function enableMiddleware()
-    {
-        $this->disableMiddleware = false;
-    }
-
-    public function disableMiddleware()
-    {
-        $this->disableMiddleware = true;
-    }
-
-    /**
-     * @param  string $method
-     * @param  string $uri
-     * @param  array  $data
-     * @param  array  $headers
-     * @param  string $content
-     * @return \Illuminate\Http\Response
-     */
-    private function request($method, $uri, array $data = [], array $headers = [], $content = null)
-    {
-        // Create a new request object for the internal request
-        $request = $this->createRequest($method, $uri, $data, $headers, $content);
-
-        // Handle the request in the kernel and prepare a response
-        $response = $this->router->prepareResponse($request, $this->app->handle($request));
-
-        return $response;
-    }
-
-    /**
-     * @param  string $method
-     * @param  string $uri
-     * @param  array  $data
-     * @param  array  $headers
-     * @param  string $content
-     * @return \Illuminate\Http\Request
-     */
-    private function createRequest($method, $uri, array $data = [], array $headers = [], $content = null)
-    {
-        $server = $this->transformHeadersToServerVariables($headers);
-
-        return $this->request->create($uri, $method, $data, [], [], $server, $content);
-    }
-
-    private function transformHeadersToUppercaseUnderscoreType($headers)
-    {
-        $transformed = [];
-
-        foreach ($headers as $headerType => $headerValue) {
-            $headerType = strtoupper(str_replace('-', '_', $headerType));
-
-            $transformed[$headerType] = $headerValue;
-        }
-
-        return $transformed;
-    }
-
-    /**
-     * https://github.com/symfony/symfony/issues/5074
-     *
-     * @param  array $headers
-     * @return array
-     */
-    private function transformHeadersToServerVariables($headers)
-    {
-        $server = [];
-
-        foreach ($headers as $headerType => $headerValue) {
-            $headerType = 'HTTP_' . $headerType;
-
-            $server[$headerType] = $headerValue;
-        }
-
-        return $server;
-    }
+	private $app;
+
+	private $router;
+
+	private $request;
+
+	private $disableMiddleware = false;
+
+	/**
+	 * @param \Illuminate\Foundation\Application $app
+	 * @param \Illuminate\Http\Request $request,
+	 * @param \Illuminate\Routing\Router $router
+	 */
+	public function __construct(Application $app, Request $request, LaravelRouter $router)
+	{
+		$this->app = $app;
+		$this->request = $request;
+		$this->router = $router;
+	}
+
+	/**
+	 * @param  string $uri
+	 * @param  array  $data
+	 * @param  array  $headers
+	 * @param  string $content
+	 * @return \Illuminate\Http\Response
+	 */
+	public function get()
+	{
+		return $this->quickCall('GET', func_get_args());
+	}
+
+	/**
+	 * @param  string $uri
+	 * @param  array  $data
+	 * @param  array  $headers
+	 * @param  string $content
+	 * @return \Illuminate\Http\Response
+	 */
+	public function post()
+	{
+		return $this->quickCall('POST', func_get_args());
+	}
+
+	/**
+	 * @param  string $uri
+	 * @param  array  $data
+	 * @param  array  $headers
+	 * @param  string $content
+	 * @return \Illuminate\Http\Response
+	 */
+	public function put()
+	{
+		return $this->quickCall('PUT', func_get_args());
+	}
+
+	/**
+	 * @param  string $uri
+	 * @param  array  $data
+	 * @param  array  $headers
+	 * @param  string $content
+	 * @return \Illuminate\Http\Response
+	 */
+	public function delete()
+	{
+		return $this->quickCall('DELETE', func_get_args());
+	}
+
+	/**
+	 * @param  array $requests An array of requests
+	 * @return array
+	 */
+	public function batchRequest(array $requests)
+	{
+		foreach ($requests as $i => $request) {
+			$requests[$i] = call_user_func_array([$this, 'singleRequest'], $request);
+		}
+
+		return $requests;
+	}
+
+	/**
+	 * @param  string $method
+	 * @param  array  $args
+	 * @return \Illuminate\Http\Response
+	 */
+	public function quickCall($method, array $args)
+	{
+		array_unshift($args, $method);
+		return call_user_func_array([$this, "singleRequest"], $args);
+	}
+
+	/**
+	 * @param  string $method
+	 * @param  string $uri
+	 * @param  array  $data
+	 * @param  array  $headers
+	 * @param  string $content
+	 * @return \Illuminate\Http\Response
+	 */
+	public function singleRequest($method, $uri, array $data = [], array $headers = [], $content = null)
+	{
+		// Save the current request so we can reset the router back to it
+		// after we've completed our internal request.
+		$currentRequest = $this->request->instance()->duplicate();
+
+		$headers = $this->overrideHeaders($currentRequest->server->getHeaders(), $headers);
+
+		if ($this->disableMiddleware) {
+			$this->app->instance('middleware.disable', true);
+		}
+
+		$response = $this->request($method, $uri, $data, $headers, $content);
+
+		if ($this->disableMiddleware) {
+			$this->app->instance('middleware.disable', false);
+		}
+
+		// Once the request has completed we reset the currentRequest of the router
+		// to match the original request.
+		$this->request->instance()->initialize(
+			$currentRequest->query->all(),
+			$currentRequest->request->all(),
+			$currentRequest->attributes->all(),
+			$currentRequest->cookies->all(),
+			$currentRequest->files->all(),
+			$currentRequest->server->all(),
+			$currentRequest->content
+		);
+
+		return $response;
+	}
+
+	private function overrideHeaders(array $default, array $headers)
+	{
+		$headers = $this->transformHeadersToUppercaseUnderscoreType($headers);
+		return array_merge($default, $headers);
+	}
+
+	public function enableMiddleware()
+	{
+		$this->disableMiddleware = false;
+	}
+
+	public function disableMiddleware()
+	{
+		$this->disableMiddleware = true;
+	}
+
+	/**
+	 * @param  string $method
+	 * @param  string $uri
+	 * @param  array  $data
+	 * @param  array  $headers
+	 * @param  string $content
+	 * @return \Illuminate\Http\Response
+	 */
+	private function request($method, $uri, array $data = [], array $headers = [], $content = null)
+	{
+		// Create a new request object for the internal request
+		$request = $this->createRequest($method, $uri, $data, $headers, $content);
+
+		// Handle the request in the kernel and prepare a response
+		$response = $this->router->prepareResponse($request, $this->app->handle($request));
+
+		return $response;
+	}
+
+	/**
+	 * @param  string $method
+	 * @param  string $uri
+	 * @param  array  $data
+	 * @param  array  $headers
+	 * @param  string $content
+	 * @return \Illuminate\Http\Request
+	 */
+	private function createRequest($method, $uri, array $data = [], array $headers = [], $content = null)
+	{
+		$server = $this->transformHeadersToServerVariables($headers);
+
+		return $this->request->create($uri, $method, $data, [], [], $server, $content);
+	}
+
+	private function transformHeadersToUppercaseUnderscoreType($headers)
+	{
+		$transformed = [];
+
+		foreach ($headers as $headerType => $headerValue) {
+			$headerType = strtoupper(str_replace('-', '_', $headerType));
+
+			$transformed[$headerType] = $headerValue;
+		}
+
+		return $transformed;
+	}
+
+	/**
+	 * https://github.com/symfony/symfony/issues/5074
+	 *
+	 * @param  array $headers
+	 * @return array
+	 */
+	private function transformHeadersToServerVariables($headers)
+	{
+		$server = [];
+
+		foreach ($headers as $headerType => $headerValue) {
+			$headerType = 'HTTP_' . $headerType;
+
+			$server[$headerType] = $headerValue;
+		}
+
+		return $server;
+	}
 }

Spacing +1 added lines, -1 removed lines

@@ -214,7 +214,7 @@
         $server = [];
 
         foreach ($headers as $headerType => $headerValue) {
-            $headerType = 'HTTP_' . $headerType;
+            $headerType = 'HTTP_'.$headerType;
 
             $server[$headerType] = $headerValue;
         }

src/Modules/Acl/Proxy/LoginProxy.php

Indentation +63 added lines, -63 removed lines

@@ -5,76 +5,76 @@
 
 class LoginProxy
 {
-    /**
-     * Attempt to create an access token using user credentials.
-     *
-     * @param  array   $credentials
-     * @param  boolean $adminLogin
-     * @return array
-     */
-    public function login($credentials, $adminLogin = false)
-    {
-        \Core::users()->login($credentials, $adminLogin);
+	/**
+	 * Attempt to create an access token using user credentials.
+	 *
+	 * @param  array   $credentials
+	 * @param  boolean $adminLogin
+	 * @return array
+	 */
+	public function login($credentials, $adminLogin = false)
+	{
+		\Core::users()->login($credentials, $adminLogin);
 
-        return $this->proxy('password', [
-            'username' => $credentials['email'],
-            'password' => $credentials['password']
-        ]);
-    }
+		return $this->proxy('password', [
+			'username' => $credentials['email'],
+			'password' => $credentials['password']
+		]);
+	}
 
-    /**
-     * Attempt to refresh the access token useing the given refresh token.
-     *
-     * @param  string $refreshToken
-     * @return array
-     */
-    public function refreshtoken($refreshToken)
-    {
-        return $this->proxy('refresh_token', [
-            'refresh_token' => $refreshToken
-        ]);
-    }
+	/**
+	 * Attempt to refresh the access token useing the given refresh token.
+	 *
+	 * @param  string $refreshToken
+	 * @return array
+	 */
+	public function refreshtoken($refreshToken)
+	{
+		return $this->proxy('refresh_token', [
+			'refresh_token' => $refreshToken
+		]);
+	}
 
-    /**
-     * Proxy a request to the OAuth server.
-     *
-     * @param string $grantType what type of grant type should be proxied
-     * @param array
-     */
-    public function proxy($grantType, array $data = [])
-    {
-        $data = array_merge($data, [
-            'client_id'     => config('skeleton.passport_client_id'),
-            'client_secret' => config('skeleton.passport_client_secret'),
-            'grant_type'    => $grantType
-        ]);
+	/**
+	 * Proxy a request to the OAuth server.
+	 *
+	 * @param string $grantType what type of grant type should be proxied
+	 * @param array
+	 */
+	public function proxy($grantType, array $data = [])
+	{
+		$data = array_merge($data, [
+			'client_id'     => config('skeleton.passport_client_id'),
+			'client_secret' => config('skeleton.passport_client_secret'),
+			'grant_type'    => $grantType
+		]);
 
-        $response = \ApiConsumer::post('/oauth/token', $data);
+		$response = \ApiConsumer::post('/oauth/token', $data);
 
-        if (! $response->isSuccessful()) {
-            if ($grantType == 'refresh_token') {
-                \ErrorHandler::invalidRefreshToken();
-            }
+		if (! $response->isSuccessful()) {
+			if ($grantType == 'refresh_token') {
+				\ErrorHandler::invalidRefreshToken();
+			}
 
-            \ErrorHandler::loginFailed();
-        }
+			\ErrorHandler::loginFailed();
+		}
 
-        $data = json_decode($response->getContent());
+		$data = json_decode($response->getContent());
 
-        return [
-            'access_token'  => $data->access_token,
-            'refresh_token' => $data->refresh_token,
-            'expires_in'    => $data->expires_in
-        ];
-    }
+		return [
+			'access_token'  => $data->access_token,
+			'refresh_token' => $data->refresh_token,
+			'expires_in'    => $data->expires_in
+		];
+	}
 
-    /**
-     * Logs out the user. We revoke access token and refresh token.
-     *
-     * @return void
-     */
-    public function logout()
-    {
-        \Core::users()->revokeAccessToken(\Auth::user()->token());
-    }
+	/**
+	 * Logs out the user. We revoke access token and refresh token.
+	 *
+	 * @return void
+	 */
+	public function logout()
+	{
+		\Core::users()->revokeAccessToken(\Auth::user()->token());
+	}
 }

Spacing +1 added lines, -1 removed lines

@@ -51,7 +51,7 @@
 
         $response = \ApiConsumer::post('/oauth/token', $data);
 
-        if (! $response->isSuccessful()) {
+        if ( ! $response->isSuccessful()) {
             if ($grantType == 'refresh_token') {
                 \ErrorHandler::invalidRefreshToken();
             }

src/Modules/Core/Facades/ApiConsumer.php

Indentation +4 added lines, -4 removed lines

@@ -4,8 +4,8 @@
 
 class ApiConsumer extends Facade
 {
-    protected static function getFacadeAccessor()
-    {
-        return 'ApiConsumer';
-    }
+	protected static function getFacadeAccessor()
+	{
+		return 'ApiConsumer';
+	}
 }

src/Modules/Core/Providers/ModuleServiceProvider.php

Indentation +47 added lines, -47 removed lines

@@ -6,52 +6,52 @@
 
 class ModuleServiceProvider extends ServiceProvider
 {
-    /**
-     * Bootstrap the module services.
-     *
-     * @return void
-     */
-    public function boot()
-    {
-        $this->loadTranslationsFrom(__DIR__.'/../Resources/Lang', 'core');
-        $this->loadViewsFrom(__DIR__.'/../Resources/Views', 'core');
-
-        $factory = app('Illuminate\Database\Eloquent\Factory');
-        $factory->load(__DIR__.'/../Database/Factories');
-    }
-
-    /**
-     * Register the module services.
-     *
-     * @return void
-     */
-    public function register()
-    {
-        //Bind Core Facade to the IoC Container
-        \App::bind('Core', function () {
-            return new \App\Modules\Core\Core;
-        });
-
-        //Bind ErrorHandler Facade to the IoC Container
-        \App::bind('ErrorHandler', function () {
-            return new \App\Modules\Core\Utl\ErrorHandler;
-        });
-
-        //Bind CoreConfig Facade to the IoC Container
-        \App::bind('CoreConfig', function () {
-            return new \App\Modules\Core\Utl\CoreConfig;
-        });
-
-        //Bind Mpgs Facade to the IoC Container
-        \App::bind('Media', function () {
-            return new \App\Modules\Core\Utl\Media;
-        });
-
-        //Bind Mpgs Facade to the IoC Container
-        \App::bind('ApiConsumer', function () {
-            return new \App\Modules\Core\Utl\ApiConsumer;
-        });
+	/**
+	 * Bootstrap the module services.
+	 *
+	 * @return void
+	 */
+	public function boot()
+	{
+		$this->loadTranslationsFrom(__DIR__.'/../Resources/Lang', 'core');
+		$this->loadViewsFrom(__DIR__.'/../Resources/Views', 'core');
+
+		$factory = app('Illuminate\Database\Eloquent\Factory');
+		$factory->load(__DIR__.'/../Database/Factories');
+	}
+
+	/**
+	 * Register the module services.
+	 *
+	 * @return void
+	 */
+	public function register()
+	{
+		//Bind Core Facade to the IoC Container
+		\App::bind('Core', function () {
+			return new \App\Modules\Core\Core;
+		});
+
+		//Bind ErrorHandler Facade to the IoC Container
+		\App::bind('ErrorHandler', function () {
+			return new \App\Modules\Core\Utl\ErrorHandler;
+		});
+
+		//Bind CoreConfig Facade to the IoC Container
+		\App::bind('CoreConfig', function () {
+			return new \App\Modules\Core\Utl\CoreConfig;
+		});
+
+		//Bind Mpgs Facade to the IoC Container
+		\App::bind('Media', function () {
+			return new \App\Modules\Core\Utl\Media;
+		});
+
+		//Bind Mpgs Facade to the IoC Container
+		\App::bind('ApiConsumer', function () {
+			return new \App\Modules\Core\Utl\ApiConsumer;
+		});
         
-        $this->app->register(RouteServiceProvider::class);
-    }
+		$this->app->register(RouteServiceProvider::class);
+	}
 }

Spacing +5 added lines, -5 removed lines

@@ -28,27 +28,27 @@
     public function register()
     {
         //Bind Core Facade to the IoC Container
-        \App::bind('Core', function () {
+        \App::bind('Core', function() {
             return new \App\Modules\Core\Core;
         });
 
         //Bind ErrorHandler Facade to the IoC Container
-        \App::bind('ErrorHandler', function () {
+        \App::bind('ErrorHandler', function() {
             return new \App\Modules\Core\Utl\ErrorHandler;
         });
 
         //Bind CoreConfig Facade to the IoC Container
-        \App::bind('CoreConfig', function () {
+        \App::bind('CoreConfig', function() {
             return new \App\Modules\Core\Utl\CoreConfig;
         });
 
         //Bind Mpgs Facade to the IoC Container
-        \App::bind('Media', function () {
+        \App::bind('Media', function() {
             return new \App\Modules\Core\Utl\Media;
         });
 
         //Bind Mpgs Facade to the IoC Container
-        \App::bind('ApiConsumer', function () {
+        \App::bind('ApiConsumer', function() {
             return new \App\Modules\Core\Utl\ApiConsumer;
         });