From 3da0cc9ad42d5d7c857f830315adf7fcc8a50787 Mon Sep 17 00:00:00 2001
From: gkolokythas <gkolokythas@cite.gr>
Date: Wed, 8 Jan 2020 13:04:04 +0200
Subject: [PATCH] Fixes not showing "New Version" option on DMP overview action
 menu.

---
 .../src/app/ui/dmp/overview/dmp-overview.component.html        | 2 +-
 dmp-frontend/src/app/ui/dmp/overview/dmp-overview.component.ts | 3 ++-
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/dmp-frontend/src/app/ui/dmp/overview/dmp-overview.component.html b/dmp-frontend/src/app/ui/dmp/overview/dmp-overview.component.html
index 87060f906..7933e2671 100644
--- a/dmp-frontend/src/app/ui/dmp/overview/dmp-overview.component.html
+++ b/dmp-frontend/src/app/ui/dmp/overview/dmp-overview.component.html
@@ -11,7 +11,7 @@
 								<mat-icon class="more-horiz">more_horiz</mat-icon>
 							</button>
 							<mat-menu #actionsMenu="matMenu" xPosition="before">
-								<button *ngIf="!this.isPublicView && isUserOwner" mat-menu-item (click)="newVersion(dmp.id, dmp.label)">
+								<button *ngIf="isUserOwner" mat-menu-item (click)="newVersion(dmp.id, dmp.label)">
 									<mat-icon>queue</mat-icon>{{'DMP-LISTING.ACTIONS.NEW-VERSION' | translate}}
 								</button>
 								<button mat-menu-item (click)="viewVersions(dmp.groupId, dmp.label)">
diff --git a/dmp-frontend/src/app/ui/dmp/overview/dmp-overview.component.ts b/dmp-frontend/src/app/ui/dmp/overview/dmp-overview.component.ts
index 9ad6bbf8c..35a749461 100644
--- a/dmp-frontend/src/app/ui/dmp/overview/dmp-overview.component.ts
+++ b/dmp-frontend/src/app/ui/dmp/overview/dmp-overview.component.ts
@@ -79,6 +79,7 @@ export class DmpOverviewComponent extends BaseComponent implements OnInit {
 						.pipe(takeUntil(this._destroyed))
 						.subscribe(data => {
 							this.dmp = data;
+							this.setIsUserOwner();
 							const breadCrumbs = [];
 							breadCrumbs.push({ parentComponentName: null, label: this.language.instant('NAV-BAR.PUBLIC-DMPS'), url: "/explore-plans" });
 							breadCrumbs.push({ parentComponentName: 'DmpListingComponent', label: this.dmp.label, url: '/plans/publicOverview/' + this.dmp.id });
@@ -91,7 +92,7 @@ export class DmpOverviewComponent extends BaseComponent implements OnInit {
 	setIsUserOwner() {
 		if (this.dmp) {
 			const principal: Principal = this.authentication.current();
-			this.isUserOwner = principal.id === this.dmp.users.find(x => x.role === Role.Owner).id;
+			 if (principal) this.isUserOwner = principal.id === this.dmp.users.find(x => x.role === Role.Owner).id;
 		}
 	}