chore: update to Angular 4.1 and TypeScript 2.3 (#775)

* chore(deps): update to Angular 4.1.0

* chore(lint): update tslint to ^5.1.0 and codelyzer to ^3.0.1

* refactor: fix lint errors

* fix: make update-app-ng-deps work with peerDeps

Author: sis0k0

Diff for: nativescript-angular/bin/update-app-ng-deps

@@ -1,33 +1,43 @@
 #!/usr/bin/env node
 
-var path = require("path");
-var fs = require("fs");
-
-var binPath = __dirname;
-var pluginPath = path.dirname(binPath);
-var pluginPackageJsonPath = path.join(pluginPath, "package.json");
-var projectPath = path.dirname(path.dirname(pluginPath));
-var appPackageJsonPath = path.join(projectPath, "package.json");
-
-var appPackageJson = JSON.parse(fs.readFileSync(appPackageJsonPath, "utf8"));
-var pluginPackageJson = JSON.parse(fs.readFileSync(pluginPackageJsonPath, "utf8"));
-
-Object.keys(pluginPackageJson.dependencies).forEach(function(dependencyName) {
-    var version = pluginPackageJson.dependencies[dependencyName];
-    if (dependencyName.startsWith("@angular") || dependencyName === "rxjs") {
-        appPackageJson.dependencies[dependencyName] = version;
-        console.log("Updated dependency '" + dependencyName + "' to version: " + version + ".");
-    }
-});
-
-Object.keys(pluginPackageJson.devDependencies).forEach(function(dependencyName) {
-    var version = pluginPackageJson.devDependencies[dependencyName];
-    if (dependencyName.startsWith("@angular") || dependencyName === "zone.js") {
-        appPackageJson.devDependencies[dependencyName] = version;
-        console.log("Updated dev dependency '" + dependencyName + "' to version: " + version + ".");
-    }
-});
-
-fs.writeFileSync(appPackageJsonPath, JSON.stringify(appPackageJson, null, 2));
+const path = require("path");
+const fs = require("fs");
+
+const binPath = __dirname;
+const pluginPath = path.dirname(binPath);
+const pluginPackageJsonPath = path.join(pluginPath, "package.json");
+const pluginPackageJson = JSON.parse(fs.readFileSync(pluginPackageJsonPath, "utf8"));
+const pluginPeerDeps = pluginPackageJson.peerDependencies;
+
+const projectPath = path.dirname(path.dirname(pluginPath));
+const appPackageJsonPath = path.join(projectPath, "package.json");
+const appPackageJson = JSON.parse(fs.readFileSync(appPackageJsonPath, "utf8"));
+
+const shouldUpdateDependency = name => name.startsWith("@angular") ||
+    name === "rxjs" ||
+    name === "zone.js";
+
+let updatedDependencies = appPackageJson.dependencies;
+
+// set app dependencies to ones required from plugin (peer)
+Object.keys(pluginPeerDeps)
+    .filter(shouldUpdateDependency)
+    .forEach(dependencyName => {
+        const version = pluginPackageJson.peerDependencies[dependencyName];
+        updatedDependencies[dependencyName] = version;
+        console.log(`Updated dependency ${dependencyName} to version: ${version}.`);
+    });
+
+// remove platform-browser-dynamic if present
+const browserDynamicDependency = "@angular/platform-browser-dynamic";
+if (updatedDependencies.hasOwnProperty(browserDynamicDependency)) {
+    delete updatedDependencies[browserDynamicDependency];
+    console.log(`Removed ${browserDynamicDependency}`);
+}
+
+let updatedPackageJson = appPackageJson;
+updatedPackageJson.dependencies = updatedDependencies;
+
+fs.writeFileSync(appPackageJsonPath, JSON.stringify(updatedPackageJson, null, 2));
 
 console.log("\nAngular dependencies updated. Don't forget to run `npm install`.");

Diff for: nativescript-angular/directives/list-view-comp.ts

@@ -168,7 +168,7 @@ export class ListViewComponent implements DoCheck, OnDestroy, AfterContentInit {
             if (!viewRef) {
                 listViewError("ViewReference not found for item " + index + ". View recycling is not working");
             }
-        };
+        }
 
         if (!viewRef) {
             listViewLog("onItemLoading: " + index + " - Creating view from template");
@@ -234,7 +234,8 @@ function getSingleViewRecursive(nodes: Array<any>, nestLevel: number): View {
 export interface ComponentView {
     rootNodes: Array<any>;
     destroy(): void;
-};
+}
+
 export type RootLocator = (nodes: Array<any>, nestLevel: number) => View;
 
 export function getItemViewRoot(viewRef: ComponentView, rootLocator: RootLocator = getSingleViewRecursive): View {

Diff for: nativescript-angular/http.ts

@@ -11,11 +11,11 @@ export { NSHttp } from "./http/ns-http";
 
 export function nsHttpFactory(backend, options, nsFileSystem) {
     return new NSHttp(backend, options, nsFileSystem);
-};
+}
 
 export function nsXSRFStrategyFactory() {
     return new NSXSRFStrategy();
-};
+}
 
 @NgModule({
     providers: [

Diff for: nativescript-angular/nativescript.module.ts

@@ -26,7 +26,7 @@ import { NS_DIRECTIVES } from "./directives";
 
 export function errorHandlerFactory() {
     return new ErrorHandler(true);
-};
+}
 
 @NgModule({
     declarations: [

Diff for: nativescript-angular/package.json

@@ -29,7 +29,7 @@
     "url": "https://github.com/NativeScript/nativescript-angular.git"
   },
   "scripts": {
-    "tslint": "tslint --project tsconfig.json --config tslint.json",
+    "tslint": "tslint --project tsconfig.json --config tslint.json --type-check",
     "postinstall": "node postinstall.js",
     "tsc": "tsc -p tsconfig.json",
     "ngc": "ngc -p tsconfig.json",
@@ -43,32 +43,32 @@
     "reflect-metadata": "^0.1.8"
   },
   "peerDependencies": {
-    "@angular/core": "~4.0.0",
-    "@angular/common": "~4.0.0",
-    "@angular/compiler": "~4.0.0",
-    "@angular/platform-browser": "~4.0.0",
-    "@angular/router": "~4.0.0",
-    "@angular/forms": "~4.0.0",
-    "@angular/http": "~4.0.0",
-    "tns-core-modules": "^3.0.0 || ^3.0.0-rc.1",
+    "@angular/common": "~4.0.0 || ~4.1.0",
+    "@angular/compiler": "~4.0.0 || ~4.1.0",
+    "@angular/core": "~4.0.0 || ~4.1.0",
+    "@angular/forms": "~4.0.0 || ~4.1.0",
+    "@angular/http": "~4.0.0 || ~4.1.0",
+    "@angular/platform-browser": "~4.0.0 || ~4.1.0",
+    "@angular/router": "~4.0.0 || ~4.1.0",
     "rxjs": "^5.0.1",
+    "tns-core-modules": "^3.0.0 || ^3.0.0-rc.1",
     "zone.js": "^0.8.4"
   },
   "devDependencies": {
-    "@angular/core": "~4.0.0",
-    "@angular/common": "~4.0.0",
-    "@angular/compiler": "~4.0.0",
-    "@angular/compiler-cli": "~4.0.0",
-    "@angular/platform-browser": "~4.0.0",
-    "@angular/animations": "~4.0.0",
-    "@angular/router": "~4.0.0",
-    "@angular/forms": "~4.0.0",
-    "@angular/http": "~4.0.0",
-    "codelyzer": "~3.0.0-beta.4",
-    "tns-core-modules": "internal-preview",
-    "tslint": "~4.5.0",
-    "typescript": "~2.2.1",
+    "@angular/animations": "~4.0.0 || ~4.1.0",
+    "@angular/common": "~4.0.0 || ~4.1.0",
+    "@angular/compiler": "~4.0.0 || ~4.1.0",
+    "@angular/compiler-cli": "~4.0.0 || ~4.1.0",
+    "@angular/core": "~4.0.0 || ~4.1.0",
+    "@angular/forms": "~4.0.0 || ~4.1.0",
+    "@angular/http": "~4.0.0 || ~4.1.0",
+    "@angular/platform-browser": "~4.0.0 || ~4.1.0",
+    "@angular/router": "~4.0.0 || ~4.1.0",
+    "codelyzer": "^3.0.1",
     "rxjs": "^5.0.1",
+    "tns-core-modules": "internal-preview",
+    "tslint": "^5.1.0",
+    "typescript": "^2.3.2",
     "zone.js": "^0.8.4"
   }
 }

Diff for: nativescript-angular/platform-common.ts

@@ -129,7 +129,7 @@ export class NativeScriptPlatformRef extends PlatformRef {
 
     get injector(): Injector {
         return this.platform.injector;
-    };
+    }
 
     destroy(): void {
         this.platform.destroy();

Diff for: nativescript-angular/tsconfig.json

@@ -31,6 +31,6 @@
     "genDir": ".",
     "skipMetadataEmit": false,
     "skipTemplateCodegen": true,
-    "debug": true
+    "strictMetadataEmit": true
   }
 }

Diff for: nativescript-angular/tslint.json

@@ -62,7 +62,6 @@
     "no-string-literal": false,
     "no-switch-case-fall-through": true,
     "no-unused-expression": true,
-    "no-use-before-declare": true,
     "no-var-keyword": true,
     "radix": false,
     "switch-default": true,