Merge pull request #131 from cjc343/master

Fix TypeScript 2.0 type checking issues

Author: flauc

src/notification.component.ts

@@ -236,8 +236,8 @@ export class NotificationComponent implements OnInit, OnDestroy {
         return this.position !== 0 ? this.position * 90 : 0;
     }
 
-    onClick($e: any): void {
-        this.item.click.emit($e);
+    onClick($e: MouseEvent): void {
+        this.item.click!.emit($e);
 
         if (this.clickToClose) {
             this.remove();

src/notifications.service.ts

@@ -55,7 +55,7 @@ export class NotificationsService {
 
   // HTML Notification method
   html(html: any, type: string, override?: any) {
-    return this.set({html: html, type: type, icon: 'bare', override: override, title: null, content: null}, true);
+    return this.set({html: html, type: type, icon: 'bare', override: override}, true);
   }
 
   // Remove all notifications method
@@ -64,4 +64,4 @@ export class NotificationsService {
     else this.emitter.next({command: 'cleanAll'});
   }
 
-}
+}

src/simple-notifications.component.ts

@@ -90,11 +90,11 @@ export class SimpleNotificationsComponent implements OnInit, OnDestroy {
                         break;
 
                     case 'clean':
-                        this.cleanSingle(item.id);
+                        this.cleanSingle(item.id!);
                         break;
 
                     case 'set':
-                        if (item.add) this.add(item.notification);
+                        if (item.add) this.add(item.notification!);
                         else this.defaultBehavior(item);
                         break;
 

tests/spec/notifications.service.spec.ts

@@ -206,8 +206,8 @@ describe('NotificationsService', () => {
             expect(notification.type).toBe('success');
             expect(notification.icon).toBe('bare');
 
-            expect(notification.title).toBeNull();
-            expect(notification.content).toBeNull();
+            expect(notification.title).toBeUndefined();
+            expect(notification.content).toBeUndefined();
             expect(notification.override).toBeUndefined();
             expect(notification.html).toBe('<B>Title</B>');
             expect(notification.state).toBeUndefined();
@@ -239,4 +239,4 @@ describe('NotificationsService', () => {
         })
     );
 
-});
+});