murrant / librenmsv2

app/Http/Controllers/UserController.php

Doc Comments +2 added lines, -3 removed lines

@@ -52,7 +52,7 @@ 
      * Store a newly created resource in storage.
      *
      * @param CreateUserRequest $request
-     * @return \Illuminate\Http\Response
+     * @return \Illuminate\Http\JsonResponse
      */
     public function store(CreateUserRequest $request)
     {
@@ -95,7 +95,6 @@ 
      * @param UpdateUserRequest|Request $request
      * @param $user_id
      * @param $type Type of update info|password|adddevice|removedevice|addpor|removeport
-     * @param  int $id
      * @return \Illuminate\Http\Response
      */
     public function update(UpdateUserRequest $request, $user_id)
@@ -111,7 +110,7 @@ 
      * Remove the specified resource from storage.
      *
      * @param  int $user_id
-     * @return \Illuminate\Http\Response
+     * @return \Illuminate\Http\JsonResponse
      */
     public function destroy(DeleteUserRequest $request, $user_id)
     {

app/Http/Controllers/UserDeviceController.php

Unused Use Statements -1 removed lines

@@ -25,7 +25,6 @@
 
 namespace App\Http\Controllers;
 
-use App\Http\Requests;
 use App\Http\Requests\AdminOnlyRequest;
 use App\Models\Device;
 use App\Models\User;

Spacing +2 added lines, -2 removed lines

@@ -48,11 +48,11 @@
         $user = User::find($user_id);
 
         $device_ids = $request->input('devices');
-        if(count($device_ids) == 0) {
+        if (count($device_ids) == 0) {
             return redirect()->back();
         }
         
-        foreach($device_ids as $device_id) {
+        foreach ($device_ids as $device_id) {
             $device = Device::find($device_id);
             $user->devices()->attach($device);
         }

app/Http/Controllers/UserPortController.php

Unused Use Statements -1 removed lines

@@ -25,7 +25,6 @@
 
 namespace App\Http\Controllers;
 
-use App\Http\Requests;
 use App\Http\Requests\AdminOnlyRequest;
 use App\Models\Port;
 use App\Models\User;

Spacing +2 added lines, -2 removed lines

@@ -54,11 +54,11 @@
         $user = User::find($user_id);
 
         $port_ids = $request->input('ports');
-        if(count($port_ids) == 0) {
+        if (count($port_ids) == 0) {
             return redirect()->back();
         }
         
-        foreach($port_ids as $port_id) {
+        foreach ($port_ids as $port_id) {
             $port = Port::find($port_id);
             $user->ports()->attach($port);
         }

app/Http/Requests/UpdateUserRequest.php

Indentation +1 added lines, -1 removed lines

@@ -55,7 +55,7 @@
     {
         if ($this->input('password')) {
             $rules = ['password'              => 'required|min:5|max:255',
-                      'password_confirmation' => 'required|same:password',
+                        'password_confirmation' => 'required|same:password',
             ];
             if (!Auth::user()->isAdmin()) {
                 $rules['current_password'] = 'required|password:user_id';

app/Models/User.php

Spacing +1 added lines, -1 removed lines

@@ -122,7 +122,7 @@
      * Encrypt passwords before saving
      * @param $password
      */
-    public function setPasswordAttribute($password){
+    public function setPasswordAttribute($password) {
         $this->attributes['password'] = bcrypt($password);
     }
 }