Skip to content

Allow VEnum types in the concat() function - #3935

Open
rjwills28 wants to merge 3 commits into
ControlSystemStudio:masterfrom
rjwills28:parse_enums_in_concat_function
Open

rjwills28 wants to merge 3 commits into
ControlSystemStudio:masterfrom
rjwills28:parse_enums_in_concat_function

Conversation

@rjwills28

Copy link
Copy Markdown
Contributor

I was asked by a user why the concat() function was returning an empty string when trying to concat 2 PVs. I realized that there PVs were VEnums, which aren't support by the concat() function. However, the labels are Strings and so it makes sense that we should be able to concat them.

I have made the changes to check if the type is VEnum and parse the value correctly.

I have also added a unit tests for the concat() function testing the VEnum and other types.

Checklist

  • Testing:

    • The feature has automated tests
    • Tests were run
    • If not, explain how you tested your changes
  • Documentation:

    • The feature is documented
    • The documentation is up to date
    • Release notes:
      • Added an entry if the change is breaking or significant
      • Added an entry when adding a new feature

@rjwills28
rjwills28 force-pushed the parse_enums_in_concat_function branch from a7589f4 to 9840c3d Compare September 22, 2026 14:38
@sonarqubecloud

Copy link
Copy Markdown

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant