Skip to content

Commit 43c0634

Browse files
authored
Merge pull request #34 from Lemoncode/update_existing_samples_in_order_to_use_new_chain_pattern_sugar
Update existing samples in order to use new chain pattern sugar
2 parents 0952fff + 2f2b5f0 commit 43c0634

File tree

2 files changed

+24
-27
lines changed

2 files changed

+24
-27
lines changed

lib/lcformvalidation.d.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
formFieldName: string;
55
vmFieldName: string;
66
}
7-
7+
88
export class FieldValidation {
99
validationFn: (vm, value) => Promise<FieldValidationResult>;
1010
filter: any;
@@ -31,8 +31,8 @@
3131
isFormPristine(): boolean;
3232
validateFullForm(vm: any): Promise<FormValidationResult>;
3333
triggerFieldValidation(vm: any, key: string, value: any, filter?: any): Promise<FieldValidationResult>;
34-
addFieldValidation(key : string, validation : (vm, value) => FieldValidationResult, filter? : any);
35-
addFieldValidationAsync(key : string, validation : (vm, value) => Promise<FieldValidationResult>, filter? : any);
34+
addFieldValidation(key : string, validation : (vm, value) => FieldValidationResult, filter? : any): IValidationEngine;
35+
addFieldValidationAsync(key : string, validation : (vm, value) => Promise<FieldValidationResult>, filter? : any): IValidationEngine;
3636
addFormValidation(validation : (vm) => FieldValidationResult);
3737
addFormValidationAsync(validation : (vm) => Promise<FieldValidationResult>);
3838
isValidationInProgress(): boolean;

samples/react/01 SignupForm/src/components/sampleSignupForm/validations/signupFormValidation.ts

Lines changed: 21 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -12,30 +12,27 @@ class SignupFormValidation extends BaseFormValidation {
1212
{formFieldName: 'login', vmFieldName: 'login'}
1313
]);
1414

15-
this._validationEngine.addFieldValidation('password',
16-
(vm, value) : FieldValidationResult => {
17-
return this.requiredValidationHandler(vm, value);
18-
});
19-
20-
this._validationEngine.addFieldValidation('confirmPassword',
21-
(vm, value) : FieldValidationResult => {
22-
return this.passwordAndConfirmPasswordValidationHandler(vm, value);
23-
});
24-
25-
this._validationEngine.addFieldValidation('confirmPassword',
26-
(vm, value) : FieldValidationResult => {
27-
return this.requiredValidationHandler(vm, value);
28-
});
29-
30-
this._validationEngine.addFieldValidationAsync('login',
31-
(vm, value) : Promise<FieldValidationResult> => {
32-
return this.loginExistOnGitHubValidationHandler(vm, value);
33-
},{ OnBlur : true });
34-
35-
this._validationEngine.addFieldValidation('login',
36-
(vm, value) : FieldValidationResult => {
37-
return this.requiredValidationHandler(vm, value);
38-
},{ OnChange: true, OnBlur : true });
15+
this._validationEngine
16+
.addFieldValidation('password',
17+
(vm, value) : FieldValidationResult => {
18+
return this.requiredValidationHandler(vm, value);
19+
})
20+
.addFieldValidation('confirmPassword',
21+
(vm, value) : FieldValidationResult => {
22+
return this.passwordAndConfirmPasswordValidationHandler(vm, value);
23+
})
24+
.addFieldValidation('confirmPassword',
25+
(vm, value) : FieldValidationResult => {
26+
return this.requiredValidationHandler(vm, value);
27+
})
28+
.addFieldValidationAsync('login',
29+
(vm, value) : Promise<FieldValidationResult> => {
30+
return this.loginExistOnGitHubValidationHandler(vm, value);
31+
},{ OnBlur : true })
32+
.addFieldValidation('login',
33+
(vm, value) : FieldValidationResult => {
34+
return this.requiredValidationHandler(vm, value);
35+
},{ OnChange: true, OnBlur : true });
3936
}
4037

4138
requiredValidationHandler(vm : any, value: any) : FieldValidationResult {

0 commit comments

Comments
 (0)