[abrahamwolk]

Summary. This pull request implements the display of description for values of types VEnum, VBoolean, and VString in the PV Table application in Phoebus. The implementation depends on the changes to the epics-vtype library implemented in epics-base/epicsCoreJava#146.

Handling the possible presence of AdvancedEnumDisplay in values of type VEnum. One complication in the implementation arises from the possible presence of AdvancedEnumDisplay in instances of VEnum. This is handled by a special case of the switch-expression that computes the description in PVTableItem:

phoebus/app/pvtable/src/main/java/org/phoebus/applications/pvtable/model/PVTableItem.java

Line 207 in 8d869c7

case VEnum vEnum when vEnum.getDisplay() instanceof DescriptionProvider descriptionProvider -> descriptionProvider.getDescription().orElse("");

This mirrors the existing implementation, which also treats this as a special case, but it would be preferable to have a unified mechanism for all VTypes.

Testing. I have tested the change manually for instances of VEnum, VBoolean, and VString that contain descriptions. I have not tested the changes for Tango values, since I don't know how to generate a Tango data source for testing.

Dependency on changes in epics-vtype. This merge request depends on the changes in epics-base/epicsCoreJava#146, and should not be merged before those changes have been merged, made part of a release, and the dependency on epics-vtype in this pull request has been updated to point to a release of epics-vtype that contains the changes.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud