Skip to content

Commit f094c27

Browse files
committed
users fixup when update user, some UI improvements
1 parent 9b77904 commit f094c27

File tree

3 files changed

+31
-13
lines changed

3 files changed

+31
-13
lines changed

app/Http/Controllers/Users/UserController.php

Lines changed: 13 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -58,18 +58,24 @@ public function update (Request $request)
5858
$this->validate($request, [
5959
'name' => 'required|string',
6060
'email' => 'required|email|unique:users,email,'.$request->id,
61-
'password' => 'string'
61+
'password' => 'string|nullable'
6262
]);
6363

6464
$user = User::find($request->id);
6565

66-
$user->name = $request->name;
67-
$user->email = $request->email;
68-
$user->password = Hash::make($request->password);
69-
$user->save();
66+
if ($user->name != $request->name) {
67+
$avatar = Avatar::create($request->name)->getImageObject()->encode('png');
68+
Storage::put('avatars/'.$user->id.'/avatar.png', (string) $avatar);
69+
$user->name = $request->name;
70+
}
71+
if ($user->email != $request->email) {
72+
$user->email = $request->email;
73+
}
74+
if ($request->password != '') {
75+
$user->password = Hash::make($request->password);
76+
}
7077

71-
$avatar = Avatar::create($user->name)->getImageObject()->encode('png');
72-
Storage::put('avatars/'.$user->id.'/avatar.png', (string) $avatar);
78+
$user->save();
7379

7480
return $user;
7581
}

resources/js/components/user/Create.vue

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,9 +2,6 @@
22
<div class="card">
33
<div class="card-header">
44
<i class="fas fa-plus"></i> New User
5-
<button class="btn btn-primary float-right" type="button" :disabled="submiting" @click="create" >
6-
<i class="fas fa-spinner fa-spin" v-if="submiting"></i> Save
7-
</button>
85
</div>
96
<div class="card-body">
107
<form class="form-horizontal">
@@ -31,6 +28,15 @@
3128
</div>
3229
</form>
3330
</div>
31+
<div class="card-footer">
32+
<div class="form-group row">
33+
<div class="col-md-9 offset-md-3">
34+
<button class="btn btn-primary float-right" type="button" :disabled="submiting" @click="create" >
35+
<i class="fas fa-spinner fa-spin" v-if="submiting"></i> Save
36+
</button>
37+
</div>
38+
</div>
39+
</div>
3440
</div>
3541
</template>
3642

resources/js/components/user/Edit.vue

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,9 +2,6 @@
22
<div class="card">
33
<div class="card-header">
44
<i class="fas fa-pencil-alt"></i> Edit User
5-
<button class="btn btn-primary float-right" type="button" :disabled="submiting" @click="update" >
6-
<i class="fas fa-spinner fa-spin" v-if="submiting"></i> Save
7-
</button>
85
</div>
96
<div class="card-body">
107
<form class="form-horizontal">
@@ -51,6 +48,15 @@
5148
</div>
5249
</form>
5350
</div>
51+
<div class="card-footer">
52+
<div class="form-group row">
53+
<div class="col-md-9 offset-md-3">
54+
<button class="btn btn-primary float-right" type="button" :disabled="submiting" @click="update" >
55+
<i class="fas fa-spinner fa-spin" v-if="submiting"></i> Save
56+
</button>
57+
</div>
58+
</div>
59+
</div>
5460
</div>
5561
</template>
5662

0 commit comments

Comments
 (0)