Skip to content

Commit d6e8e5c

Browse files
committed
User update validation
1 parent bab19a7 commit d6e8e5c

File tree

1 file changed

+26
-22
lines changed

1 file changed

+26
-22
lines changed

app/Http/Controllers/UserController.php

Lines changed: 26 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,6 @@
22

33
namespace App\Http\Controllers;
44

5-
use App\Http\Requests;
65
use App\User;
76
use Auth;
87
use Bican\Roles\Models\Permission;
@@ -119,19 +118,33 @@ public function getShow($id)
119118
*
120119
* @return JSON success message
121120
*/
122-
public function putShow()
121+
public function putShow(Request $request)
123122
{
124-
$userForm = Input::get('data');
125-
$userId = intval($userForm['id']);
123+
$userForm = array_dot(
124+
app('request')->only(
125+
'data.name',
126+
'data.email',
127+
'data.id'
128+
)
129+
);
130+
131+
$userId = intval($userForm['data.id']);
132+
133+
$user = User::find($userId);
134+
135+
$this->validate($request, [
136+
'data.id' => 'required|integer',
137+
'data.name' => 'required|min:3',
138+
'data.email' => 'required|email|unique:users,email,'.$user->id,
139+
]);
126140

127141
$userData = [
128-
'name' => $userForm['name'],
129-
'email' => $userForm['email'],
142+
'name' => $userForm['data.name'],
143+
'email' => $userForm['data.email'],
130144
];
131145

132146
$affectedRows = User::where('id', '=', $userId)->update($userData);
133147

134-
$user = User::find($userId);
135148
$user->detachAllRoles();
136149

137150
foreach (Input::get('data.role') as $setRole) {
@@ -142,24 +155,15 @@ public function putShow()
142155
}
143156

144157
/**
145-
* Responds to requests to GET /users/admin-profile.
146-
*/
147-
public function getAdminProfile()
148-
{
149-
//
150-
}
151-
152-
/**
153-
* Responds to requests to POST /users/profile.
158+
* Delete User Data.
159+
*
160+
* @return JSON success message
154161
*/
155-
public function postProfile()
156-
{
157-
//
158-
}
159-
160162
public function deleteUser($id)
161163
{
162-
return response()->success(compact('id'));
164+
// $user = User::find($id);
165+
// $user->delete();
166+
return response()->success('success');
163167
}
164168

165169
/**

0 commit comments

Comments
 (0)