Merge pull request #3228 from katysaintin/master

shroffk · web-flow · commit dd813fc07cfe · 2025-01-14T13:33:57.000-05:00
Fix issue in app-pvtable for PV that contains dot in the name the last part of the name is removed
diff --git a/app/pvtable/src/main/java/org/phoebus/applications/pvtable/model/PVTableItem.java b/app/pvtable/src/main/java/org/phoebus/applications/pvtable/model/PVTableItem.java
@@ -18,6 +18,8 @@
 import org.epics.vtype.Alarm;
 import org.epics.vtype.AlarmSeverity;
 import org.epics.vtype.AlarmStatus;
+import org.epics.vtype.Display;
+import org.epics.vtype.DisplayProvider;
 import org.epics.vtype.Time;
 import org.epics.vtype.VByteArray;
 import org.epics.vtype.VEnum;
@@ -30,6 +32,7 @@
 import org.phoebus.core.vtypes.VTypeHelper;
 import org.phoebus.pv.PV;
 import org.phoebus.pv.PVPool;
+import org.phoebus.pv.PVPool.TypedName;
 
 import io.reactivex.rxjava3.disposables.Disposable;
 
@@ -51,7 +54,8 @@ public class PVTableItem
     private volatile VType value;
 
     /** Value of the PV's description */
-    private volatile String desc_value = "";
+    private volatile String desc_value = null;
+    private volatile String desc_name = "";
 
     /** Saved (snapshot) value */
     private volatile Optional<SavedValue> saved = Optional.empty();
@@ -77,6 +81,9 @@ public class PVTableItem
 
     /** Listener to description PV */
     private volatile Disposable desc_flow;
+    
+    private static final String DESC_FIELD = "DESC";
+    private static final String DOT = ".";
 
     /** Initialize
      *
@@ -122,16 +129,24 @@ private void createPVs(final String name)
             updateValue(null);
             return;
         }
+        PV new_pv = null;
         try
         {
             updateValue(VString.of("", Alarm.disconnected(), Time.now()));
-            final PV new_pv = PVPool.getPV(name);
+            new_pv = PVPool.getPV(name);
             value_flow = new_pv.onValueEvent()
                                .throttleLatest(Settings.max_update_period_ms, TimeUnit.MILLISECONDS)
                                .subscribe(this::updateValue);
             permission_flow = new_pv.onAccessRightsEvent()
                                     .subscribe(writable -> listener.tableItemChanged(PVTableItem.this));
             pv.set(new_pv);
+            // read the value for getting description
+            if (new_pv != null) {
+                    VType newVal = new_pv.read();
+                    if(newVal != null){
+                        updateValue(newVal);
+                    }
+            }
         }
         catch (Exception ex)
         {
@@ -141,19 +156,25 @@ private void createPVs(final String name)
                                    Time.now()));
         }
 
-        // For CA PVs, check the .DESC field
-        // Hardcoded knowledge to avoid non-record PVs.
-        if (Settings.show_description &&
-            ! (name.startsWith("sim:") ||
-               name.startsWith("loc:")))
-        {
-            // Determine DESC field.
-            // If name already includes a field,
-            // replace it with DESC field.
-            final int sep = name.lastIndexOf('.');
-            final String desc_name = sep >= 0
-                    ? name.substring(0, sep) + ".DESC"
-                    : name + ".DESC";
+        // First try to get description from value or pv
+        updateDescription();
+        // If still no description found and channel access source
+        final TypedName type_name = TypedName.analyze(name);
+        String dataType = type_name != null ? type_name.type : null;
+        boolean channelAccess = dataType.equals("ca");
+        if (Settings.show_description && desc_value == null && channelAccess) {
+            // For CA PVs, check the .DESC field
+            // Hardcoded knowledge to avoid non-record PVs.
+            // First get default datasource
+            desc_name = name + DOT + DESC_FIELD; // by default add .DESC
+            if (!name.endsWith(DOT + DESC_FIELD) && name.contains(DOT)) {
+                final int sep = name.lastIndexOf('.');
+                String fieldVal = name.substring(sep + 1);
+                // then replace by .DESC
+                // Determine DESC field include dot in case of variable name such as variableEGUName
+                desc_name = name.replace(DOT + fieldVal, DOT + DESC_FIELD);
+            }
+            
             try
             {
                 final PV new_desc_pv = PVPool.getPV(desc_name);
@@ -175,6 +196,27 @@ private void createPVs(final String name)
             }
         }
     }
+    
+    private void updateDescription() {
+        if(desc_value == null) {
+            //update description from value or pv
+            VType currentValue = getValue();
+            if(currentValue != null) {
+                PV thePV = pv.get();
+             // DisplayProvider is an optional interface for VType values,
+             // not PVs, but the custum datasource as Muscade happens to implement
+             // DisplayProvider for enum and bool PVs, so check for that here 
+                Display display =  thePV instanceof DisplayProvider ? ((DisplayProvider) thePV).getDisplay() : null;
+                display = display == null && currentValue instanceof DisplayProvider ? ((DisplayProvider) currentValue).getDisplay(): display;
+                if (display != null) {
+                    String description = display.getDescription();
+                    desc_value = description != null ? description : null;
+                    desc_name = desc_value != null ? "Description of " + name + " PV" : "no description";
+                    desc_flow = value_flow;
+                }
+            }
+        }
+    }
 
     /** @return <code>true</code> if item is selected to be restored */
     public boolean isSelected()
@@ -240,9 +282,13 @@ public VType getValue()
     }
 
     /** @return Description */
-    public String getDescription()
-    {
-        return desc_value;
+    public String getDescription() {
+        return desc_value == null ? "" : desc_value;
+    }
+    
+    /** @return description pv name **/
+    public String getDescriptionName() {
+        return desc_name;
     }
 
     /** @return Enum options for current value, <code>null</code> if not enumerated */
@@ -285,14 +331,17 @@ public void setValue(String new_value)
                 throw new Exception("Not connected");
 
             final VType pv_type = the_pv.read();
-            if (pv_type instanceof VNumber)
-            {
-                if (Settings.show_units)
-                {   // Strip units so that only the number gets written
-                    final String units = ((VNumber)pv_type).getDisplay().getUnit();
-                    if (units.length() > 0  &&  new_value.endsWith(units))
+            Display display = the_pv instanceof DisplayProvider ? ((DisplayProvider) the_pv).getDisplay() : null;
+            display = display == null && pv_type instanceof DisplayProvider ? ((DisplayProvider) pv_type).getDisplay():null;
+
+            if (display != null && Settings.show_units) {
+                // Strip units so that only the number gets written
+                    final String units = display.getUnit();
+                    if (units.length() > 0 && new_value.endsWith(units))
                         new_value = new_value.substring(0, new_value.length() - units.length()).trim();
                 }
+            if (pv_type instanceof VNumber)
+            {
                 the_pv.write(Double.parseDouble(new_value));
             }
             else if (pv_type instanceof VEnum)
diff --git a/app/pvtable/src/main/java/org/phoebus/applications/pvtable/ui/Messages.java b/app/pvtable/src/main/java/org/phoebus/applications/pvtable/ui/Messages.java
@@ -14,6 +14,7 @@ public class Messages
 {
     /** Externalized strings */
     public static String Alarm,
+                         AddPVList,
                          Description,
                          CheckAll,
                          CheckAll_TT,
diff --git a/app/pvtable/src/main/java/org/phoebus/applications/pvtable/ui/PVTable.java b/app/pvtable/src/main/java/org/phoebus/applications/pvtable/ui/PVTable.java
@@ -69,7 +69,9 @@
 import javafx.scene.control.TableRow;
 import javafx.scene.control.TableView;
 import javafx.scene.control.TableView.TableViewSelectionModel;
+import javafx.scene.control.TextArea;
 import javafx.scene.control.TextField;
+import javafx.scene.control.TextInputControl;
 import javafx.scene.control.ToolBar;
 import javafx.scene.control.Tooltip;
 import javafx.scene.control.cell.TextFieldTableCell;
@@ -83,8 +85,6 @@
 import javafx.scene.paint.Color;
 import javafx.util.converter.DefaultStringConverter;
 
-import static org.phoebus.ui.application.PhoebusApplication.logger;
-
 
 /** PV Table and its toolbar
  *  @author Kay Kasemir
@@ -95,6 +95,7 @@ public class PVTable extends VBox
     private static final String comment_style = "-fx-text-fill: blue;";
     private static final String new_item_style = "-fx-text-fill: gray;";
     private static final String changed_style = "-fx-background-color: -fx-table-cell-border-color, cyan;-fx-background-insets: 0, 0 0 1 0;";
+    private static final String SPLIT_PV = "[ \\t\\n\\r,]+";
 
     /** When sorting, keep the 'NEW_ITEM' row at the bottom **/
     private static final Comparator<TableItemProxy> SORT_NEW_ITEM_LAST = (a, b) ->
@@ -164,12 +165,31 @@ protected void updateItem(final Boolean selected, final boolean empty)
             }
         }
     }
+    
+    private static class DescriptionTableCell extends TableCell<TableItemProxy, String>
+    {
+        @Override
+        protected void updateItem(final String item, final boolean empty)
+        {
+            super.updateItem(item, empty);
+            setText(item);
+            final int row = getIndex();
+            final List<TableItemProxy> items = getTableView().getItems();
+            if (! empty  &&  row >= 0  &&  row < items.size()) {
+                final TableItemProxy itemCell = items.get(row);
+                if(itemCell != null && itemCell.getItem() != null) {
+                    setTooltip(new Tooltip(itemCell.getItem().getDescriptionName()));
+                }
+            }
+        }
+    }
 
     /** Table cell for 'name' column, colors comments */
     private static class PVNameTableCell extends TextFieldTableCell<TableItemProxy, String>
     {
-        private TextField textField;
-
+        private TextInputControl textField;
+        private static ContextMenu contextMenu;
+        
         public PVNameTableCell()
         {
             super(new DefaultStringConverter());
@@ -179,11 +199,26 @@ public PVNameTableCell()
         public void startEdit()
         {
             super.startEdit();
-            textField = new TextField();
-            textField.setOnAction(event -> commitEdit(textField.getText()));
+            final int index = getIndex();
+            boolean newPv = index == getTableView().getItems().size() - 1;
+            if(newPv) {
+                textField = new TextArea();
+                textField.setMaxHeight(100);
+                if(contextMenu == null) {
+                    MenuItem addPVMenu = new MenuItem(Messages.AddPVList);
+                    addPVMenu.setOnAction(event -> commitEdit(textField.getText()));
+                    contextMenu = new ContextMenu(addPVMenu);
+                }
+                textField.setContextMenu(contextMenu);
+            }
+            else {
+                textField = new TextField();
+                ((TextField)textField).setOnAction(event -> commitEdit(textField.getText()));
+            }
             PVAutocompleteMenu.INSTANCE.attachField(textField);
             showCurrentValue();
         }
+      
 
         private void showCurrentValue()
         {
@@ -759,8 +794,14 @@ private void createTableColumns()
             final TableItemProxy proxy = event.getRowValue();
             if (proxy == TableItemProxy.NEW_ITEM)
             {
-                if (!new_name.isEmpty())
-                    model.addItem(new_name);
+                if (!new_name.isEmpty()) {
+                    //Can be a list of pv
+                    final String[] pvs = new_name.split(SPLIT_PV);
+                    //Add a list
+                    for(String pv : pvs) {
+                         model.addItem(pv);
+                    }
+                }
                 // else: No name entered, do nothing
             }
             else
@@ -784,6 +825,7 @@ private void createTableColumns()
         {
             col = new TableColumn<>(Messages.Description);
             col.setCellValueFactory(cell -> cell.getValue().desc_value);
+            col.setCellFactory(column -> new DescriptionTableCell());
             table.getColumns().add(col);
         }
 
@@ -959,7 +1001,7 @@ else if (db.hasString())
 
     private void addPVsFromString(final PVTableItem existing, final String pv_text)
     {
-        final String[] pvs = pv_text.split("[ \\t\\n\\r,]+");
+        final String[] pvs = pv_text.split(SPLIT_PV);
         for (String pv : pvs)
             if (! pv.isEmpty())
                 model.addItemAbove(existing, pv);
diff --git a/app/pvtable/src/main/resources/org/phoebus/applications/pvtable/ui/messages.properties b/app/pvtable/src/main/resources/org/phoebus/applications/pvtable/ui/messages.properties
@@ -1,4 +1,5 @@
 Alarm=Alarm
+AddPVList=Add a list of pv
 Description=Description
 CheckAll=Check All
 CheckAll_TT=Check all PVs in table
diff --git a/app/pvtable/src/main/resources/org/phoebus/applications/pvtable/ui/messages_fr.properties b/app/pvtable/src/main/resources/org/phoebus/applications/pvtable/ui/messages_fr.properties
@@ -1,4 +1,5 @@
 Alarm=Alarme
+AddPVList=Ajout d une liste de pv
 Description=Description
 CheckAll=Tout sélectionner
 CheckAll_TT=Sélectionner tous les PVs dans le tableau
diff --git a/jitpack.yml b/jitpack.yml
@@ -0,0 +1,5 @@
+before_install:
+  # update maven version 3.9.9
+  - sdk install maven 3.9.9
+install:
+   - mvn install -DskipTests

Original file line number	Diff line number	Diff line change
`@@ -14,6 +14,7 @@ public class Messages`
`14`	`14`	`{`
`15`	`15`	`/** Externalized strings */`
`16`	`16`	`public static String Alarm,`
	`17`	`+ AddPVList,`
`17`	`18`	`Description,`
`18`	`19`	`CheckAll,`
`19`	`20`	`CheckAll_TT,`
Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,5 @@`
`1`	`1`	`Alarm=Alarm`
	`2`	`+AddPVList=Add a list of pv`
`2`	`3`	`Description=Description`
`3`	`4`	`CheckAll=Check All`
`4`	`5`	`CheckAll_TT=Check all PVs in table`
Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,5 @@`
`1`	`1`	`Alarm=Alarme`
	`2`	`+AddPVList=Ajout d une liste de pv`
`2`	`3`	`Description=Description`
`3`	`4`	`CheckAll=Tout sélectionner`
`4`	`5`	`CheckAll_TT=Sélectionner tous les PVs dans le tableau`