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

Construct expression with multiple EvaluationOptions #30

Open
Bykiev opened this issue Aug 19, 2021 · 2 comments
Open

Construct expression with multiple EvaluationOptions #30

Bykiev opened this issue Aug 19, 2021 · 2 comments
Labels

Comments

@Bykiev
Copy link
Contributor

Bykiev commented Aug 19, 2021

@yallie, first of all, thank you for this awesome library!

We need to use multiple EvaluationOptions to disable caching and enable IgnoreCase. It seems when EvaluateOptions.NoCache | EvaluateOptions.IgnoreCase is used the first one is ignored and only IgnoreCase is applied. How to apply both of them?

@yallie
Copy link
Member

yallie commented Aug 19, 2021

@Bykiev Hello, I'm not the author, all credits to Sébastien Ros :)
I just picked up the abandoned library and hosted it here because I use it in my projects.

It seems when EvaluateOptions.NoCache | EvaluateOptions.IgnoreCase is used the first one is ignored and only IgnoreCase is applied.

Looks like a bug. Would you mind creating a unit test demonstrating the issue?

Thanks!

@Bykiev
Copy link
Contributor Author

Bykiev commented Aug 20, 2021

Thank you, I did some research and tests, but enums parsed correctly. For some reason, when the caching is enabled, it calcs incorrectly, but if NoCache option specified, all is working. When cache is disabled via Expression and EvaluateOptions.IgnoreCase specified it calcs incorrectly. Still investigating the issue...

@gumbarros gumbarros added bug needs author feedback Needs feedback from the author to be closed or worked on. and removed needs author feedback Needs feedback from the author to be closed or worked on. labels Apr 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants