Fixed validation for frånvaro-report
This commit is contained in:
@@ -1,5 +1,6 @@
|
|||||||
import { AbstractControl, ValidatorFn } from '@angular/forms';
|
import { AbstractControl, ValidatorFn } from '@angular/forms';
|
||||||
import { ANNAN_KAND_ORSAK_ID, ANNAN_ORSAK_ID } from '@msfa-constants/franvaro-reasons';
|
import { ANNAN_KAND_ORSAK_ID, ANNAN_ORSAK_ID } from '@msfa-constants/franvaro-reasons';
|
||||||
|
import { CHARACTER_REGEX } from '@msfa-constants/regex';
|
||||||
import { Franvaro } from '@msfa-models/franvaro.model';
|
import { Franvaro } from '@msfa-models/franvaro.model';
|
||||||
|
|
||||||
const TIME_REGEX = /^([0-1]?[0-9]|2[0-4]):([0-5][0-9])(:[0-5][0-9])?$/;
|
const TIME_REGEX = /^([0-1]?[0-9]|2[0-4]):([0-5][0-9])(:[0-5][0-9])?$/;
|
||||||
@@ -36,11 +37,18 @@ export class FranvaroReportValidator {
|
|||||||
...errors,
|
...errors,
|
||||||
otherKnownReason: 'Känd orsak måste väljas',
|
otherKnownReason: 'Känd orsak måste väljas',
|
||||||
};
|
};
|
||||||
} else if (otherKnownReason === ANNAN_ORSAK_ID && !knownReasonComment) {
|
} else if (otherKnownReason === ANNAN_ORSAK_ID) {
|
||||||
|
if (!knownReasonComment) {
|
||||||
errors = {
|
errors = {
|
||||||
...errors,
|
...errors,
|
||||||
knownReasonComment: 'Beskrivning av frånvaro är obligatorisk',
|
knownReasonComment: 'Beskrivning av frånvaro är obligatorisk',
|
||||||
};
|
};
|
||||||
|
} else if (!CHARACTER_REGEX.test(knownReasonComment)) {
|
||||||
|
errors = {
|
||||||
|
...errors,
|
||||||
|
knownReasonComment: 'Beskrivning av frånvaro har en ogiltig värde',
|
||||||
|
};
|
||||||
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
if (!date) {
|
if (!date) {
|
||||||
|
|||||||
Reference in New Issue
Block a user