diff options
Diffstat (limited to 'client')
2 files changed, 24 insertions, 9 deletions
diff --git a/client/src/app/+my-account/my-account-settings/my-account-change-password/my-account-change-password.component.html b/client/src/app/+my-account/my-account-settings/my-account-change-password/my-account-change-password.component.html index 767ef0336..913b570cb 100644 --- a/client/src/app/+my-account/my-account-settings/my-account-change-password/my-account-change-password.component.html +++ b/client/src/app/+my-account/my-account-settings/my-account-change-password/my-account-change-password.component.html | |||
@@ -6,6 +6,7 @@ | |||
6 | <input | 6 | <input |
7 | type="password" id="new-password" i18n-placeholder placeholder="New password" | 7 | type="password" id="new-password" i18n-placeholder placeholder="New password" |
8 | formControlName="new-password" [ngClass]="{ 'input-error': formErrors['new-password'] }" | 8 | formControlName="new-password" [ngClass]="{ 'input-error': formErrors['new-password'] }" |
9 | (change)="validateNewPassword()" (blur)="printAnError()" | ||
9 | > | 10 | > |
10 | <div *ngIf="formErrors['new-password']" class="form-error"> | 11 | <div *ngIf="formErrors['new-password']" class="form-error"> |
11 | {{ formErrors['new-password'] }} | 12 | {{ formErrors['new-password'] }} |
@@ -13,8 +14,8 @@ | |||
13 | 14 | ||
14 | <input | 15 | <input |
15 | type="password" id="new-confirmed-password" i18n-placeholder placeholder="Confirm new password" | 16 | type="password" id="new-confirmed-password" i18n-placeholder placeholder="Confirm new password" |
16 | formControlName="new-confirmed-password" | 17 | formControlName="new-confirmed-password" (change)="validateNewPassword()" (blur)="printAnError()" |
17 | > | 18 | > |
18 | 19 | ||
19 | <input type="submit" i18n-value value="Change password" [disabled]="!form.valid"> | 20 | <input type="submit" i18n-value value="Change password" [disabled]="!form.valid || unsendable"> |
20 | </form> | 21 | </form> |
diff --git a/client/src/app/+my-account/my-account-settings/my-account-change-password/my-account-change-password.component.ts b/client/src/app/+my-account/my-account-settings/my-account-change-password/my-account-change-password.component.ts index 7be7aabc2..0707d8f9a 100644 --- a/client/src/app/+my-account/my-account-settings/my-account-change-password/my-account-change-password.component.ts +++ b/client/src/app/+my-account/my-account-settings/my-account-change-password/my-account-change-password.component.ts | |||
@@ -12,6 +12,7 @@ import { UserValidatorsService } from '@app/shared/forms/form-validators/user-va | |||
12 | }) | 12 | }) |
13 | export class MyAccountChangePasswordComponent extends FormReactive implements OnInit { | 13 | export class MyAccountChangePasswordComponent extends FormReactive implements OnInit { |
14 | error: string = null | 14 | error: string = null |
15 | unsendable = true // default to true to not have to not the if in change password | ||
15 | 16 | ||
16 | constructor ( | 17 | constructor ( |
17 | protected formValidatorService: FormValidatorService, | 18 | protected formValidatorService: FormValidatorService, |
@@ -30,18 +31,31 @@ export class MyAccountChangePasswordComponent extends FormReactive implements On | |||
30 | }) | 31 | }) |
31 | } | 32 | } |
32 | 33 | ||
33 | changePassword () { | 34 | validateNewPassword () { |
34 | const newPassword = this.form.value['new-password'] | 35 | if (this.form.value['new-password'] && this.form.value['new-confirmed-password']) { |
35 | const newConfirmedPassword = this.form.value['new-confirmed-password'] | 36 | if (this.form.value['new-password'] === this.form.value['new-confirmed-password']) { |
36 | 37 | this.error = null | |
37 | this.error = null | 38 | this.unsendable = false |
39 | return | ||
40 | } | ||
41 | } | ||
42 | this.unsendable = true | ||
43 | } | ||
38 | 44 | ||
39 | if (newPassword !== newConfirmedPassword) { | 45 | printAnError () { |
46 | console.log(this.unsendable) | ||
47 | this.validateNewPassword() | ||
48 | if (this.unsendable) { | ||
40 | this.error = this.i18n('The new password and the confirmed password do not correspond.') | 49 | this.error = this.i18n('The new password and the confirmed password do not correspond.') |
50 | } | ||
51 | } | ||
52 | |||
53 | changePassword () { | ||
54 | if (this.unsendable) { | ||
41 | return | 55 | return |
42 | } | 56 | } |
43 | 57 | ||
44 | this.userService.changePassword(newPassword).subscribe( | 58 | this.userService.changePassword(this.form.value['new-password']).subscribe( |
45 | () => this.notificationsService.success(this.i18n('Success'), this.i18n('Password updated.')), | 59 | () => this.notificationsService.success(this.i18n('Success'), this.i18n('Password updated.')), |
46 | 60 | ||
47 | err => this.error = err.message | 61 | err => this.error = err.message |