manuel/converter
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

Add component tests #8

Merged
manuel merged 11 commits from feature/collapsible-intermediate-input into develop 2018-09-07 12:49:20 +02:00
Conversation 0 Commits 11 Files changed 20 +6 -11
1 changed files with 6 additions and 11 deletions
Download patch file Download diff file Expand all files Collapse all files
Showing only changes of commit 86574e8616 - Show all commits

15
src/app/converter-registry.service.ts
View file

@ -36,12 +36,7 @@ export class ConverterRegistryService {
}
public getConverter(id: string): Converter {
for (let i = 0; i < this.converters.length; i++) {
if (this.converters[i].getId() === id) {
return this.converters[i];
}
}
return undefined;
return this.converters.find((converter: Converter): boolean => converter.getId() === id);
}
private init(): void {
@ -67,11 +62,11 @@ export class ConverterRegistryService {
}
private registerConverter(converter: Converter): void {
this.converters.forEach((c: Converter) => {
if (c.getId() === converter.getId()) {
throw new Error('Converter-ID ' + converter.getId() + ' is already registered!');
// Don't allow duplicate registration of the same converter id
if (this.converters.some((c: Converter): boolean => c.getId() === converter.getId())) {
throw new Error(`Converter-ID ${converter.getId()} is already registered!`);
}
});
this.converters.push(converter);
}
}