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.

Sign up for GitHub

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

Jump to bottom

else if condition not being parsed #2

Open
Ahmedfir opened this issue Jun 12, 2024 · 2 comments
Open

else if condition not being parsed #2

Ahmedfir opened this issue Jun 12, 2024 · 2 comments

Comments

@Ahmedfir
Copy link
Owner

Thank you @Asma-Ha for reporting this bug. I have just added two tests in CliRequestTest revealing it (I commented out the failing assertions). If you want, you can propose a pull request to fix it.
Ahmedfir added a commit that referenced this issue Jun 12, 2024
@Ahmedfir
added 2 tests that fail because of issue #2: else if condition not be…
4c44bd6 
…ing parsed.
@Ahmedfir
Copy link
Owner Author

Fix in progress

Ahmedfir added a commit that referenced this issue Jun 13, 2024
@Ahmedfir
added one test of else if with brackets to fix with #2
d327de7
@Ahmedfir
Copy link
Owner Author

@Asma-Ha I have added a few more tests. I found another issue: when a return statement is in the same line as the if, the returned value is not masked. e.g.: if (CDT) return VAL; --> VAL is ignored.

Ahmedfir added a commit that referenced this issue Jun 19, 2024
@Ahmedfir
Merge pull request #4 from Ahmedfir/experimental
f9d65c1 
bug fixing #2 and implementation of full conditions masking
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

When branches are created from issues, their pull requests are automatically linked.

2-else-if-condition-not-being-parsed Ahmedfir/java-business-locations
1 participant
@Ahmedfir