Saluki / Barmate

app/Http/Controllers/ProductController.php

Spacing +1 added lines, -1 removed lines

@@ -51,7 +51,7 @@
 
         if (is_null($product)) {
             return response()->json(['code' => RepositoryException::RESOURCE_NOT_FOUND,
-                'message' => 'Product with ID ' . $id . ' not found'], 404);
+                'message' => 'Product with ID '.$id.' not found'], 404);
         }
 
         return response()->json($this->repository->APIFormat($product), 200);

app/Http/Controllers/PublicController.php

Indentation +5 added lines, -5 removed lines

@@ -2,10 +2,10 @@
 
 class PublicController extends Controller {
 	
-	// UNUSED
-	public function home()
-	{        
-		return view('public.home');
-	}
+    // UNUSED
+    public function home()
+    {        
+        return view('public.home');
+    }
 
 }

app/Http/Controllers/StatsController.php

Spacing +7 added lines, -7 removed lines

@@ -20,7 +20,7 @@ 
         $this->productRepository = $productRepository;
     }
 
-    public function dashboard($interval=self::DEFAULT_INTERVAL)
+    public function dashboard($interval = self::DEFAULT_INTERVAL)
     {
         $intervalData = $this->extractIntervalData($interval);
 
@@ -48,18 +48,18 @@ 
         $type = substr($intervalString, -1);
         $nb = intval(substr($intervalString, 0, -1));
 
-        if( $type=='h' )
+        if ($type == 'h')
         {
-            if( $nb>24 )
+            if ($nb>24)
             {
                 $nb = 24;
             }
 
             return CarbonInterval::hours($nb);
         }
-        elseif( $type=='d' )
+        elseif ($type == 'd')
         {
-            if( $nb>30 )
+            if ($nb>30)
             {
                 $nb = 30;
             }
@@ -78,10 +78,10 @@ 
      */
     private function getTitleFromInterval(CarbonInterval $interval)
     {
-        if( $interval->h>0 )
+        if ($interval->h>0)
             return 'Last '.$interval->h.' hours';
 
-        if( $interval->d>0 )
+        if ($interval->d>0)
             return 'Last '.$interval->d.' days';
 
         return 'Last '.$interval->forHumans();

Braces +7 added lines, -6 removed lines

@@ -56,8 +56,7 @@ 
             }
 
             return CarbonInterval::hours($nb);
-        }
-        elseif( $type=='d' )
+        } elseif( $type=='d' )
         {
             if( $nb>30 )
             {
@@ -78,11 +77,13 @@ 
      */
     private function getTitleFromInterval(CarbonInterval $interval)
     {
-        if( $interval->h>0 )
-            return 'Last '.$interval->h.' hours';
+        if( $interval->h>0 ) {
+                    return 'Last '.$interval->h.' hours';
+        }
 
-        if( $interval->d>0 )
-            return 'Last '.$interval->d.' days';
+        if( $interval->d>0 ) {
+                    return 'Last '.$interval->d.' days';
+        }
 
         return 'Last '.$interval->forHumans();
     }

app/Http/Controllers/StockController.php

Spacing +1 added lines, -1 removed lines

@@ -15,7 +15,7 @@
     public function dashboard()
     {
         $categories = Categories::allAPI();
-        $stockEmptyAlert = intval( $this->settingRepository->getValue(SettingRepository::STOCK_EMPTY_ALERT) );
+        $stockEmptyAlert = intval($this->settingRepository->getValue(SettingRepository::STOCK_EMPTY_ALERT));
 
         return view('stock.app')->with('categories', $categories)
                                 ->with('stockEmptyAlert', $stockEmptyAlert);

app/Http/Controllers/UsersController.php

Indentation +50 added lines, -50 removed lines

@@ -8,34 +8,34 @@ 
 
 class UsersController extends Controller {
 	
-	private $userRepository;
+    private $userRepository;
     private $connectRepository;
 
-	public function __construct(UserRepository $repository, ConnectRepository $connectRepository)
-	{
-		$this->userRepository = $repository;
+    public function __construct(UserRepository $repository, ConnectRepository $connectRepository)
+    {
+        $this->userRepository = $repository;
         $this->connectRepository = $connectRepository;
-	}
+    }
 
     public function getActiveUsers()
     {
-    	$allUsers = $this->userRepository->allByStatus(true);
+        $allUsers = $this->userRepository->allByStatus(true);
         $otherUsersCount = count($this->userRepository->allByStatus(false));
 
         return view('users.app')->with('users', $allUsers)
-        						->with('roles', $this->getRoleNames())
-        						->with('isActive', true)
+                                ->with('roles', $this->getRoleNames())
+                                ->with('isActive', true)
                                 ->with('otherUsersCount', $otherUsersCount);
     }
     
     public function getDisabledUsers()
     {
-    	$allUsers = $this->userRepository->allByStatus(false);
+        $allUsers = $this->userRepository->allByStatus(false);
         $otherUsersCount = count($this->userRepository->allByStatus(true));
 
-    	return view('users.app')->with('users', $allUsers)
-    							->with('roles', $this->getRoleNames())
-    							->with('isActive', false)
+        return view('users.app')->with('users', $allUsers)
+                                ->with('roles', $this->getRoleNames())
+                                ->with('isActive', false)
                                 ->with('otherUsersCount', $otherUsersCount);
     }
 
@@ -50,70 +50,70 @@ 
     
     public function getRegisterForm()
     {
-    	return view('users.register');
+        return view('users.register');
     }
     
     public function registerNewUser()
     {
-    	try {
-    		$user = $this->userRepository->store(Input::all());
-    	}
+        try {
+            $user = $this->userRepository->store(Input::all());
+        }
         catch(ValidationException $e) {
             return redirect('app/users/register')->withInput()
-                                                 ->with('error', $e->getMessageBag()->first());
+                                                    ->with('error', $e->getMessageBag()->first());
         }
         catch (RepositoryException $e) {
-    		return redirect('app/users/register')->with('error', 'Could not add new user: '.strtolower($e->getMessage()))
-                                                 ->withInput();
-    	}
+            return redirect('app/users/register')->with('error', 'Could not add new user: '.strtolower($e->getMessage()))
+                                                    ->withInput();
+        }
     	    	
-    	return redirect('app/users')->with('success', $user->firstname.' account added');
+        return redirect('app/users')->with('success', $user->firstname.' account added');
     }
     
     public function changeAccountStatus($userId)
     {    	
-    	try {
-    		$user = $this->userRepository->changeStatus($userId);
-    	}
-    	catch(RepositoryException $e) {
-    		return redirect('app/users')->with('error', 'Could not change account status');
-    	}
+        try {
+            $user = $this->userRepository->changeStatus($userId);
+        }
+        catch(RepositoryException $e) {
+            return redirect('app/users')->with('error', 'Could not change account status');
+        }
     	
-    	if($user->is_active)
-    		return redirect('app/users/disabled')->with('success', 'User <b>'.$user->firstname.'</b> enabled');
-    	else
-    		return redirect('app/users')->with('success', 'User <b>'.$user->firstname.'</b> disabled');
+        if($user->is_active)
+            return redirect('app/users/disabled')->with('success', 'User <b>'.$user->firstname.'</b> enabled');
+        else
+            return redirect('app/users')->with('success', 'User <b>'.$user->firstname.'</b> disabled');
     }
     
     public function changeAccountRole($userId)
     {
-    	try {
-    		$user = $this->userRepository->changeRole($userId);
-    	}
-    	catch(RepositoryException $e) {
-    		return redirect('app/users')->with('error', 'Could not change user role');
-    	}
+        try {
+            $user = $this->userRepository->changeRole($userId);
+        }
+        catch(RepositoryException $e) {
+            return redirect('app/users')->with('error', 'Could not change user role');
+        }
     	
-    	$redirectUrl = 'app/users';
-    	if( !$user->is_active )
-    		$redirectUrl .= '/disabled';
+        $redirectUrl = 'app/users';
+        if( !$user->is_active )
+            $redirectUrl .= '/disabled';
     	
-    	$roleNames = ['ADMN'=>'administrator','MNGR'=>'manager','USER'=>'user'];
-    	$roleName = $roleNames[$user->role];
+        $roleNames = ['ADMN'=>'administrator','MNGR'=>'manager','USER'=>'user'];
+        $roleName = $roleNames[$user->role];
     	
-    	return redirect($redirectUrl)->with('success', 'User <b>'.$user->firstname.'</b> is now <i>'.$roleName.'</i>');
+        return redirect($redirectUrl)->with('success', 'User <b>'.$user->firstname.'</b> is now <i>'.$roleName.'</i>');
     }
     
     public function deleteUser($userId)
     {
-    	try {
-    		$user = $this->userRepository->softDelete($userId);
-    	}
-    	catch(RepositoryException $e) {
-    		return redirect('app/users')->with('error', 'User could not be deleted: '.$e->getMessage());
-    	}
+        try {
+            $user = $this->userRepository->softDelete($userId);
+        }
+        catch(RepositoryException $e) {
+            return redirect('app/users')->with('error', 'User could not be deleted: '.$e->getMessage());
+        }
     	
-    	return redirect('app/users')->with('success', 'User <b>'.$user->firstname.'</b> deleted');
+        return redirect('app/users')->with('success', 'User <b>'.$user->firstname.'</b> deleted');
     }
 
     private function getRoleNames()

Spacing +8 added lines, -8 removed lines

@@ -58,7 +58,7 @@ 
     	try {
     		$user = $this->userRepository->store(Input::all());
     	}
-        catch(ValidationException $e) {
+        catch (ValidationException $e) {
             return redirect('app/users/register')->withInput()
                                                  ->with('error', $e->getMessageBag()->first());
         }
@@ -75,11 +75,11 @@ 
     	try {
     		$user = $this->userRepository->changeStatus($userId);
     	}
-    	catch(RepositoryException $e) {
+    	catch (RepositoryException $e) {
     		return redirect('app/users')->with('error', 'Could not change account status');
     	}
     	
-    	if($user->is_active)
+    	if ($user->is_active)
     		return redirect('app/users/disabled')->with('success', 'User <b>'.$user->firstname.'</b> enabled');
     	else
     		return redirect('app/users')->with('success', 'User <b>'.$user->firstname.'</b> disabled');
@@ -90,15 +90,15 @@ 
     	try {
     		$user = $this->userRepository->changeRole($userId);
     	}
-    	catch(RepositoryException $e) {
+    	catch (RepositoryException $e) {
     		return redirect('app/users')->with('error', 'Could not change user role');
     	}
     	
     	$redirectUrl = 'app/users';
-    	if( !$user->is_active )
+    	if (!$user->is_active)
     		$redirectUrl .= '/disabled';
     	
-    	$roleNames = ['ADMN'=>'administrator','MNGR'=>'manager','USER'=>'user'];
+    	$roleNames = ['ADMN'=>'administrator', 'MNGR'=>'manager', 'USER'=>'user'];
     	$roleName = $roleNames[$user->role];
     	
     	return redirect($redirectUrl)->with('success', 'User <b>'.$user->firstname.'</b> is now <i>'.$roleName.'</i>');
@@ -109,7 +109,7 @@ 
     	try {
     		$user = $this->userRepository->softDelete($userId);
     	}
-    	catch(RepositoryException $e) {
+    	catch (RepositoryException $e) {
     		return redirect('app/users')->with('error', 'User could not be deleted: '.$e->getMessage());
     	}
     	
@@ -118,7 +118,7 @@ 
 
     private function getRoleNames()
     {
-        return ['ADMN'=>'Administrator','MNGR'=>'Manager','USER'=>'User'];
+        return ['ADMN'=>'Administrator', 'MNGR'=>'Manager', 'USER'=>'User'];
     }
 
 }

Braces +13 added lines, -16 removed lines

@@ -57,12 +57,10 @@ 
     {
     	try {
     		$user = $this->userRepository->store(Input::all());
-    	}
-        catch(ValidationException $e) {
+    	} catch(ValidationException $e) {
             return redirect('app/users/register')->withInput()
                                                  ->with('error', $e->getMessageBag()->first());
-        }
-        catch (RepositoryException $e) {
+        } catch (RepositoryException $e) {
     		return redirect('app/users/register')->with('error', 'Could not add new user: '.strtolower($e->getMessage()))
                                                  ->withInput();
     	}
@@ -74,29 +72,29 @@ 
     {    	
     	try {
     		$user = $this->userRepository->changeStatus($userId);
-    	}
-    	catch(RepositoryException $e) {
+    	} catch(RepositoryException $e) {
     		return redirect('app/users')->with('error', 'Could not change account status');
     	}
     	
-    	if($user->is_active)
-    		return redirect('app/users/disabled')->with('success', 'User <b>'.$user->firstname.'</b> enabled');
-    	else
-    		return redirect('app/users')->with('success', 'User <b>'.$user->firstname.'</b> disabled');
+    	if($user->is_active) {
+    	    		return redirect('app/users/disabled')->with('success', 'User <b>'.$user->firstname.'</b> enabled');
+    	} else {
+    	    		return redirect('app/users')->with('success', 'User <b>'.$user->firstname.'</b> disabled');
+    	}
     }
     
     public function changeAccountRole($userId)
     {
     	try {
     		$user = $this->userRepository->changeRole($userId);
-    	}
-    	catch(RepositoryException $e) {
+    	} catch(RepositoryException $e) {
     		return redirect('app/users')->with('error', 'Could not change user role');
     	}
     	
     	$redirectUrl = 'app/users';
-    	if( !$user->is_active )
-    		$redirectUrl .= '/disabled';
+    	if( !$user->is_active ) {
+    	    		$redirectUrl .= '/disabled';
+    	}
     	
     	$roleNames = ['ADMN'=>'administrator','MNGR'=>'manager','USER'=>'user'];
     	$roleName = $roleNames[$user->role];
@@ -108,8 +106,7 @@ 
     {
     	try {
     		$user = $this->userRepository->softDelete($userId);
-    	}
-    	catch(RepositoryException $e) {
+    	} catch(RepositoryException $e) {
     		return redirect('app/users')->with('error', 'User could not be deleted: '.$e->getMessage());
     	}
     	

app/Http/Kernel.php

Indentation +23 added lines, -23 removed lines

@@ -4,33 +4,33 @@
 
 class Kernel extends HttpKernel {
 
-	/**
-	 * The application's global HTTP middleware stack.
-	 *
-	 * @var array
-	 */
-	protected $middleware = [
-		'Illuminate\Foundation\Http\Middleware\CheckForMaintenanceMode',
-		'Illuminate\Cookie\Middleware\EncryptCookies',
-		'Illuminate\Cookie\Middleware\AddQueuedCookiesToResponse',
-		'Illuminate\Session\Middleware\StartSession',
-		'Illuminate\View\Middleware\ShareErrorsFromSession',
-		'App\Http\Middleware\VerifyCsrfToken',
-	];
+    /**
+     * The application's global HTTP middleware stack.
+     *
+     * @var array
+     */
+    protected $middleware = [
+        'Illuminate\Foundation\Http\Middleware\CheckForMaintenanceMode',
+        'Illuminate\Cookie\Middleware\EncryptCookies',
+        'Illuminate\Cookie\Middleware\AddQueuedCookiesToResponse',
+        'Illuminate\Session\Middleware\StartSession',
+        'Illuminate\View\Middleware\ShareErrorsFromSession',
+        'App\Http\Middleware\VerifyCsrfToken',
+    ];
 
-	/**
-	 * The application's route middleware.
-	 *
-	 * @var array
-	 */
-	protected $routeMiddleware = [
-		'auth'          => 'App\Http\Middleware\Authenticate',
-		'auth.basic'    => 'Illuminate\Auth\Middleware\AuthenticateWithBasicAuth',
-		'guest'         => 'App\Http\Middleware\RedirectIfAuthenticated',
+    /**
+     * The application's route middleware.
+     *
+     * @var array
+     */
+    protected $routeMiddleware = [
+        'auth'          => 'App\Http\Middleware\Authenticate',
+        'auth.basic'    => 'Illuminate\Auth\Middleware\AuthenticateWithBasicAuth',
+        'guest'         => 'App\Http\Middleware\RedirectIfAuthenticated',
         'user'          => 'App\Http\Middleware\User',
         'manager'       => 'App\Http\Middleware\Manager',
         'administrator' => 'App\Http\Middleware\Administrator',
         'installed'     => 'App\Http\Middleware\RedirectIfInstalled',
-	];
+    ];
 
 }

app/Http/Middleware/Administrator.php

Spacing +1 added lines, -1 removed lines

@@ -14,7 +14,7 @@
 
 	public function handle($request, Closure $next)
 	{
-        if ($this->auth->guest() || session('role')!=='ADMN' )
+        if ($this->auth->guest() || session('role') !== 'ADMN')
         {
             if ($request->ajax())
             {

Indentation +4 added lines, -4 removed lines

@@ -12,8 +12,8 @@ 
         $this->auth = $auth;
     }
 
-	public function handle($request, Closure $next)
-	{
+    public function handle($request, Closure $next)
+    {
         if ($this->auth->guest() || (session('role')!=='USER' && session('role')!=='MNGR' && session('role')!=='ADMN') )
         {
             if ($request->ajax())
@@ -26,7 +26,7 @@ 
             }
         }
 
-		return $next($request);
-	}
+        return $next($request);
+    }
 
 }

Braces +1 added lines, -2 removed lines

@@ -19,8 +19,7 @@
             if ($request->ajax())
             {
                 return response('Unauthorized.', 401);
-            }
-            else
+            } else
             {
                 return redirect()->guest('/');
             }

app/Http/Middleware/Authenticate.php

Braces +1 added lines, -2 removed lines

@@ -37,8 +37,7 @@
 			if ($request->ajax())
 			{
 				return response('Unauthorized.', 401);
-			}
-			else
+			} else
 			{
 				return redirect()->guest('/');
 			}

Indentation +37 added lines, -37 removed lines

@@ -5,45 +5,45 @@
 
 class Authenticate {
 
-	/**
-	 * The Guard implementation.
-	 *
-	 * @var Guard
-	 */
-	protected $auth;
+    /**
+     * The Guard implementation.
+     *
+     * @var Guard
+     */
+    protected $auth;
 
-	/**
-	 * Create a new filter instance.
-	 *
-	 * @param  Guard  $auth
-	 */
-	public function __construct(Guard $auth)
-	{
-		$this->auth = $auth;
-	}
+    /**
+     * Create a new filter instance.
+     *
+     * @param  Guard  $auth
+     */
+    public function __construct(Guard $auth)
+    {
+        $this->auth = $auth;
+    }
 
-	/**
-	 * Handle an incoming request.
-	 *
-	 * @param  \Illuminate\Http\Request  $request
-	 * @param  \Closure  $next
-	 * @return mixed
-	 */
-	public function handle($request, Closure $next)
-	{
-		if ($this->auth->guest())
-		{
-			if ($request->ajax())
-			{
-				return response('Unauthorized.', 401);
-			}
-			else
-			{
-				return redirect()->guest('/');
-			}
-		}
+    /**
+     * Handle an incoming request.
+     *
+     * @param  \Illuminate\Http\Request  $request
+     * @param  \Closure  $next
+     * @return mixed
+     */
+    public function handle($request, Closure $next)
+    {
+        if ($this->auth->guest())
+        {
+            if ($request->ajax())
+            {
+                return response('Unauthorized.', 401);
+            }
+            else
+            {
+                return redirect()->guest('/');
+            }
+        }
 
-		return $next($request);
-	}
+        return $next($request);
+    }
 
 }

app/Http/Middleware/Manager.php

Spacing +1 added lines, -1 removed lines

@@ -14,7 +14,7 @@
 
 	public function handle($request, Closure $next)
 	{
-        if ($this->auth->guest() || (session('role')!=='MNGR' && session('role')!=='ADMN') )
+        if ($this->auth->guest() || (session('role') !== 'MNGR' && session('role') !== 'ADMN'))
         {
             if ($request->ajax())
             {

Indentation +4 added lines, -4 removed lines

@@ -12,8 +12,8 @@ 
         $this->auth = $auth;
     }
 
-	public function handle($request, Closure $next)
-	{
+    public function handle($request, Closure $next)
+    {
         if ($this->auth->guest() || (session('role')!=='USER' && session('role')!=='MNGR' && session('role')!=='ADMN') )
         {
             if ($request->ajax())
@@ -26,7 +26,7 @@ 
             }
         }
 
-		return $next($request);
-	}
+        return $next($request);
+    }
 
 }

Braces +1 added lines, -2 removed lines

@@ -19,8 +19,7 @@
             if ($request->ajax())
             {
                 return response('Unauthorized.', 401);
-            }
-            else
+            } else
             {
                 return redirect()->guest('/');
             }