feat(openlayers): migrate to 5 version

BREAKING CHANGE: Migration from `openlayers` package to `ol`

Author: samuel-girard

.gitignore

@@ -31,6 +31,7 @@ node_modules
 /libpeerconnection.log
 npm-debug.log
 yarn-error.log
+yarn.lock
 testem.log
 /typings
 

angular.json

@@ -48,7 +48,7 @@
             ],
             "styles": [
               "src/styles.css",
-              "projects/ngx-openlayers/node_modules/openlayers/dist/ol.css"
+              "node_modules/ol/ol.css"
             ],
             "scripts": []
           },

package.json

@@ -27,6 +27,7 @@
     "geojson": "^0.5.0",
     "hammer-timejs": "^1.1.0",
     "hammerjs": "^2.0.8",
+    "ol": "^5.3.1",
     "rxjs": "6.3.3",
     "zone.js": "^0.8.26"
   },

projects/ngx-openlayers/ng-package.json

@@ -2,9 +2,6 @@
   "$schema": "../../node_modules/ng-packagr/ng-package.schema.json",
   "dest": "../../dist/ngx-openlayers",
   "deleteDestPath": false,
-  "whitelistedNonPeerDependencies": [
-   "openlayers", "@types/openlayers"
-  ],
   "lib": {
     "entryFile": "src/public_api.ts"
   }

projects/ngx-openlayers/package.json

@@ -9,10 +9,8 @@
   },
   "peerDependencies": {
     "@angular/common": "^6.0.0-rc.0 || ^6.0.0 || ^7.0.0",
-    "@angular/core": "^6.0.0-rc.0 || ^6.0.0 || ^7.0.0"
+    "@angular/core": "^6.0.0-rc.0 || ^6.0.0 || ^7.0.0",
+    "ol": "^5.3.1"
   },
-  "dependencies": {
-    "@types/openlayers": "4.3.x || 4.4.x",
-    "openlayers": "4.3.x || 4.4.x"
-  }
+  "dependencies": {}
 }

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

@@ -1,5 +1,5 @@
 import { Component, ElementRef, OnInit } from '@angular/core';
-import { Attribution } from 'openlayers';
+import { Attribution } from 'ol/control';
 
 @Component({
   selector: 'aol-attribution',

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

@@ -1,5 +1,5 @@
 import { AfterViewInit, Component, ContentChildren, Host, QueryList } from '@angular/core';
-import { Attribution } from 'openlayers';
+import { Attribution } from 'ol/control';
 import { SourceComponent } from './sources/source.component';
 import { AttributionComponent } from './attribution.component';
 

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

@@ -1,5 +1,5 @@
 import { Component, ElementRef, Input, OnDestroy, OnInit } from '@angular/core';
-import { control } from 'openlayers';
+import { Attribution } from 'ol/control';
 import { MapComponent } from '../map.component';
 
 @Component({
@@ -8,7 +8,7 @@ import { MapComponent } from '../map.component';
 })
 export class ControlAttributionComponent implements OnInit, OnDestroy {
   public componentType = 'control';
-  instance: control.Attribution;
+  instance: Attribution;
   target: Element;
   @Input()
   collapsible: boolean;
@@ -18,7 +18,7 @@ export class ControlAttributionComponent implements OnInit, OnDestroy {
   ngOnInit() {
     this.target = this.element.nativeElement;
     // console.log('ol.control.Attribution init: ', this);
-    this.instance = new control.Attribution(this);
+    this.instance = new Attribution(this);
     this.map.instance.addControl(this.instance);
   }
 

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

@@ -1,5 +1,5 @@
 import { Component, ContentChild, OnDestroy, OnInit } from '@angular/core';
-import { control } from 'openlayers';
+import { Control } from 'ol/control';
 import { MapComponent } from '../map.component';
 import { ContentComponent } from '../content.component';
 
@@ -11,7 +11,7 @@ import { ContentComponent } from '../content.component';
 })
 export class ControlComponent implements OnInit, OnDestroy {
   public componentType = 'control';
-  instance: control.Control;
+  instance: Control;
   element: Element;
   @ContentChild(ContentComponent)
   content: ContentComponent;
@@ -21,7 +21,7 @@ export class ControlComponent implements OnInit, OnDestroy {
   ngOnInit() {
     if (this.content) {
       this.element = this.content.elementRef.nativeElement;
-      this.instance = new control.Control(this);
+      this.instance = new Control(this);
       this.map.instance.addControl(this.instance);
     }
   }

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

@@ -1,31 +1,36 @@
 import { Component, OnDestroy, OnInit, Input } from '@angular/core';
-import { control, Collection } from 'openlayers';
+import { Control, defaults } from 'ol/control';
+import { Collection } from 'ol';
+import { Options as AttributionOptions } from 'ol/control/Attribution';
+import { Options as RotateOptions } from 'ol/control/Rotate';
+import { Options as ZoomOptions } from 'ol/control/Zoom';
+
 import { MapComponent } from '../map.component';
 
 @Component({
   selector: 'aol-control-defaults',
   template: '',
 })
 export class DefaultControlComponent implements OnInit, OnDestroy {
-  instance: Collection<control.Control>;
+  instance: Collection<Control>;
   @Input()
   attribution: boolean;
   @Input()
-  attributionOptions: olx.control.AttributionOptions;
+  attributionOptions: AttributionOptions;
   @Input()
   rotate: boolean;
   @Input()
-  rotateOptions: olx.control.RotateOptions;
+  rotateOptions: RotateOptions;
   @Input()
   zoom: boolean;
   @Input()
-  zoomOptions: olx.control.ZoomOptions;
+  zoomOptions: ZoomOptions;
 
   constructor(private map: MapComponent) {}
 
   ngOnInit() {
     // console.log('ol.control.defaults init: ', this);
-    this.instance = control.defaults(this);
+    this.instance = defaults(this);
     this.instance.forEach(c => this.map.instance.addControl(c));
   }