From a73de946215f87495e83355dfa7c5155bf833120 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Arkadiusz=20Pa=C5=82ka?= <arkadiusz.palka@valueadd.pl>
Date: Fri, 24 Jan 2020 11:57:55 +0100
Subject: [PATCH] fix(pmp-web): fix error while trying get properties of null

object
---
 .../repository-statistics.component.html      |  2 +-
 .../repository-statistics.component.ts        | 47 ++++++++++---------
 2 files changed, 25 insertions(+), 24 deletions(-)

diff --git a/libs/pmp-web/repository/feature-repository-statistics/src/lib/repository-statistics/repository-statistics.component.html b/libs/pmp-web/repository/feature-repository-statistics/src/lib/repository-statistics/repository-statistics.component.html
index 1cc41728..f81125c8 100644
--- a/libs/pmp-web/repository/feature-repository-statistics/src/lib/repository-statistics/repository-statistics.component.html
+++ b/libs/pmp-web/repository/feature-repository-statistics/src/lib/repository-statistics/repository-statistics.component.html
@@ -1,6 +1,6 @@
 <div class="repository-statistics__container">
   <pmp-header-container
-    [avatarLabel]="repository.fullName"
+    [avatarLabel]="repository.name"
     [avatarUrl]="repository.repositoryPictureUrl"
     label="Pull requests pending"
   ></pmp-header-container>
diff --git a/libs/pmp-web/repository/feature-repository-statistics/src/lib/repository-statistics/repository-statistics.component.ts b/libs/pmp-web/repository/feature-repository-statistics/src/lib/repository-statistics/repository-statistics.component.ts
index 4df4123c..af30985b 100644
--- a/libs/pmp-web/repository/feature-repository-statistics/src/lib/repository-statistics/repository-statistics.component.ts
+++ b/libs/pmp-web/repository/feature-repository-statistics/src/lib/repository-statistics/repository-statistics.component.ts
@@ -5,6 +5,28 @@ import { untilDestroyed } from 'ngx-take-until-destroy';
 import { RepositoryModel } from '@pimp-my-pr/shared/domain';
 import { first } from 'rxjs/operators';
 
+const mockedData = {
+  // TODO mocked data for repository
+  owner: 'valueadd',
+  fullName: 'pimp-my-pr',
+  name: 'pmp',
+  repositoryPictureUrl: 'https://homepages.cae.wisc.edu/~ece533/images/airplane.png',
+  prsStatistics: [
+    {
+      author: 'asda',
+      authorAvatarUrl: 'https://homepages.cae.wisc.edu/~ece533/images/airplane.png',
+      commentsCount: 123,
+      createdAt: '12/12/12',
+      linesOfCodeToCheck: 3245,
+      reviewCommentsCount: 234,
+      id: 2345,
+      timeWaiting: '124',
+      title: '12414',
+      url: 'https://homepages.cae.wisc.edu/~ece533/images/airplane.png'
+    }
+  ]
+};
+
 @Component({
   selector: 'pimp-my-pr-repository-statistics',
   templateUrl: './repository-statistics.component.html',
@@ -12,27 +34,7 @@ import { first } from 'rxjs/operators';
 })
 export class RepositoryStatisticsComponent implements OnDestroy, OnInit {
   repositoryName: string | null;
-  repository: RepositoryModel = {
-    // TODO mocked data for repository
-    owner: 'valueadd',
-    fullName: 'pimp-my-pr',
-    name: 'pmp',
-    repositoryPictureUrl: 'https://homepages.cae.wisc.edu/~ece533/images/airplane.png',
-    prsStatistics: [
-      {
-        author: 'asda',
-        authorAvatarUrl: 'https://homepages.cae.wisc.edu/~ece533/images/airplane.png',
-        commentsCount: 123,
-        createdAt: '12/12/12',
-        linesOfCodeToCheck: 3245,
-        reviewCommentsCount: 234,
-        id: 2345,
-        timeWaiting: '124',
-        title: '12414',
-        url: 'https://homepages.cae.wisc.edu/~ece533/images/airplane.png'
-      }
-    ]
-  };
+  repository: RepositoryModel;
 
   constructor(
     private cdr: ChangeDetectorRef,
@@ -43,14 +45,13 @@ export class RepositoryStatisticsComponent implements OnDestroy, OnInit {
   ngOnDestroy(): void {}
 
   ngOnInit(): void {
-    console.log('elo');
     this.initGetRepositoryStatistics();
     this.initSubscribeRepositoryStatistics();
   }
 
   private initSubscribeRepositoryStatistics(): void {
     this.facade.repositoryStatistics$.pipe(untilDestroyed(this)).subscribe(repository => {
-      this.repository = repository;
+      this.repository = mockedData; /* TODO repository instead mockedData */
       this.cdr.markForCheck();
     });
   }