Skip to content

Commit

Permalink
Merge pull request #2100 from ORCID/DoNotValidateISBN
Browse files Browse the repository at this point in the history
Still not working
  • Loading branch information
leomendoza123 authored Nov 30, 2023
2 parents 102b0e2 + 61aa9cf commit 4304839
Showing 1 changed file with 11 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,7 @@ import { SnackbarService } from 'src/app/cdk/snackbar/snackbar.service'
import { WorkIdentifiers } from 'src/app/shared/validators/work-identifiers/work-identifiers.validator'
import { workCitationValidator } from 'src/app/shared/validators/citation/work-citation.validator'
import { translatedTitleValidator } from 'src/app/shared/validators/translated-title/translated-title.validator'
import { merge, Subject } from 'rxjs'
import { merge, Subject, of } from 'rxjs'
import { RecordService } from 'src/app/core/record/record.service'

@Component({
Expand Down Expand Up @@ -266,8 +266,15 @@ export class WorkFormComponent implements OnInit {
formGroup: UntypedFormGroup,
externalIdentifierType: string
): AsyncValidatorFn {
return (control: AbstractControl) => {
return this._workService
return (control: AbstractControl) => {
if(externalIdentifierType === 'isbn') {
if(!(control.value?.length === 10 || control.value?.length === 13)) {
return of({ validFormat: true, })
} else {
return of({})
}
} else {
return this._workService
.validateWorkIdTypes(externalIdentifierType, control.value)
.pipe(
map((value) => {
Expand Down Expand Up @@ -312,6 +319,7 @@ export class WorkFormComponent implements OnInit {
}
})
)
}
}
}

Expand Down

0 comments on commit 4304839

Please sign in to comment.