From 386ab547bd1e5b1288778006ab74a439199a1794 Mon Sep 17 00:00:00 2001
From: Eneko Nieto <eneko@baturamobile.com>
Date: Sun, 7 Feb 2021 23:16:24 +0100
Subject: [PATCH] Work in edit travel. Login DOES NOT wok!

---
 src/app/app.component.ts                      |  6 +-----
 src/app/auth.config.ts                        |  2 +-
 .../edit-travel/edit-travel.component.html    |  2 +-
 .../edit-travel/edit-travel.component.ts      | 21 ++++++++++++++++++-
 src/app/pages/travel/travel.component.html    |  2 +-
 src/app/pages/travel/travel.component.ts      |  9 +++++++-
 src/app/services/api.service.ts               |  4 ++++
 src/app/views/header/header.component.ts      |  1 +
 .../travel-form/travel-form.component.html    | 12 +++++------
 .../travel-form/travel-form.component.ts      |  5 ++---
 10 files changed, 45 insertions(+), 19 deletions(-)

diff --git a/src/app/app.component.ts b/src/app/app.component.ts
index 1c506c0..12f8df2 100644
--- a/src/app/app.component.ts
+++ b/src/app/app.component.ts
@@ -30,10 +30,6 @@ export class AppComponent {
         this.router.navigate(['/']);
       }
     });
-
-    // Optional
-    if (authConfig.useSilentRefresh) {
-      this.oauthService.setupAutomaticSilentRefresh();
-    }
+    this.oauthService.setupAutomaticSilentRefresh();
   }
 }
diff --git a/src/app/auth.config.ts b/src/app/auth.config.ts
index be83559..3416eab 100644
--- a/src/app/auth.config.ts
+++ b/src/app/auth.config.ts
@@ -37,7 +37,7 @@ export const authConfig: AuthConfig = {
 
   useSilentRefresh: useSilentRefreshForCodeFlow,
 
-  showDebugInformation: false,
+  showDebugInformation: true,
 
   sessionChecksEnabled: true,
   sessionCheckIntervall: 10000,
diff --git a/src/app/pages/edit-travel/edit-travel.component.html b/src/app/pages/edit-travel/edit-travel.component.html
index 8ddae6a..585a22b 100644
--- a/src/app/pages/edit-travel/edit-travel.component.html
+++ b/src/app/pages/edit-travel/edit-travel.component.html
@@ -1,4 +1,4 @@
 <div>
     <h1>Editar viaje</h1>
-    <app-travel-form></app-travel-form>
+    <app-travel-form (travelSubmitted)="editTravel($event)" (travelCanceled)="cancel()" [travel]="travel" [acceptButtonText]="acceptButtonText"></app-travel-form>
 </div>
\ No newline at end of file
diff --git a/src/app/pages/edit-travel/edit-travel.component.ts b/src/app/pages/edit-travel/edit-travel.component.ts
index 07d8043..72f6107 100644
--- a/src/app/pages/edit-travel/edit-travel.component.ts
+++ b/src/app/pages/edit-travel/edit-travel.component.ts
@@ -1,5 +1,5 @@
 import { Component, OnInit, ViewChild } from '@angular/core';
-import { ActivatedRoute } from '@angular/router';
+import { ActivatedRoute, Router } from '@angular/router';
 import { TravelFormComponent } from 'src/app/views/travel-form/travel-form.component';
 import { PUBLIC_API_URL } from '../../app.config';
 import { Travel, TravelId } from '../../entities/travel';
@@ -12,10 +12,12 @@ import { ApiService } from '../../services/api.service';
 })
 export class EditTravelComponent implements OnInit {
   travel: Travel;
+  acceptButtonText = 'Save';
 
   @ViewChild(TravelFormComponent, { static: true }) form: TravelFormComponent;
 
   constructor(
+    private router: Router,
     private apiService: ApiService,
     private route: ActivatedRoute
   ) { }
@@ -35,4 +37,21 @@ export class EditTravelComponent implements OnInit {
         });
     });
   }
+
+  editTravel(travel: Travel): void {
+    this.apiService.createTravel(travel)
+      .subscribe(res => {
+        if (res.success) {
+          console.log('Travel edited');
+          this.router.navigateByUrl('/');
+        }
+        else {
+          console.error('Error creating travel: ' + res.error);
+        }
+      });
+  }
+
+  cancel(): void {
+    this.router.navigateByUrl('/');
+  }
 }
diff --git a/src/app/pages/travel/travel.component.html b/src/app/pages/travel/travel.component.html
index 18845c4..e20d2b2 100644
--- a/src/app/pages/travel/travel.component.html
+++ b/src/app/pages/travel/travel.component.html
@@ -13,4 +13,4 @@
 </mat-card>
 
 <button mat-raised-button class="btn btn-default" (click)="goBack()">Back</button>
-<button mat-raised-button class="btn btn-default" (click)="edit()">Edit</button>
+<button *ngIf="logged" mat-raised-button class="btn btn-default" (click)="edit()">Edit</button>
diff --git a/src/app/pages/travel/travel.component.ts b/src/app/pages/travel/travel.component.ts
index 5a3595f..4830eaa 100644
--- a/src/app/pages/travel/travel.component.ts
+++ b/src/app/pages/travel/travel.component.ts
@@ -1,5 +1,6 @@
 import { Component, OnInit } from '@angular/core';
 import { ActivatedRoute, Router } from '@angular/router';
+import { OAuthService } from 'angular-oauth2-oidc';
 import { Travel, TravelId } from '../../entities/travel';
 import { ApiService } from '../../services/api.service';
 
@@ -15,6 +16,7 @@ export class TravelComponent implements OnInit {
 
   constructor(
     private router: Router,
+    private oauthService: OAuthService,
     private apiService: ApiService,
     private route: ActivatedRoute
   ) { }
@@ -40,6 +42,11 @@ export class TravelComponent implements OnInit {
   }
 
   edit(): void {
-    this.router.navigateByUrl('/');
+    this.router.navigateByUrl(`/travel/${this.travelId}/edit`);
+  }
+
+  get logged(): boolean {
+    return this.oauthService.getIdentityClaims() != null;
+    // return this.oauthService.hasValidIdToken() && this.oauthService.hasValidAccessToken();
   }
 }
diff --git a/src/app/services/api.service.ts b/src/app/services/api.service.ts
index 0e96d07..54d6e4d 100644
--- a/src/app/services/api.service.ts
+++ b/src/app/services/api.service.ts
@@ -39,6 +39,10 @@ export class ApiService {
     return this.callApi<void>(TRAVEL_API_URL + '/create', null, travel);
   }
 
+  editTravel = (travel: Travel) => {
+    return this.callApi<void>(TRAVEL_API_URL + '/edit', null, travel);
+  }
+
   private callApi<T>(
     url: string,
     params?: { [param: string]: any },
diff --git a/src/app/views/header/header.component.ts b/src/app/views/header/header.component.ts
index b12a09f..b566545 100644
--- a/src/app/views/header/header.component.ts
+++ b/src/app/views/header/header.component.ts
@@ -15,6 +15,7 @@ export class HeaderComponent implements OnInit {
   ) { }
 
   ngOnInit(): void {
+    console.log('INIT logged=' + String(this.logged));
     this.oauthService.configure(authConfig);
     this.oauthService.loadDiscoveryDocumentAndTryLogin().then(success => {
       console.log('Autologin success=' + success + ' logged=' + String(this.logged));
diff --git a/src/app/views/travel-form/travel-form.component.html b/src/app/views/travel-form/travel-form.component.html
index 04cf224..b0af5d2 100644
--- a/src/app/views/travel-form/travel-form.component.html
+++ b/src/app/views/travel-form/travel-form.component.html
@@ -1,8 +1,8 @@
-<form #travelForm="ngForm">
+<form *ngIf="travel" #travelForm="ngForm">
     <div>
         <label for="departureDate">Fecha de salida</label>
         <input matInput id="departureDate" name="departureDate" [ngxMatDatetimePicker]="picker"
-            placeholder="Choose a date" [min]="minDate" [(ngModel)]="newTravel.departureDate">
+            placeholder="Choose a date" [min]="minDate" [(ngModel)]="travel.departureDate">
         <mat-datepicker-toggle matSuffix [for]="$any(picker)"></mat-datepicker-toggle>
         <ngx-mat-datetime-picker #picker [showSeconds]="$any('false')" [stepMinute]="$any('15')">
         </ngx-mat-datetime-picker>
@@ -13,7 +13,7 @@
 
     <div>
         <label for="origin">Origen</label>
-        <input type="text" id="origin" required [(ngModel)]="newTravel.origin" name="origin">
+        <input type="text" id="origin" required [(ngModel)]="travel.origin" name="origin">
         <div hidden="origin.valid || origin.pristine" class="alert alert-danger">
             El origen es obligatorio
         </div>
@@ -21,7 +21,7 @@
 
     <div>
         <label for="destination">Destino</label>
-        <input type="text" id="destination" required [(ngModel)]="newTravel.destination" name="destination">
+        <input type="text" id="destination" required [(ngModel)]="travel.destination" name="destination">
         <div hidden="destination.valid || destination.pristine" class="alert alert-danger">
             El destino es obligatorio
         </div>
@@ -29,7 +29,7 @@
 
     <div>
         <label for="places">Plazas</label>
-        <input type="number" id="places" required [(ngModel)]="newTravel.places" name="places">
+        <input type="number" id="places" required [(ngModel)]="travel.places" name="places">
         <div hidden="places.valid || places.pristine" class="alert alert-danger">
             El destino es obligatorio
         </div>
@@ -37,7 +37,7 @@
 
     <div>
         <label for="description">Descripción</label>
-        <input type="text" id="description" [(ngModel)]="newTravel.description" name="description">
+        <input type="text" id="description" [(ngModel)]="travel.description" name="description">
     </div>
 
 
diff --git a/src/app/views/travel-form/travel-form.component.ts b/src/app/views/travel-form/travel-form.component.ts
index 161cf84..be2aa09 100644
--- a/src/app/views/travel-form/travel-form.component.ts
+++ b/src/app/views/travel-form/travel-form.component.ts
@@ -12,19 +12,18 @@ export class TravelFormComponent implements OnInit {
 
   @Input() travel: Travel;
   @Input() acceptButtonText: string;
-  newTravel: Travel;
   submitted = false;
   minDate = new Date();
 
   constructor() { }
 
   ngOnInit(): void {
-    this.newTravel = { ...this.travel };
+    this.travel = this.travel;
   }
 
   onSubmit(): void {
     this.submitted = true;
-    this.travelSubmitted.emit(this.newTravel);
+    this.travelSubmitted.emit(this.travel);
   }
 
   onCancel(): void {