idevelopment / RingMe

app/Http/Controllers/HomeController.php

Unused Use Statements -3 removed lines

@@ -2,9 +2,6 @@
 
 namespace App\Http\Controllers;
 
-use App\Http\Requests;
-use Illuminate\Http\Request;
-
 class HomeController extends Controller
 {
     /**

Braces +2 added lines, -6 removed lines

@@ -30,13 +30,9 @@
 
       if(Bouncer::is($user)->a('Administrator', 'Manager')) {
           return redirect()->route('dashboard.administration');
-      }
-
-      elseif(Bouncer::is($user)->an('Agent')) {
+      } elseif(Bouncer::is($user)->an('Agent')) {
         return redirect()->route('dashboard.agent');
-      }
-
-      elseif(Bouncer::is($user)->an('Customer')) {
+      } elseif(Bouncer::is($user)->an('Customer')) {
           return redirect('/');
       }
         return redirect('/');

Doc Comments +1 added lines, -1 removed lines

@@ -50,7 +50,7 @@
     /**
      * Save the customer.
      *
-     * @return \Illuminate\Http\Response
+     * @return \Illuminate\Http\RedirectResponse
      */
     public function registerCustomer(Request $request)
     {

Indentation +39 added lines, -39 removed lines

@@ -27,19 +27,19 @@ 
      */
     public function index()
     {
-      $user = auth()->user();
+        $user = auth()->user();
 
-      if(Bouncer::is($user)->a('Administrator', 'Manager')) {
-          return redirect()->route('dashboard.administration');
-      }
+        if(Bouncer::is($user)->a('Administrator', 'Manager')) {
+            return redirect()->route('dashboard.administration');
+        }
 
-      elseif(Bouncer::is($user)->an('Agent')) {
+        elseif(Bouncer::is($user)->an('Agent')) {
         return redirect()->route('dashboard.agent');
-      }
+        }
 
-      elseif(Bouncer::is($user)->an('Customer')) {
-          return redirect('/');
-      }
+        elseif(Bouncer::is($user)->an('Customer')) {
+            return redirect('/');
+        }
         return redirect('/');
     }
 
@@ -51,28 +51,28 @@ 
     public function administration()
     {
 
-      if (\Auth::check()) {
+        if (\Auth::check()) {
     // The user is logged in...
 
-  $CountOpenRequests = Callback::where('status', 'open')->count();
-  $data["chart"] = Charts::create('percentage', 'justgage')
-  ->Title('')
-  ->values([$CountOpenRequests, 0, 4])
-  ->ElementLabel('Open requests')
-  ->Responsive(false)
-  ->Height(250)
-  ->Width(0);
-
-  $CountOverdueRequests = Callback::where('status', 'overdue')->count();
-  $data["overdue"] = Charts::create('percentage', 'justgage')
-  ->Title('')
-  ->ElementLabel('Overdue requests')
-  ->values([$CountOverdueRequests, 0 ,1])
-  ->Responsive(false)
-  ->Height(250)
-  ->Width(0);
-
-  $data["assigned"] = Charts::create('percentage', 'justgage')
+    $CountOpenRequests = Callback::where('status', 'open')->count();
+    $data["chart"] = Charts::create('percentage', 'justgage')
+    ->Title('')
+    ->values([$CountOpenRequests, 0, 4])
+    ->ElementLabel('Open requests')
+    ->Responsive(false)
+    ->Height(250)
+    ->Width(0);
+
+    $CountOverdueRequests = Callback::where('status', 'overdue')->count();
+    $data["overdue"] = Charts::create('percentage', 'justgage')
+    ->Title('')
+    ->ElementLabel('Overdue requests')
+    ->values([$CountOverdueRequests, 0 ,1])
+    ->Responsive(false)
+    ->Height(250)
+    ->Width(0);
+
+    $data["assigned"] = Charts::create('percentage', 'justgage')
 ->Title('')
 ->ElementLabel('Tasks assigned')
 ->Values([10, 0, 2])
@@ -92,9 +92,9 @@ 
      */
     public function agent()
     {
-      $user = auth()->user();
+        $user = auth()->user();
 
-      if (\Auth::check()) {
+        if (\Auth::check()) {
 
         $CountOpenRequests = Callback::where('status', 'open')->count();
         $data["chart"] = Charts::create('percentage', 'justgage')
@@ -115,14 +115,14 @@ 
         ->Width(0);
 
         $data["assigned"] = Charts::create('percentage', 'justgage')
-      ->Title('')
-      ->ElementLabel('Tasks assigned')
-      ->values([80,0,100])
-      ->Responsive(false)
-      ->Height(200)
-      ->Width(0);
-
-      $data["callbacks"] = Callback::where('agent_id', $user->id)->with('users', 'customers', 'departments')->get();
+        ->Title('')
+        ->ElementLabel('Tasks assigned')
+        ->values([80,0,100])
+        ->Responsive(false)
+        ->Height(200)
+        ->Width(0);
+
+        $data["callbacks"] = Callback::where('agent_id', $user->id)->with('users', 'customers', 'departments')->get();
 
         return view('home/agent', $data);
 

Spacing +16 added lines, -16 removed lines

@@ -29,15 +29,15 @@ 
     {
       $user = auth()->user();
 
-      if(Bouncer::is($user)->a('Administrator', 'Manager')) {
+      if (Bouncer::is($user)->a('Administrator', 'Manager')) {
           return redirect()->route('dashboard.administration');
       }
 
-      elseif(Bouncer::is($user)->an('Agent')) {
+      elseif (Bouncer::is($user)->an('Agent')) {
         return redirect()->route('dashboard.agent');
       }
 
-      elseif(Bouncer::is($user)->an('Customer')) {
+      elseif (Bouncer::is($user)->an('Customer')) {
           return redirect('/');
       }
         return redirect('/');
@@ -55,27 +55,27 @@ 
     // The user is logged in...
 
   $CountOpenRequests = Callback::where('status', 'open')->count();
-  $data["chart"] = Charts::create('percentage', 'justgage')
+  $data[ "chart" ] = Charts::create('percentage', 'justgage')
   ->Title('')
-  ->values([$CountOpenRequests, 0, 4])
+  ->values([ $CountOpenRequests, 0, 4 ])
   ->ElementLabel('Open requests')
   ->Responsive(false)
   ->Height(250)
   ->Width(0);
 
   $CountOverdueRequests = Callback::where('status', 'overdue')->count();
-  $data["overdue"] = Charts::create('percentage', 'justgage')
+  $data[ "overdue" ] = Charts::create('percentage', 'justgage')
   ->Title('')
   ->ElementLabel('Overdue requests')
-  ->values([$CountOverdueRequests, 0 ,1])
+  ->values([ $CountOverdueRequests, 0, 1 ])
   ->Responsive(false)
   ->Height(250)
   ->Width(0);
 
-  $data["assigned"] = Charts::create('percentage', 'justgage')
+  $data[ "assigned" ] = Charts::create('percentage', 'justgage')
 ->Title('')
 ->ElementLabel('Tasks assigned')
-->Values([10, 0, 2])
+->Values([ 10, 0, 2 ])
 ->Responsive(false)
 ->Height(250)
 ->Width(0);
@@ -97,32 +97,32 @@ 
       if (\Auth::check()) {
 
         $CountOpenRequests = Callback::where('status', 'open')->count();
-        $data["chart"] = Charts::create('percentage', 'justgage')
+        $data[ "chart" ] = Charts::create('percentage', 'justgage')
         ->Title('')
-        ->values([$CountOpenRequests, 0, 10])
+        ->values([ $CountOpenRequests, 0, 10 ])
         ->ElementLabel('Open requests')
         ->Responsive(false)
         ->Height(200)
         ->Width(0);
 
         $CountOverdueRequests = Callback::where('status', 'overdue')->count();
-        $data["overdue"] = Charts::create('percentage', 'justgage')
+        $data[ "overdue" ] = Charts::create('percentage', 'justgage')
         ->Title('')
         ->ElementLabel('Overdue requests')
-        ->values([$CountOverdueRequests, 0 ,100])
+        ->values([ $CountOverdueRequests, 0, 100 ])
         ->Responsive(false)
         ->Height(200)
         ->Width(0);
 
-        $data["assigned"] = Charts::create('percentage', 'justgage')
+        $data[ "assigned" ] = Charts::create('percentage', 'justgage')
       ->Title('')
       ->ElementLabel('Tasks assigned')
-      ->values([80,0,100])
+      ->values([ 80, 0, 100 ])
       ->Responsive(false)
       ->Height(200)
       ->Width(0);
 
-      $data["callbacks"] = Callback::where('agent_id', $user->id)->with('users', 'customers', 'departments')->get();
+      $data[ "callbacks" ] = Callback::where('agent_id', $user->id)->with('users', 'customers', 'departments')->get();
 
         return view('home/agent', $data);
 

app/Providers/RouteServiceProvider.php

Spacing +1 added lines, -1 removed lines

@@ -54,7 +54,7 @@
     {
         $router->group([
             'namespace' => $this->namespace, 'middleware' => 'web',
-        ], function ($router) {
+        ], function($router) {
             require app_path('Http/routes.php');
         });
     }

app/Http/Middleware/languageMiddleware.php

Spacing +1 added lines, -1 removed lines

@@ -17,7 +17,7 @@
     /**
      * @var array
      */
-    protected static $supportedLanguages = ['nl', 'en', 'fr'];
+    protected static $supportedLanguages = [ 'nl', 'en', 'fr' ];
     /**
      * Handle an incoming request.
      *

app/Providers/AppServiceProvider.php

Spacing +1 added lines, -1 removed lines

@@ -14,7 +14,7 @@
      */
     public function boot()
     {
-        Bouncer::seeder(function () {
+        Bouncer::seeder(function() {
             Bouncer::allow('available')->to('');
             Bouncer::allow('unavailable')->to('');
 

app/Http/Middleware/RoleMiddlware.php

Spacing +1 added lines, -1 removed lines

@@ -16,7 +16,7 @@
      */
     public function handle($request, Closure $next, $role)
     {
-        if (! auth()->user()->is($role)) {
+        if (!auth()->user()->is($role)) {
             return redirect()->back(302);
         }
 

app/Http/Controllers/CustomersController.php

Unused Use Statements -2 removed lines

@@ -3,8 +3,6 @@
 namespace App\Http\Controllers;
 
 use App\Customer;
-use Illuminate\Http\Request;
-
 use App\Http\Requests;
 
 class CustomersController extends Controller

Indentation +9 added lines, -9 removed lines

@@ -27,12 +27,12 @@ 
      */
     public function index()
     {
-      $user = auth()->user();
+        $user = auth()->user();
 
-      if ($user->is('Agent') || $user->is('Manager') || $user->is('Administrator')) {
+        if ($user->is('Agent') || $user->is('Manager') || $user->is('Administrator')) {
         $data['customers'] = Customer::orderBy('status' , 'asc')->paginate(15);
         return view('customers/index', $data);
-      }
+        }
         return redirect()->back();
     }
 
@@ -65,9 +65,9 @@ 
 
         if (! $user->is('Guest') || ! $user->is('Agent') || ! $user->is('Manager') || ! $user->is('Administrator'))
         {
-          Customer::create($input->except('_token'));
-          session()->flash('message', 'Customer created');
-          return redirect()->to('/customers');
+            Customer::create($input->except('_token'));
+            session()->flash('message', 'Customer created');
+            return redirect()->to('/customers');
         }
 
         return redirect()->back();
@@ -84,10 +84,10 @@ 
         $user = auth()->user();
 
         if(! $user->is('Customer') || ! $user->is('Agent') || ! $user->is('Manager') || ! $user->is('Administrator')) {
-          $data['customer'] = Customer::where('id', $id)->get();
-          return view('customers/edit', $data);
+            $data['customer'] = Customer::where('id', $id)->get();
+            return view('customers/edit', $data);
 
-      }
+        }
 
         return redirect()->back();
     }

Spacing +5 added lines, -5 removed lines

@@ -30,7 +30,7 @@ 
       $user = auth()->user();
 
       if ($user->is('Agent') || $user->is('Manager') || $user->is('Administrator')) {
-        $data['customers'] = Customer::orderBy('status' , 'asc')->paginate(15);
+        $data[ 'customers' ] = Customer::orderBy('status', 'asc')->paginate(15);
         return view('customers/index', $data);
       }
         return redirect()->back();
@@ -46,7 +46,7 @@ 
         $user = auth()->user();
 
         if ($user->is('Guest') || $user->is('Agent') || $user->is('Manager') || $user->is('Administrator')) {
-            $data['countries'] = Countries::all();
+            $data[ 'countries' ] = Countries::all();
             return view('customers/register', $data);
         }
 
@@ -63,7 +63,7 @@ 
     {
         $user = auth()->user();
 
-        if (! $user->is('Guest') || ! $user->is('Agent') || ! $user->is('Manager') || ! $user->is('Administrator'))
+        if (!$user->is('Guest') || !$user->is('Agent') || !$user->is('Manager') || !$user->is('Administrator'))
         {
           Customer::create($input->except('_token'));
           session()->flash('message', 'Customer created');
@@ -83,8 +83,8 @@ 
     {
         $user = auth()->user();
 
-        if(! $user->is('Customer') || ! $user->is('Agent') || ! $user->is('Manager') || ! $user->is('Administrator')) {
-          $data['customer'] = Customer::where('id', $id)->get();
+        if (!$user->is('Customer') || !$user->is('Agent') || !$user->is('Manager') || !$user->is('Administrator')) {
+          $data[ 'customer' ] = Customer::where('id', $id)->get();
           return view('customers/edit', $data);
 
       }

app/Http/Controllers/DepartmentController.php

Doc Comments +3 added lines, -3 removed lines

@@ -49,7 +49,7 @@ 
      * Store a newly created resource in storage.
      *
      * @param  Requests\DepartmentsValidator $input
-     * @return \Illuminate\Http\Response
+     * @return \Illuminate\Http\RedirectResponse
      */
     public function store(Requests\DepartmentsValidator $input)
     {
@@ -86,7 +86,7 @@ 
      *
      * @param  Requests\DepartmentsValidator $input
      * @param  int $id the department id in the database.
-     * @return \Illuminate\Http\Response
+     * @return \Illuminate\Http\RedirectResponse
      */
     public function update(Requests\DepartmentsValidator $input, $id)
     {
@@ -98,7 +98,7 @@ 
      * Remove the specified resource from storage.
      *
      * @param  int  $id the department id in the database.
-     * @return \Illuminate\Http\Response
+     * @return \Illuminate\Http\RedirectResponse
      */
     public function destroy($id)
     {

Unused Use Statements -1 removed lines

@@ -4,7 +4,6 @@
 
 use App\Departments;
 use Illuminate\Http\Request;
-
 use App\Http\Requests;
 
 /**

Spacing +9 added lines, -9 removed lines

@@ -32,7 +32,7 @@ 
      */
     public function index()
     {
-        $data['query'] = Departments::with('managers')->paginate(15);
+        $data[ 'query' ] = Departments::with('managers')->paginate(15);
         return view('departments.index', $data);
     }
 
@@ -55,7 +55,7 @@ 
     public function search(Request $request)
     {
         $term = $request->get('name');
-        $data['query'] = Departments::where('name', 'LIKE', "%$term%")->with('managers')->paginate(15);
+        $data[ 'query' ] = Departments::where('name', 'LIKE', "%$term%")->with('managers')->paginate(15);
         return view('departments.index', $data);
     }
 
@@ -79,7 +79,7 @@ 
      */
     public function show($id)
     {
-        $data['query'] = Departments::find($id);
+        $data[ 'query' ] = Departments::find($id);
         return view('departments.specific', $data);
     }
 
@@ -91,7 +91,7 @@ 
      */
     public function edit($id)
     {
-        $data['query'] = Departments::find($id);
+        $data[ 'query' ] = Departments::find($id);
         return view('departments.update', $data);
     }
 
@@ -128,12 +128,12 @@ 
     public function get_departments()
     {
         $items = Departments::all();
-        $data2 = [];
+        $data2 = [ ];
 
-        foreach($items as $department) {
-            $data2[] = [
-                'value' => $department["id"],
-                'text'  => $department["name"]
+        foreach ($items as $department) {
+            $data2[ ] = [
+                'value' => $department[ "id" ],
+                'text'  => $department[ "name" ]
             ];
         }
 

app/Http/Controllers/ApiKeyController.php

Unused Use Statements -2 removed lines

@@ -2,10 +2,8 @@
 
 namespace App\Http\Controllers;
 
-use App\User;
 use Illuminate\Http\Request;
 use Chrisbjr\ApiGuard\Models\ApiKey;
-use App\Http\Requests;
 
 class ApiKeyController extends Controller
 {

app/Http/Requests/PasswordValidation.php

Spacing +1 added lines, -1 removed lines

@@ -26,6 +26,6 @@
      */
     public function rules()
     {
-        return ['password' => 'required|min:6|confirmed'];
+        return [ 'password' => 'required|min:6|confirmed' ];
     }
 }