Skip to content

Commit

Permalink
fix: enforce noSpecialChars for firstName and lastName
Browse files Browse the repository at this point in the history
  • Loading branch information
dhhyi committed Aug 21, 2020
1 parent 88b6af8 commit f01c2de
Show file tree
Hide file tree
Showing 6 changed files with 17 additions and 12 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ <h1>{{ 'account.update_profile.heading' | translate }}</h1>
label="account.update_profile.firstname.label"
[errorMessages]="{
required: 'account.firstname.error.required',
noSpecialChars: 'error.forbiddensymbols'
noSpecialChars: 'account.name.error.forbidden.chars'
}"
></ish-input>

Expand All @@ -30,7 +30,7 @@ <h1>{{ 'account.update_profile.heading' | translate }}</h1>
label="account.update_profile.lastname.label"
[errorMessages]="{
required: 'account.lastname.error.required',
noSpecialChars: 'error.forbiddensymbols'
noSpecialChars: 'account.name.error.forbidden.chars'
}"
></ish-input>

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@ import { Injectable } from '@angular/core';
import { FormBuilder, Validators } from '@angular/forms';

import { AddressFormFactory } from 'ish-shared/address-forms/components/address-form/address-form.factory';
import { SpecialValidators } from 'ish-shared/forms/validators/special-validators';

@Injectable()
export class AddressFormDEFactory extends AddressFormFactory {
Expand All @@ -14,8 +15,8 @@ export class AddressFormDEFactory extends AddressFormFactory {
group() {
return this.fb.group({
title: [''],
firstName: ['', Validators.required],
lastName: ['', Validators.required],
firstName: ['', [Validators.required, SpecialValidators.noSpecialChars]],
lastName: ['', [Validators.required, SpecialValidators.noSpecialChars]],
addressLine1: ['', Validators.required],
addressLine2: [],
addressLine3: [],
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@ import { Injectable } from '@angular/core';
import { FormBuilder, Validators } from '@angular/forms';

import { AddressFormFactory } from 'ish-shared/address-forms/components/address-form/address-form.factory';
import { SpecialValidators } from 'ish-shared/forms/validators/special-validators';

@Injectable()
export class AddressFormDefaultFactory extends AddressFormFactory {
Expand All @@ -13,8 +14,8 @@ export class AddressFormDefaultFactory extends AddressFormFactory {

group() {
return this.fb.group({
firstName: ['', Validators.required],
lastName: ['', Validators.required],
firstName: ['', [Validators.required, SpecialValidators.noSpecialChars]],
lastName: ['', [Validators.required, SpecialValidators.noSpecialChars]],
title: [''],
addressLine1: ['', Validators.required],
addressLine2: [],
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@ import { Injectable } from '@angular/core';
import { FormBuilder, Validators } from '@angular/forms';

import { AddressFormFactory } from 'ish-shared/address-forms/components/address-form/address-form.factory';
import { SpecialValidators } from 'ish-shared/forms/validators/special-validators';

@Injectable()
export class AddressFormFRFactory extends AddressFormFactory {
Expand All @@ -14,8 +15,8 @@ export class AddressFormFRFactory extends AddressFormFactory {
group() {
return this.fb.group({
title: [''],
firstName: ['', Validators.required],
lastName: ['', Validators.required],
firstName: ['', [Validators.required, SpecialValidators.noSpecialChars]],
lastName: ['', [Validators.required, SpecialValidators.noSpecialChars]],
addressLine1: ['', Validators.required],
addressLine2: [],
postalCode: ['', [Validators.required, Validators.pattern('[0-9]{5}')]],
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@ import { Injectable } from '@angular/core';
import { FormBuilder, Validators } from '@angular/forms';

import { AddressFormFactory } from 'ish-shared/address-forms/components/address-form/address-form.factory';
import { SpecialValidators } from 'ish-shared/forms/validators/special-validators';

@Injectable()
export class AddressFormGBFactory extends AddressFormFactory {
Expand All @@ -14,8 +15,8 @@ export class AddressFormGBFactory extends AddressFormFactory {
group() {
return this.fb.group({
title: [''],
firstName: ['', Validators.required],
lastName: ['', Validators.required],
firstName: ['', [Validators.required, SpecialValidators.noSpecialChars]],
lastName: ['', [Validators.required, SpecialValidators.noSpecialChars]],
addressLine1: ['', Validators.required],
addressLine2: [],
addressLine3: [],
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@ import { Injectable } from '@angular/core';
import { FormBuilder, Validators } from '@angular/forms';

import { AddressFormFactory } from 'ish-shared/address-forms/components/address-form/address-form.factory';
import { SpecialValidators } from 'ish-shared/forms/validators/special-validators';

@Injectable()
export class AddressFormUSFactory extends AddressFormFactory {
Expand All @@ -13,8 +14,8 @@ export class AddressFormUSFactory extends AddressFormFactory {

group() {
return this.fb.group({
firstName: ['', Validators.required],
lastName: ['', Validators.required],
firstName: ['', [Validators.required, SpecialValidators.noSpecialChars]],
lastName: ['', [Validators.required, SpecialValidators.noSpecialChars]],
addressLine1: ['', Validators.required],
addressLine2: [],
city: ['', Validators.required],
Expand Down

0 comments on commit f01c2de

Please sign in to comment.