Merge pull request TrilonIO#5 from viperguynaz/master

implemented remaining methods

Author: MarkPieszak

npm-debug.log.1124403154

@@ -5,8 +5,7 @@
     "lint": "tslint src/**/*.ts",
     "test": "tsc && karma start",
     "clean": "rimraf ./dist",
-    "build": "nmp run clean && npm run tsc",
-    "prepublish": "tsc",
+    "build": "npm run clean && npm run tsc",
     "tsc": "tsc"
   },
   "repository": {
@@ -24,13 +23,14 @@
   "main": "./dist/index.js",
   "dependencies": {
     "@angular/router": "^3.4.5",
-    "@types/applicationinsights-js": "^0.23.5",
-    "applicationinsights-js": "^1.0.7",
+    "@types/applicationinsights-js": "^1.0.1",
+    "applicationinsights-js": "^1.0.8",
     "rxjs": "^5.0.3"
   },
   "devDependencies": {
     "@angular/core": "^2.0.0",
     "@angular/common": "^2.0.0",
+    "@angular/platform-browser": "^2.4.10",
     "codelyzer": "^0.0.28",
     "tslint": "^3.15.1",
     "typescript": "2.0.2",

package.json

@@ -17,8 +17,8 @@ export class AppInsightsService {
   // https://github.com/Microsoft/ApplicationInsights-JS/blob/master/API-reference.md#trackevent
   // trackEvent(name: string, properties?: {[string]:string}, measurements?: {[string]:number})
   // Log a user action or other occurrence.
-  trackEvent(eventName: string, eventProperties?: {string: string}, metricProperty?: {string: number}) {
-    if (!this.isBrowser) {
+  static trackEvent(eventName: string, eventProperties?: {[name: string]: string}, metricProperty?: {[name: string]: number}) {
+    if (!AppInsightsService.isBrowser) {
       return;
     }
 
@@ -33,8 +33,8 @@ export class AppInsightsService {
   // https://github.com/Microsoft/ApplicationInsights-JS/blob/master/API-reference.md#trackpageview
   // trackPageView(name?: string, url?: string, properties?:{[string]:string}, measurements?: {[string]:number}, duration?: number)
   // Logs that a page or similar container was displayed to the user.
-  trackPageView(name?: string, url?: string, properties?: {[name: string]: string}, measurements?: {[name: string]: number}, duration?: number) {
-    if (!this.isBrowser) {
+  static trackPageView(name?: string, url?: string, properties?: {[name: string]: string}, measurements?: {[name: string]: number}, duration?: number) {
+    if (!AppInsightsService.isBrowser) {
       return;
     }
 
@@ -50,8 +50,8 @@ export class AppInsightsService {
   // Starts the timer for tracking a page view. Use this instead of trackPageView if you want to control when the 
   // page view timer starts and stops, but don't want to calculate the duration yourself. This method doesn't send any 
   // telemetry. Call stopTrackPage to log the end of the page view and send the event.
-  startTrackPage(name?: string) {
-    if (!this.isBrowser) {
+  static startTrackPage(name?: string) {
+    if (!AppInsightsService.isBrowser) {
       return;
     }
 
@@ -66,8 +66,8 @@ export class AppInsightsService {
   // stopTrackPage(name?: string, url?: string, properties?: Object, measurements?: Object)
   // Stops the timer that was started by calling startTrackPage and sends the page view telemetry with the 
   // specified properties and measurements. The duration of the page view will be the time between calling startTrackPage and stopTrackPage.
-  stopTrackPage(name?: string, url?: string, properties?: {[name: string]: string}, measurements?: {[name: string]: number}) {
-    if (!this.isBrowser) {
+  static stopTrackPage(name?: string, url?: string, properties?: {[name: string]: string}, measurements?: {[name: string]: number}) {
+    if (!AppInsightsService.isBrowser) {
       return;
     }
 
@@ -78,48 +78,117 @@ export class AppInsightsService {
     }
   }
 
-  // [[ TODO ]] **
-
   // https://github.com/Microsoft/ApplicationInsights-JS/blob/master/API-reference.md#trackmetric
   // trackMetric(name: string, average: number, sampleCount?: number, min?: number, max?: number, properties?: {[string]:string})
   // Log a positive numeric value that is not associated with a specific event. 
   // Typically used to send regular reports of performance indicators.
-  trackMetric() { return this.notImplemented('trackMetric'); }
+  static trackMetric(name: string, average: number, sampleCount?: number, min?: number, max?: number, properties?: {[name: string]: string}) {
+    if (!AppInsightsService.isBrowser) {
+      return;
+    }
+
+    try {
+      AppInsights.trackMetric(name, average, sampleCount, min, max, properties);
+    } catch (ex) {
+      console.warn('Angular application insights Error [trackTrace]: ', ex);
+    }
+  }
 
   // https://github.com/Microsoft/ApplicationInsights-JS/blob/master/API-reference.md#trackexception
   // trackException(exception: Error, handledAt?: string, properties?: {[string]:string}, measurements?: {[string]:number}, severityLevel?: AI.SeverityLevel)
   // Log an exception you have caught. (Exceptions caught by the browser are also logged.)
-  trackException() { return this.notImplemented('trackException'); }
+  static trackException(exception: Error, handledAt?: string, properties?: {[name: string]: string},
+                        measurements?: {[name: string]: number}, severityLevel?: AI.SeverityLevel) {
+    if (!AppInsightsService.isBrowser) {
+      return;
+    }
+
+    try {
+      AppInsights.trackException(exception, handledAt, properties, measurements, severityLevel);
+    } catch (ex) {
+      console.warn('Angular application insights Error [trackException]: ', ex);
+    }
+  }
 
   // https://github.com/Microsoft/ApplicationInsights-JS/blob/master/API-reference.md#tracktrace
   // trackTrace(message: string, properties?: {[string]:string}, measurements?: {[string]:number})
   // Log a diagnostic event such as entering or leaving a method.
-  trackTrace() { return this.notImplemented('trackTrace'); }
+  static trackTrace(message: string, properties?: {[name: string]: string}) {
+    if (!AppInsightsService.isBrowser) {
+      return;
+    }
+
+    try {
+      AppInsights.trackTrace(message, properties);
+    } catch (ex) {
+      console.warn('Angular application insights Error [trackTrace]: ', ex);
+    }
+  }
 
   // https://github.com/Microsoft/ApplicationInsights-JS/blob/master/API-reference.md#trackdependency
   // trackDependency(id: string, method: string, absoluteUrl: string, pathName: string, totalTime: number, success: boolean, resultCode: number)
   // Log a dependency call (for instance: ajax)
-  trackDependency() { return this.notImplemented('trackDependency'); }
+  static trackDependency(id: string, method: string, absoluteUrl: string, pathName: string, totalTime: number, success: boolean, resultCode: number) {
+    if (!AppInsightsService.isBrowser) {
+      return;
+    }
+
+    try {
+      AppInsights.trackDependency(id, method, absoluteUrl, pathName, totalTime, success, resultCode);
+    } catch (ex) {
+      console.warn('Angular application insights Error [trackDependency]: ', ex);
+    }
+  }
 
   // https://github.com/Microsoft/ApplicationInsights-JS/blob/master/API-reference.md#flush
   // flush()
   // Immediately send all queued telemetry. Synchronous.
   // * You don't usually have to use this, as it happens automatically on window closing.
-  flush() { return this.notImplemented('flush'); }
+  static flush() {
+    if (!AppInsightsService.isBrowser) {
+      return;
+    }
+    try {
+      AppInsights.flush();
+    } catch (ex) {
+      console.warn('Angular application insights Error [flush]: ', ex);
+    }
+
+  }
 
   // https://github.com/Microsoft/ApplicationInsights-JS/blob/master/API-reference.md#setauthenticatedusercontext
   // setAuthenticatedUserContext(authenticatedUserId: string, accountId?: string)
   // Set the authenticated user id and the account id in this session. Use this when you have identified a specific 
   // signed-in user. Parameters must not contain spaces or ,;=|
-  setAuthenticatedUserContext() { return this.notImplemented('setAuthenticatedUserContext'); }
+  static setAuthenticatedUserContext(authenticatedUserId: string, accountId?: string) {
+    if (!AppInsightsService.isBrowser) {
+      return;
+    }
+
+    try {
+      AppInsights.setAuthenticatedUserContext(authenticatedUserId, accountId);
+    } catch (ex) {
+      console.warn('Angular application insights Error [setAuthenticatedUserContext]: ', ex);
+    }
+  }
 
   // https://github.com/Microsoft/ApplicationInsights-JS/blob/master/API-reference.md#clearauthenticatedusercontext
   // clearAuthenticatedUserContext ()
   // Clears the authenticated user id and the account id from the user context, and clears the associated cookie.
-  clearAuthenticatedUserContext() { return this.notImplemented('clearAuthenticatedUserContext'); }
+  static clearAuthenticatedUserContext() {
+    if (!AppInsightsService.isBrowser) {
+      return;
+    }
+
+    try {
+      AppInsights.clearAuthenticatedUserContext();
+    } catch (ex) {
+      console.warn('Angular application insights Error [clearAuthenticatedUserContext]: ', ex);
+    }
+  }
 
   public init(): void {
-    if (!this.isBrowser) {
+    if (!AppInsightsService.isBrowser) {
       return;
     }
 
@@ -128,27 +197,20 @@ export class AppInsightsService {
 
       this.router.events.filter(event => event instanceof NavigationStart)
       .subscribe((event: NavigationStart) => {
-        this.startTrackPage(event.url);
+        AppInsightsService.startTrackPage(event.url);
       });
 
       this.router.events.filter(event => event instanceof NavigationEnd)
       .subscribe((event: NavigationEnd) => {
-        this.stopTrackPage(event.url);
+        AppInsightsService.stopTrackPage(event.url);
       });
     } catch (ex) {
       console.warn('Angular application insights Error [downloadAndSetup]: ', ex);
     }
   }
 
-  private isBrowser(): boolean {
+  private static isBrowser(): boolean {
     return (typeof (<any>window) !== undefined);
   }
-
-  private notImplemented(methodName: string) {
-    return new Error(`
-      This method is not implemented in ApplicationInsightsService: [ ${methodName} ]
-      Please file any issues here: https://github.com/MarkPieszak/angular-application-insights/issues`);
-  }
-
 }