From a23fb5d7bb424ea40426faf35a4c7ee9550c46a6 Mon Sep 17 00:00:00 2001
From: lcaouen <loic.caouen@cea.fr>
Date: Fri, 3 Apr 2026 15:43:17 +0200
Subject: [PATCH] incorrect pv name added to the new pv map

---
 .../builder/runtime/WidgetRuntime.java        | 49 ++++++++++---------
 .../runtime/WidgetRuntimeException.java       | 11 +++++
 2 files changed, 38 insertions(+), 22 deletions(-)
 create mode 100644 app/display/runtime/src/main/java/org/csstudio/display/builder/runtime/WidgetRuntimeException.java

diff --git a/app/display/runtime/src/main/java/org/csstudio/display/builder/runtime/WidgetRuntime.java b/app/display/runtime/src/main/java/org/csstudio/display/builder/runtime/WidgetRuntime.java
index b065d77d28..e495a5e697 100644
--- a/app/display/runtime/src/main/java/org/csstudio/display/builder/runtime/WidgetRuntime.java
+++ b/app/display/runtime/src/main/java/org/csstudio/display/builder/runtime/WidgetRuntime.java
@@ -65,11 +65,6 @@ public class WidgetRuntime<MW extends Widget> {
      */
     public final static Logger logger = Logger.getLogger(WidgetRuntime.class.getPackageName());
 
-    /**
-     * Extension point for contributing custom widget runtime
-     */
-    public static final String EXTENSION_POINT = "org.csstudio.display.builder.runtime.widgets";
-
     /**
      * The widget handled by this runtime
      */
@@ -83,7 +78,7 @@ public class WidgetRuntime<MW extends Widget> {
     /**
      * start() involves background jobs to start script support etc.
      * This latch indicates that they have completed
-     * and lazily set variables (action_scripts, writable_pvs, ..)
+     * and lazily set variables (action_scripts, writable_pvs, ...)
      * can now be used.
      */
     private volatile CountDownLatch started = new CountDownLatch(1);
@@ -237,7 +232,8 @@ public void start() {
                     try {
                         final String expanded = MacroHandler.replace(widget.getMacrosOrProperties(), pv_name);
                         final RuntimePV pv = PVFactory.getPV(expanded);
-                        action_pvs.put(expanded, pv);
+                        String cleanPvName = getCleanPvName(expanded);
+                        action_pvs.put(cleanPvName, pv);
                         addPV(pv, true);
                     } catch (Exception ex) {
                         logger.log(Level.WARNING, widget + " cannot start action to write PV '" + pv_name + "'", ex);
@@ -258,6 +254,26 @@ public void start() {
             started.countDown();
     }
 
+    /**
+     * Get PV Name without initialisation value
+     */
+    private String getCleanPvName(String expandedName) {
+
+        String nameToCheck = expandedName;
+        // For local PV,
+        if (nameToCheck.startsWith("loc://")) {
+            // strip optional data type ...
+            int sep = nameToCheck.indexOf('<');
+            if (sep > 0)
+                nameToCheck = nameToCheck.substring(0, sep);
+            // or initializer ...
+            sep = nameToCheck.indexOf('(');
+            if (sep > 0)
+                nameToCheck = nameToCheck.substring(0, sep);
+        }
+        return nameToCheck;
+    }
+
     /**
      * Are Scripts or Rules defined for Widget?
      */
@@ -395,31 +411,20 @@ public void writePrimaryPV(final Object value) {
      */
     public void writePV(final String pv_name, final Object value) throws Exception {
         final String expanded = MacroHandler.replace(widget.getMacrosOrProperties(), pv_name);
-        String name_to_check = expanded;
-        // For local PV,
-        if (name_to_check.startsWith("loc://")) {
-            // strip optional data type ...
-            int sep = name_to_check.indexOf('<');
-            if (sep > 0)
-                name_to_check = name_to_check.substring(0, sep);
-            // or initializer ..
-            sep = name_to_check.indexOf('(');
-            if (sep > 0)
-                name_to_check = name_to_check.substring(0, sep);
-        }
+        String nameToCheck = getCleanPvName(expanded);
         awaitStartup();
         final Map<String, RuntimePV> safe_pvs = writable_pvs;
         if (safe_pvs != null) {
-            final RuntimePV pv = safe_pvs.get(name_to_check);
+            final RuntimePV pv = safe_pvs.get(nameToCheck);
             if(pv != null) {
                     try {
                         pv.write(value);
                     } catch (final Exception ex) {
-                        throw new Exception("Failed to write " + value + " to PV " + name_to_check, ex);
+                        throw new WidgetRuntimeException("Failed to write " + value + " to PV " + nameToCheck, ex);
                     }
                 }
             else {
-                throw new Exception("Unknown PV '" + pv_name + "' (expanded: '" + name_to_check + "')");
+                throw new WidgetRuntimeException("Unknown PV '" + pv_name + "' (expanded: '" + nameToCheck + "')");
             }
         }
     }
diff --git a/app/display/runtime/src/main/java/org/csstudio/display/builder/runtime/WidgetRuntimeException.java b/app/display/runtime/src/main/java/org/csstudio/display/builder/runtime/WidgetRuntimeException.java
new file mode 100644
index 0000000000..b9265057c0
--- /dev/null
+++ b/app/display/runtime/src/main/java/org/csstudio/display/builder/runtime/WidgetRuntimeException.java
@@ -0,0 +1,11 @@
+package org.csstudio.display.builder.runtime;
+
+public class WidgetRuntimeException extends Exception {
+    public WidgetRuntimeException(String message) {
+        super(message);
+    }
+
+    public WidgetRuntimeException(String message, Throwable cause) {
+        super(message, cause);
+    }
+}