strict null check

Author: akirasosa

src/app/components/micropost-list/micropost-list.component.ts

@@ -47,7 +47,7 @@ export class MicropostListComponent implements OnInit {
     this.list(lastPost.id);
   }
 
-  private list(maxId: number = null): void {
+  private list(maxId: number|null = null): void {
     this.userMicropostService.list(this.userId, {maxId: maxId, count: 5})
       .subscribe(posts => {
         this.posts = this.posts.concat(posts);
@@ -58,8 +58,8 @@ export class MicropostListComponent implements OnInit {
 
   private loadUserStats() {
     this.userService.get(this.userId)
-      .subscribe(resp => {
-        this.userStats = resp.userStats;
+      .subscribe(user => {
+        if (user.userStats) this.userStats = user.userStats;
       }, e => this.errorHandler.handle(e))
     ;
   }

src/app/components/micropost-list/micropost-list.service.ts

@@ -12,7 +12,7 @@ export class MicropostListService {
   constructor(private http: JsonHttp) {
   }
 
-  list(userId: string, params: {maxId: number, count: number}): Observable<Micropost[]> {
+  list(userId: string, params: {maxId: number|null, count: number}): Observable<Micropost[]> {
     return this.http.get(url(userId), {search: objToSearchParams(params)})
       .map(res => res.json())
       ;

src/app/components/related-user-list/related-user-list.component.ts

@@ -10,7 +10,7 @@ import {styles} from "./related-user-list.component.styles";
 })
 export class RelatedUserListComponent implements OnInit {
 
-  @Input() listProvider: (params: {maxId: number, count: number}) => Observable<RelatedUser[]>;
+  @Input() listProvider: (params: {maxId: number|null, count: number}) => Observable<RelatedUser[]>;
 
   styles: any = styles;
   users: RelatedUser[] = [];
@@ -29,7 +29,7 @@ export class RelatedUserListComponent implements OnInit {
     this.list(lastUser.relationshipId);
   }
 
-  private list(maxId = null) {
+  private list(maxId: number|null = null) {
     this.listProvider({maxId: maxId, count: 5})
       .subscribe(users => {
           this.users = this.users.concat(users);

src/app/core/services/json-http.ts

@@ -8,7 +8,7 @@ import {
   Headers
 } from "@angular/http";
 
-const mergeAuthToken = (options: RequestOptionsArgs) => {
+const mergeAuthToken = (options: RequestOptionsArgs = {}) => {
   let newOptions = new RequestOptions({}).merge(options);
   let newHeaders = new Headers(newOptions.headers);
   const jwt = localStorage.getItem('jwt');

src/app/core/toast/toast.service.ts

@@ -1,13 +1,13 @@
 import {Injectable} from "@angular/core";
 import {Subject, Observable} from "rxjs";
-import {Toast} from "./toast";
+import {Toast, ToastEvent} from "./toast";
 
 @Injectable()
 export class ToastService {
 
-  private toastEvents: Subject<Toast> = new Subject<Toast>();
+  private toastEvents: Subject<ToastEvent> = new Subject<Toast>();
 
-  get events(): Observable<Toast> {
+  get events(): Observable<ToastEvent> {
     return this.toastEvents;
   }
 
@@ -23,7 +23,7 @@ export class ToastService {
     this.publish({message: message, level: 'error'});
   }
 
-  private publish(toast: Toast) {
+  private publish(toast: ToastEvent) {
     this.toastEvents.next(toast);
   }
 

src/app/core/toast/toast.ts

@@ -1,5 +1,10 @@
+export interface ToastEvent {
+  message: string;
+  level: string;
+}
+
 export interface Toast {
   message: string;
   level: string;
-  styles?: any[];
+  styles: any[];
 }

tsconfig.json

@@ -12,7 +12,8 @@
       "es2015",
       "dom"
     ],
-    "strictNullChecks": false,
+    "strictNullChecks": true,
+    "skipLibCheck": true,
     "baseUrl": "./src",
     "paths": {},
     "types": [