Fix validation rules

This commit is contained in:
Ryan Scherler
2020-12-01 22:29:45 -08:00
parent ce864012de
commit 730c6821bc
2 changed files with 13 additions and 19 deletions
+12 -19
View File
@@ -21,26 +21,10 @@ class Account extends Component
public $password_confirmation;
protected $rules = [
'user.name' => 'required|min:2',
'user.email' => [
'required',
'email',
],
'password' => 'nullable|required_with:password_confirmation|min:8|confirmed',
'password_confirmation' => 'nullable|same:password',
'user.name' => 'required',
'user.email' => 'required',
];
public function updated($user)
{
$this->validateOnly($user, [
'user.email' => [
'required',
'email',
Rule::unique('users', 'email')->ignore($this->user->id),
],
]);
}
public function updatedAvatar($value)
{
$extension = pathinfo($value->getFilename(), PATHINFO_EXTENSION);
@@ -70,7 +54,16 @@ class Account extends Component
public function submit()
{
$this->validate();
$this->validate([
'user.name' => 'required|min:2',
'user.email' => [
'required',
'email',
Rule::unique('users', 'email')->ignore($this->user->id)
],
'password' => 'nullable|required_with:password_confirmation|min:8|confirmed',
'password_confirmation' => 'nullable|same:password',
]);
if ($this->avatar) {
$this->user->avatar = $this->avatar->store('avatars', config('filament.storage_disk'));
+1
View File
@@ -54,6 +54,7 @@ class ProfileAccountTest extends TestCase
$this->accountComponent()
->set('user.email', 'test@example.com')
->call('submit')
->assertHasErrors(['user.email' => 'unique']);
}