From 39a4b94ecca1f2b9381a96b3859201b9fd217d52 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Patrick=20Ha=C3=9Fel?= <git@patrick-hassel.de>
Date: Fri, 31 Oct 2025 08:57:47 +0100
Subject: [PATCH] force reload on same url navigation + refresh after websocket
 connect

---
 src/main/angular/src/app/app.html             |  5 ++++-
 src/main/angular/src/app/app.less             |  8 ++++++++
 src/main/angular/src/app/app.ts               | 13 ++++++++++--
 src/main/angular/src/app/common.ts            | 20 +++++++++++--------
 .../app/location/detail/location-detail.ts    | 15 +++++++++++---
 5 files changed, 47 insertions(+), 14 deletions(-)

diff --git a/src/main/angular/src/app/app.html b/src/main/angular/src/app/app.html
index a385b55..fd6fdf9 100644
--- a/src/main/angular/src/app/app.html
+++ b/src/main/angular/src/app/app.html
@@ -6,12 +6,15 @@
     <div class="MainMenuItem MainMenuTitle">
       {{ menuService.title }}
     </div>
+    @if(!ws.connected){
+      <div class="MainMenuItem MainMenuNotConnected">NICHT VERBUNDEN</div>
+    }
   </div>
 </div>
 
 <div class="MainMenuDrawer NoUserSelect" [hidden]="!showDrawer">
   @for (location of locationList; track location.id) {
-    <div class="MainMenuItem" routerLink="Location/{{ location.id }}" routerLinkActive="MainMenuItemActive" (click)="showDrawer = false">{{ location.name }}</div>
+    <div class="MainMenuItem" (click)="navigate('Location/' + location.id); showDrawer = false" routerLinkActive="MainMenuItemActive">{{ location.name }}</div>
   }
 </div>
 
diff --git a/src/main/angular/src/app/app.less b/src/main/angular/src/app/app.less
index 1c76b0b..d12de75 100644
--- a/src/main/angular/src/app/app.less
+++ b/src/main/angular/src/app/app.less
@@ -21,6 +21,14 @@
       background-color: lightskyblue;
     }
 
+    .MainMenuTitle {
+      flex: 1;
+    }
+
+    .MainMenuNotConnected {
+      color: red;
+    }
+
   }
 
 }
diff --git a/src/main/angular/src/app/app.ts b/src/main/angular/src/app/app.ts
index 0fc25cb..5636f9a 100644
--- a/src/main/angular/src/app/app.ts
+++ b/src/main/angular/src/app/app.ts
@@ -1,14 +1,15 @@
 import {Component, OnDestroy, OnInit} from '@angular/core';
-import {RouterLink, RouterLinkActive, RouterOutlet} from '@angular/router';
+import {Router, RouterLinkActive, RouterOutlet} from '@angular/router';
 import {FaIconComponent} from '@fortawesome/angular-fontawesome';
 import {faBars} from '@fortawesome/free-solid-svg-icons';
 import {MenuService} from './menu-service';
 import {Location} from './location/Location';
 import {LocationService} from './location/location-service';
+import {WebsocketService} from './common';
 
 @Component({
   selector: 'app-root',
-  imports: [RouterOutlet, FaIconComponent, RouterLink, RouterLinkActive],
+  imports: [RouterOutlet, FaIconComponent, RouterLinkActive],
   templateUrl: './app.html',
   styleUrl: './app.less'
 })
@@ -23,6 +24,8 @@ export class App implements OnInit, OnDestroy {
   constructor(
     readonly locationService: LocationService,
     readonly menuService: MenuService,
+    readonly router: Router,
+    readonly ws: WebsocketService,
   ) {
     //
   }
@@ -36,4 +39,10 @@ export class App implements OnInit, OnDestroy {
     this.menuService.title = "";
   }
 
+  navigate(url: string): void {
+    this.router.navigateByUrl('/', {skipLocationChange: true}).then(() => {
+      this.router.navigate([url]);
+    })
+  }
+
 }
diff --git a/src/main/angular/src/app/common.ts b/src/main/angular/src/app/common.ts
index 090cc16..a6b5172 100644
--- a/src/main/angular/src/app/common.ts
+++ b/src/main/angular/src/app/common.ts
@@ -71,24 +71,28 @@ export function stompServiceFactory() {
   providedIn: 'root'
 })
 export class WebsocketService {
+
+  private _connected: boolean = false;
+
+  get connected(): boolean {
+    return this._connected;
+  }
+
   constructor(
     readonly stompService: StompService,
   ) {
-    this.websocketConnected(() => this._websocketError = false);
-    this.websocketDisconnected(() => this._websocketError = true);
+    this.onChange(connected => this._connected = connected);
   }
 
-  private _websocketError: boolean = false;
-
-  get websocketError(): boolean {
-    return this._websocketError;
+  onChange(next: Next<boolean>): Subscription {
+    return this.stompService.connectionState$.pipe(map(state => state === RxStompState.OPEN)).subscribe(next);
   }
 
-  websocketConnected(next: Next<void>): Subscription {
+  onConnect(next: Next<void>): Subscription {
     return this.stompService.connectionState$.pipe(filter(state => state === RxStompState.OPEN)).subscribe(_ => next());
   }
 
-  websocketDisconnected(next: Next<void>): Subscription {
+  onDisconnect(next: Next<void>): Subscription {
     return this.stompService.connectionState$.pipe(filter(state => state !== RxStompState.OPEN)).subscribe(_ => next());
   }
 
diff --git a/src/main/angular/src/app/location/detail/location-detail.ts b/src/main/angular/src/app/location/detail/location-detail.ts
index e97bf42..634eb45 100644
--- a/src/main/angular/src/app/location/detail/location-detail.ts
+++ b/src/main/angular/src/app/location/detail/location-detail.ts
@@ -13,6 +13,7 @@ import {SeriesHistory} from './history/series-history';
 import {Interval} from '../../series/Interval';
 import {MenuService} from '../../menu-service';
 import {DatePipe} from '@angular/common';
+import {WebsocketService} from '../../common';
 
 function yesterday(now: any) {
   const yesterday = new Date(now.getTime());
@@ -56,19 +57,26 @@ export class LocationDetail implements OnInit, OnDestroy {
     readonly seriesService: SeriesService,
     readonly activatedRoute: ActivatedRoute,
     readonly menuService: MenuService,
+    readonly websocketService: WebsocketService,
     @Inject(LOCALE_ID) readonly locale: string,
   ) {
     this.datePipe = new DatePipe(locale);
   }
 
   ngOnInit(): void {
-    this.activatedRoute.params.subscribe(params => {
+    this.subs.push(this.websocketService.onChange((connected) => {
+      if (connected) {
+        this.seriesService.findAll(list => this.series = list);
+      } else {
+        this.location = null;
+      }
+    }));
+    this.subs.push(this.activatedRoute.params.subscribe(params => {
       this.locationService.getById(params['id'], location => {
         this.location = location;
         this.menuService.title = this.location.name;
       });
-    });
-    this.seriesService.findAll(list => this.series = list);
+    }));
     this.subs.push(this.seriesService.subscribe(this.updateSeries));
     this.subs.push(timer(1000, 1000).subscribe(() => {
       this.now = new Date();
@@ -77,6 +85,7 @@ export class LocationDetail implements OnInit, OnDestroy {
   }
 
   ngOnDestroy(): void {
+    this.location = null;
     this.menuService.title = "";
     this.subs.forEach(sub => sub.unsubscribe());
   }