refactor(tsconfig): activate strict templates

Author: jeremy-eychenne

package.json

@@ -40,6 +40,7 @@
     "@angular/compiler-cli": "^14.3.0",
     "@angular/language-service": "^14.3.0",
     "@types/arcgis-rest-api": "^10.4.5",
+    "@types/hammerjs": "^2.0.41",
     "@types/jasmine": "~4.3.1",
     "@types/node": "^12.11.1",
     "@types/topojson-specification": "^1.0.2",

projects/ngx-openlayers/src/lib/attribution.component.ts

@@ -11,7 +11,7 @@ export class AttributionComponent implements OnInit {
 
   constructor(private elementRef: ElementRef) {}
 
-  ngOnInit() {
+  ngOnInit(): void {
     this.html = this.elementRef.nativeElement.innerHTML;
     this.instance = new Attribution();
   }

projects/ngx-openlayers/src/lib/attributions.component.ts

@@ -15,7 +15,7 @@ export class AttributionsComponent implements AfterViewInit {
   constructor(@Host() private source: SourceComponent) {}
 
   /* we can do this at the very end */
-  ngAfterViewInit() {
+  ngAfterViewInit(): void {
     if (this.attributions.length) {
       this.instance = this.attributions.map((cmp) => cmp.html);
       // console.log('setting attributions:', this.instance);

projects/ngx-openlayers/src/lib/collectioncoordinates.component.ts

@@ -44,22 +44,22 @@ export class CollectionCoordinatesComponent implements OnChanges, OnInit {
     }
   }
 
-  ngOnInit() {
+  ngOnInit(): void {
     this.map.instance.on('change:view', (e) => this.onMapViewChanged(e));
     this.mapSrid = this.map.instance.getView().getProjection().getCode();
     this.transformCoordinates();
   }
 
-  ngOnChanges(changes: SimpleChanges) {
+  ngOnChanges(changes: SimpleChanges): void {
     this.transformCoordinates();
   }
 
-  private onMapViewChanged(event) {
+  private onMapViewChanged(event): void {
     this.mapSrid = event.target.get(event.key).getProjection().getCode();
     this.transformCoordinates();
   }
 
-  private transformCoordinates() {
+  private transformCoordinates(): void {
     let transformedCoordinates: Coordinate[] | Coordinate[][] | Coordinate[][][];
 
     if (this.srid === this.mapSrid) {

projects/ngx-openlayers/src/lib/controls/attribution.component.ts

@@ -16,14 +16,14 @@ export class ControlAttributionComponent implements OnInit, OnDestroy {
 
   constructor(private map: MapComponent, private element: ElementRef) {}
 
-  ngOnInit() {
+  ngOnInit(): void {
     this.target = this.element.nativeElement;
     // console.log('ol.control.Attribution init: ', this);
     this.instance = new Attribution(this);
     this.map.instance.addControl(this.instance);
   }
 
-  ngOnDestroy() {
+  ngOnDestroy(): void {
     // console.log('removing aol-control-attribution');
     this.map.instance.removeControl(this.instance);
   }

projects/ngx-openlayers/src/lib/controls/control.component.ts

@@ -17,15 +17,15 @@ export class ControlComponent implements OnInit, OnDestroy {
 
   constructor(private map: MapComponent) {}
 
-  ngOnInit() {
+  ngOnInit(): void {
     if (this.content) {
       this.element = this.content.elementRef.nativeElement;
       this.instance = new Control(this);
       this.map.instance.addControl(this.instance);
     }
   }
 
-  ngOnDestroy() {
+  ngOnDestroy(): void {
     if (this.instance) {
       this.map.instance.removeControl(this.instance);
     }

projects/ngx-openlayers/src/lib/controls/default.component.ts

@@ -29,13 +29,13 @@ export class DefaultControlComponent implements OnInit, OnDestroy {
 
   constructor(private map: MapComponent) {}
 
-  ngOnInit() {
+  ngOnInit(): void {
     // console.log('ol.control.defaults init: ', this);
     this.instance = defaults(this);
     this.instance.forEach((c) => this.map.instance.addControl(c));
   }
 
-  ngOnDestroy() {
+  ngOnDestroy(): void {
     // console.log('removing aol-control-defaults');
     this.instance.forEach((c) => this.map.instance.removeControl(c));
   }

projects/ngx-openlayers/src/lib/controls/fullscreen.component.ts

@@ -24,12 +24,12 @@ export class ControlFullScreenComponent implements OnInit, OnDestroy {
     // console.log('instancing aol-control-fullscreen');
   }
 
-  ngOnInit() {
+  ngOnInit(): void {
     this.instance = new FullScreen(this);
     this.map.instance.addControl(this.instance);
   }
 
-  ngOnDestroy() {
+  ngOnDestroy(): void {
     // console.log('removing aol-control-fullscreen');
     this.map.instance.removeControl(this.instance);
   }

projects/ngx-openlayers/src/lib/controls/mouseposition.component.ts

@@ -19,14 +19,14 @@ export class ControlMousePositionComponent implements OnInit, OnDestroy {
 
   constructor(private map: MapComponent, private element: ElementRef) {}
 
-  ngOnInit() {
+  ngOnInit(): void {
     this.target = this.element.nativeElement;
     // console.log('ol.control.MousePosition init: ', this);
     this.instance = new MousePosition(this);
     this.map.instance.addControl(this.instance);
   }
 
-  ngOnDestroy() {
+  ngOnDestroy(): void {
     // console.log('removing aol-control-mouseposition');
     this.map.instance.removeControl(this.instance);
   }

projects/ngx-openlayers/src/lib/controls/overviewmap.component.ts

@@ -30,22 +30,22 @@ export class ControlOverviewMapComponent implements OnInit, OnChanges, OnDestroy
 
   constructor(private map: MapComponent) {}
 
-  ngOnInit() {
+  ngOnInit(): void {
     this.instance = new OverviewMap(this);
     this.map.instance.addControl(this.instance);
   }
 
-  ngOnDestroy() {
+  ngOnDestroy(): void {
     this.map.instance.removeControl(this.instance);
   }
 
-  ngOnChanges(changes: SimpleChanges) {
+  ngOnChanges(changes: SimpleChanges): void {
     if (this.instance != null && changes.hasOwnProperty('view')) {
       this.reloadInstance();
     }
   }
 
-  private reloadInstance() {
+  private reloadInstance(): void {
     this.map.instance.removeControl(this.instance);
     this.instance = new OverviewMap(this);
     this.map.instance.addControl(this.instance);