From b175d1b6bd0bc78621a600a2a319f79780cfa99c Mon Sep 17 00:00:00 2001
From: lolodomo <lg.hc@free.fr>
Date: Mon, 1 Jan 2024 16:05:14 +0100
Subject: [PATCH] [BasicUI] Fix image element being link to a sub page (#2238)

Fix #2233

Signed-off-by: Laurent Garnier <lg.hc@free.fr>
---
 .../org.openhab.ui.basic/web-src/smarthome.js | 42 +++++++++++--------
 1 file changed, 25 insertions(+), 17 deletions(-)

diff --git a/bundles/org.openhab.ui.basic/web-src/smarthome.js b/bundles/org.openhab.ui.basic/web-src/smarthome.js
index acfc0f4ec7..c0cc552869 100644
--- a/bundles/org.openhab.ui.basic/web-src/smarthome.js
+++ b/bundles/org.openhab.ui.basic/web-src/smarthome.js
@@ -709,19 +709,25 @@
 		_t.periodButton = null;
 		_t.upscaleButton = null;
 		_t.refreshButton = null;
-		_t.displayLegend = _t.parentNode.getAttribute("data-legend") === "true";
-		_t.period = null;
-		_t.upscale = false;
-
-		_t.legendButton = _t.formHeaderRow.querySelector(o.image.legendButton);
-		_t.periodButton = _t.formHeaderRow.querySelector(o.image.periodButton);
-		_t.upscaleButton = _t.formHeaderRow.querySelector(o.image.upscaleButton);
-		_t.refreshButton = _t.formHeaderRow.querySelector(o.image.refreshButton);
-		if (_t.legendButton !== null) {
-			if (_t.displayLegend) {
-				_t.legendButton.classList.add(o.buttonActiveClass);
-			} else {
-				_t.legendButton.classList.remove(o.buttonActiveClass);
+
+		if (_t.headerRow !== null) {
+			_t.legendButton = _t.formHeaderRow.querySelector(o.image.legendButton);
+			_t.periodButton = _t.formHeaderRow.querySelector(o.image.periodButton);
+			_t.upscaleButton = _t.formHeaderRow.querySelector(o.image.upscaleButton);
+			_t.refreshButton = _t.formHeaderRow.querySelector(o.image.refreshButton);
+			if (_t.legendButton !== null) {
+				_t.displayLegend = _t.parentNode.getAttribute("data-legend") === "true";
+				if (_t.displayLegend) {
+					_t.legendButton.classList.add(o.buttonActiveClass);
+				} else {
+					_t.legendButton.classList.remove(o.buttonActiveClass);
+				}
+			}
+			if (_t.periodButton !== null) {
+				_t.period = null;
+			}
+			if (_t.upscaleButton !== null) {
+				_t.upscale = false;
 			}
 		}
 
@@ -912,8 +918,9 @@
 				componentHandler.downgradeElements([ _t.refreshButton ]);
 				_t.refreshButton.removeEventListener("click", onRefreshClick);
 			}
-
-			_t.parentNode.parentNode.removeEventListener("click", toggleHeaderRow);
+			if (_t.headerRow !== null) {
+				_t.parentNode.parentNode.removeEventListener("click", toggleHeaderRow);
+			}
 		};
 
 		if (_t.legendButton !== null) {
@@ -928,8 +935,9 @@
 		if (_t.refreshButton !== null) {
 			_t.refreshButton.addEventListener("click", onRefreshClick);
 		}
-
-		_t.parentNode.parentNode.addEventListener("click", toggleHeaderRow);
+		if (_t.headerRow !== null) {
+			_t.parentNode.parentNode.addEventListener("click", toggleHeaderRow);
+		}
 
 		if (_t.visible) {
 			_t.activateRefresh();