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

date and time does not properly parse as a variable #62

Closed
nikku opened this issue Feb 7, 2024 · 1 comment · Fixed by #63
Closed

date and time does not properly parse as a variable #62

nikku opened this issue Feb 7, 2024 · 1 comment · Fixed by #63
Labels
bug Something isn't working

Comments

@nikku
Copy link
Owner

nikku commented Feb 7, 2024

Describe the Bug

FEEL is context sensitive and as such shall allow users, though not encouraged, to override special names such as date and date and time.

This seems to not be properly supported by feelin at this stage (playground ref)

Steps to Reproduce

Cf. playground ref.

Expected Behavior

With the following context:

{
  "date and time": "DATE AND TIME",
  "date": "DATE"
}

Evaluating the following expression:

{
  foo: date and time,
  bar: date,
  date and time: 10,
  date: 100,
  sum: date and time + date
}

Shall yield the following result:

{
  "foo": "DATE AND TIME",
  "bar": "DATE",
  "date and time": 10,
  "date": 100,
  "sum": 110
}

Environment

  • Host (Browser/Node version), if applicable: [e.g. MS Edge 18, Chrome 69, Node 10 LTS]
  • OS: [e.g. Windows 7]
  • Library version: [e.g. 2.0.0]
@nikku nikku added the bug Something isn't working label Feb 7, 2024
@nikku nikku mentioned this issue Feb 7, 2024
Closed
nikku added a commit to nikku/lezer-feel that referenced this issue Feb 7, 2024
nikku added a commit to nikku/lezer-feel that referenced this issue Feb 7, 2024
@nikku
Copy link
Owner Author

nikku commented Feb 7, 2024

There is two aspects to this:

nikku added a commit that referenced this issue Feb 7, 2024
Caters towards FEEL context sensitive nature.

Closes #62
@nikku nikku closed this as completed in #63 Feb 7, 2024
nikku added a commit that referenced this issue Feb 7, 2024
Caters towards FEEL context sensitive nature.

Closes #62
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant