Saluki / Barmate

app/Http/Controllers/UsersController.php

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

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())
             {

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())
             {

app/Http/Middleware/RedirectIfInstalled.php

Spacing +1 added lines, -1 removed lines

@@ -18,7 +18,7 @@
     public function handle($request, Closure $next)
     {
         // Installation lock disabled, we can't run the install process anymore
-        if (!file_exists(base_path() . '/' . self::INSTALLATION_FILE)) {
+        if (!file_exists(base_path().'/'.self::INSTALLATION_FILE)) {
             return redirect('/');
         }
 

app/Http/Middleware/User.php

Spacing +1 added lines, -1 removed lines

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

app/Http/Middleware/VerifyCsrfToken.php

Spacing +1 added lines, -1 removed lines

@@ -17,7 +17,7 @@
 	public function handle($request, Closure $next)
 	{
 		// Disable CSRF control for xHttp Requests
-		if($request->isXmlHttpRequest())	
+		if ($request->isXmlHttpRequest())	
 		{
 			return $next($request);
 		}

app/Http/routes.php

Spacing +10 added lines, -10 removed lines

@@ -2,7 +2,7 @@ 
 
 // INSTALLATION ROUTES
 
-Route::group(['prefix'=>'install', 'middleware'=>'installed'], function(){
+Route::group(['prefix'=>'install', 'middleware'=>'installed'], function() {
 
     Route::get('/', 'InstallController@displayWelcome');
 
@@ -28,7 +28,7 @@ 
 
 // PUBLIC ROUTES
 
-Route::group(['middleware'=>'guest'], function(){
+Route::group(['middleware'=>'guest'], function() {
 
     Route::get('/', 'LoginController@getLoginForm');
 
@@ -38,7 +38,7 @@ 
 
 // USER ROUTES
 
-Route::group(['prefix'=>'app', 'middleware'=>'user'], function(){
+Route::group(['prefix'=>'app', 'middleware'=>'user'], function() {
 
 	// POS MODULE
 
@@ -58,15 +58,15 @@ 
 
 // MANAGER ROUTES
 
-Route::group(['prefix'=>'app', 'middleware'=>'manager'], function(){
+Route::group(['prefix'=>'app', 'middleware'=>'manager'], function() {
 
 	// STOCK MODULE
 			
     Route::get('stock', 'StockController@dashboard');
 
-	Route::resource('stock/category', 'CategoryController', ['except' => ['create','edit']]);
+	Route::resource('stock/category', 'CategoryController', ['except' => ['create', 'edit']]);
 
-	Route::resource('stock/product', 'ProductController', ['except' => ['create','edit']]);
+	Route::resource('stock/product', 'ProductController', ['except' => ['create', 'edit']]);
 
 	// CASH MODULE
 
@@ -92,7 +92,7 @@ 
 
 // ADMIN ROUTES
 
-Route::group(['prefix'=>'app', 'middleware'=>'administrator'], function(){
+Route::group(['prefix'=>'app', 'middleware'=>'administrator'], function() {
 
     // USERS MODULE
 	
@@ -106,11 +106,11 @@ 
 			
 	Route::post('users/register', 'UsersController@registerNewUser');
 	
-	Route::get('users/change-status/{id}', 'UsersController@changeAccountStatus')->where('id', '[0-9]+');;
+	Route::get('users/change-status/{id}', 'UsersController@changeAccountStatus')->where('id', '[0-9]+'); ;
 	
-	Route::get('users/change-role/{id}', 'UsersController@changeAccountRole')->where('id', '[0-9]+');;
+	Route::get('users/change-role/{id}', 'UsersController@changeAccountRole')->where('id', '[0-9]+'); ;
 	
-	Route::get('users/delete/{id}', 'UsersController@deleteUser')->where('id', '[0-9]+');;
+	Route::get('users/delete/{id}', 'UsersController@deleteUser')->where('id', '[0-9]+'); ;
 
     // SETTINGS MODULE
 	

app/Models/CashSnapshots.php

Spacing +3 added lines, -3 removed lines

@@ -8,13 +8,13 @@
 	public $primaryKey = 'cs_id';
 	public $timestamps = false;
 
-	public static $validationRules = [	'title'       => 'required|name',
+	public static $validationRules = ['title'       => 'required|name',
 										'description' => '',
-										'amount'      => 'required|numeric|between:0,9999' ];
+										'amount'      => 'required|numeric|between:0,9999'];
 
 	public function details()
 	{
-		return $this->hasMany('App\Models\SnapshotDetails','cs_id');
+		return $this->hasMany('App\Models\SnapshotDetails', 'cs_id');
 	}
 
 	public function scopeWithUsers($query)

app/Models/Categories.php

Spacing +3 added lines, -3 removed lines

@@ -15,7 +15,7 @@ 
 
 	protected $guarded = ['category_id'];
 
-	public function getKeyName(){
+	public function getKeyName() {
 
         return 'category_id';
     }
@@ -28,7 +28,7 @@ 
 					WHERE c.group_id = ?
 					AND c.deleted_at IS NULL';
 
-		return DB::select($query, [ Session::get('groupID') ]);
+		return DB::select($query, [Session::get('groupID')]);
     }
 
 	/**
@@ -42,7 +42,7 @@ 
 	 */
 	public function scopeFromGroup($query) {
 
-		return $query->where('group_id', Session::get('groupID') );
+		return $query->where('group_id', Session::get('groupID'));
 	}
 
 	/**