Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support list and context values for string() function #650

Closed
saig0 opened this issue May 30, 2023 · 2 comments · Fixed by #782
Closed

Support list and context values for string() function #650

saig0 opened this issue May 30, 2023 · 2 comments · Fixed by #782

Comments

@saig0
Copy link
Member

saig0 commented May 30, 2023

Describe the bug
Currently, the string() function doesn't work for list or context values. Instead, it returns null.

To Reproduce
Steps to reproduce the behavior:

  1. Evaluate the expression string([1,2,3])
  2. Verify that the result is null

Expected behavior
The string() function returns the string representation of list and context values. (see related DMN TCK case).

string([1, 2, 3])
// "[1, 2, 3]"

string({x: 1})
// "{x: 1}"

Environment

  • FEEL engine version: 1.16.0
  • Affects:
    • Camunda Automation Platform 7: [7.x]
    • Zeebe broker: [0.x]
@saig0
Copy link
Member Author

saig0 commented Jun 14, 2023

Related OMG issue:

@saig0
Copy link
Member Author

saig0 commented Sep 21, 2023

The issue was fixed by #717 in version 1.17.0 as a side effect. 🚀

We need to add a few test cases to verify the behavior of the string() function before closing the issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant