Skip to content

Commit

Permalink
Use "noImplicitOverride": true as default (openmediavault#1762)
Browse files Browse the repository at this point in the history
Signed-off-by: Volker Theile <votdev@gmx.de>
  • Loading branch information
votdev committed May 6, 2024
1 parent d6bd2b8 commit 5996430
Show file tree
Hide file tree
Showing 14 changed files with 24 additions and 25 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -82,7 +82,7 @@ export class DatatablePageComponent extends AbstractPageComponent<DatatablePageC
/**
* Append the current selection to the page context.
*/
get pageContext(): PageContext {
override get pageContext(): PageContext {
const result = _.merge(
{
_selected: this.selection.selected
Expand Down Expand Up @@ -351,7 +351,7 @@ export class DatatablePageComponent extends AbstractPageComponent<DatatablePageC
}
}

protected sanitizeConfig() {
protected override sanitizeConfig() {
_.defaultsDeep(this.config, {
columnMode: 'flex',
hasActionBar: true,
Expand Down Expand Up @@ -409,7 +409,7 @@ export class DatatablePageComponent extends AbstractPageComponent<DatatablePageC
}
}

protected onRouteParams() {
protected override onRouteParams() {
// Format tokenized configuration properties.
this.formatConfig([
'title',
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -111,7 +111,7 @@ export class FormPageComponent
/**
* Append the current page mode. This can be editing or creating.
*/
get pageContext(): PageContext {
override get pageContext(): PageContext {
return _.merge(
{
_editing: this.editing
Expand All @@ -120,7 +120,7 @@ export class FormPageComponent
);
}

ngOnInit(): void {
override ngOnInit(): void {
super.ngOnInit();
// Flatten all form field configurations into an array to be able to
// iterate over them easily.
Expand All @@ -144,7 +144,7 @@ export class FormPageComponent
});
}

ngAfterViewInit(): void {
override ngAfterViewInit(): void {
super.ngAfterViewInit();
// Process all specified constraints per button.
if (_.some(this.config.buttons, (button) => _.isPlainObject(button.enabledConstraint))) {
Expand Down Expand Up @@ -446,7 +446,7 @@ export class FormPageComponent
}
}

protected sanitizeConfig() {
protected override sanitizeConfig() {
_.defaultsDeep(this.config, {
buttonAlign: 'end',
buttons: []
Expand Down Expand Up @@ -494,7 +494,7 @@ export class FormPageComponent
this.config.icon = _.get(Icon, this.config.icon, this.config.icon);
}

protected onRouteParams() {
protected override onRouteParams() {
const allFields = flattenFormFieldConfig(this.config.fields);
// Format tokenized configuration properties.
this.formatConfig(['title', 'subTitle', 'request.get.method', 'request.get.params']);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ import { Icon } from '~/app/shared/enum/icon.enum';
styleUrls: ['./form-datatable.component.scss']
})
export class FormDatatableComponent extends AbstractFormFieldComponent {
protected sanitizeConfig(): void {
protected override sanitizeConfig(): void {
super.sanitizeConfig();
if (this.config.hasSearchField || this.config.actions.length) {
this.config.hasActionBar = true;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,6 @@ import { catchError, concatMap, map, startWith, takeUntil, tap, toArray } from '

import { AbstractFormFieldComponent } from '~/app/core/components/intuition/form/components/abstract-form-field-component';
import { Unsubscribe } from '~/app/decorators';
import { Icon } from '~/app/shared/enum/icon.enum';
import { RpcService } from '~/app/shared/services/rpc.service';

@Component({
Expand All @@ -45,7 +44,6 @@ export class FormFolderbrowserComponent
@Unsubscribe()
private subscriptions = new Subscription();

icon = Icon;
isOpen = false;
folders: string[] = [];
filteredFolders$: Observable<string[]>;
Expand Down Expand Up @@ -83,7 +81,7 @@ export class FormFolderbrowserComponent
return this.joinPaths(this.currentPaths);
}

ngOnInit(): void {
override ngOnInit(): void {
super.ngOnInit();
this.viewportRuler
.change()
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ export class FormSelectComponent extends AbstractFormFieldComponent implements O
super();
}

ngOnInit(): void {
override ngOnInit(): void {
super.ngOnInit();
this.doLoadStore();

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@ export class FormSharedfolderSelectComponent extends FormSelectComponent {
});
}

protected sanitizeConfig(): void {
protected override sanitizeConfig(): void {
super.sanitizeConfig();
_.merge(this.config, {
valueField: 'uuid',
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ export class FormSshcertSelectComponent extends FormSelectComponent {
});
}

protected sanitizeConfig(): void {
protected override sanitizeConfig(): void {
super.sanitizeConfig();
_.merge(this.config, {
valueField: 'uuid',
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ export class FormSslcertSelectComponent extends FormSelectComponent {
});
}

protected sanitizeConfig(): void {
protected override sanitizeConfig(): void {
super.sanitizeConfig();
_.merge(this.config, {
valueField: 'uuid',
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ export class FormTextInputComponent extends AbstractFormFieldComponent implement
super();
}

ngOnInit(): void {
override ngOnInit(): void {
super.ngOnInit();
this.abstractControl = this.formGroup.get(this.config.name);
this.filteredOptions$ = this.abstractControl.valueChanges.pipe(
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -70,7 +70,7 @@ export class RrdPageComponent extends AbstractPageComponent<RrdPageConfig> imple
});
}

ngOnInit(): void {
override ngOnInit(): void {
super.ngOnInit();
this.time = unixTimeStamp();
if (this.config?.store) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -64,7 +64,7 @@ export class SelectionListPageComponent
super(activatedRoute, authSessionService, router);
}

ngOnInit(): void {
override ngOnInit(): void {
super.ngOnInit();
this.loadData();
}
Expand Down Expand Up @@ -156,7 +156,7 @@ export class SelectionListPageComponent
});
}

protected sanitizeConfig() {
protected override sanitizeConfig() {
_.defaultsDeep(this.config, {
hasSelectAllButton: false,
buttonAlign: 'end',
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ export class TabsPageComponent extends AbstractPageComponent<TabsPageConfig> {
super(activatedRoute, authSessionService, router);
}

protected sanitizeConfig() {
protected override sanitizeConfig() {
_.defaultsDeep(this.config, {
singleRoute: true,
tabs: []
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,7 @@ export class TextPageComponent
super(activatedRoute, authSessionService, router);
}

ngOnInit(): void {
override ngOnInit(): void {
super.ngOnInit();
this.reloadSubscription = timer(
0,
Expand All @@ -72,7 +72,7 @@ export class TextPageComponent
});
}

ngOnDestroy(): void {
override ngOnDestroy(): void {
this.reloadSubscription?.unsubscribe();
super.ngOnDestroy();
}
Expand Down Expand Up @@ -117,7 +117,7 @@ export class TextPageComponent
}
}

protected sanitizeConfig() {
protected override sanitizeConfig() {
_.defaultsDeep(this.config, {
autoReload: false,
hasReloadButton: false,
Expand All @@ -143,7 +143,7 @@ export class TextPageComponent
});
}

protected onRouteParams() {
protected override onRouteParams() {
// Format tokenized configuration properties.
this.formatConfig(['title', 'subTitle', 'request.get.method', 'request.get.params']);
}
Expand Down
1 change: 1 addition & 0 deletions deb/openmediavault/workbench/tsconfig.json
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@
"noUnusedLocals": true,
"noUnusedParameters": false,
"noFallthroughCasesInSwitch": true,
"noImplicitOverride": true,
"noImplicitThis": true,
"noImplicitReturns": true,
"noImplicitAny": false,
Expand Down

0 comments on commit 5996430

Please sign in to comment.