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

Fix END-IF error message for unexpected END-FOR #319

Closed
wants to merge 1 commit into from

Conversation

davidjb
Copy link
Contributor

@davidjb davidjb commented Jul 25, 2023

If a template an errant END-FOR command, the current behaviour is to throw an error with the text:

Unexpected END-IF outside of IF statement context

However, the command isn't an END-IF and it isn't within an IF context - the message is just hardcoded to state IF.

This PR changes the outputted error message to report the command name used and corresponding context.

This can be reproduced with a template like this, with just the following text and nothing else:

+++END-FOR person+++

@davidjb
Copy link
Contributor Author

davidjb commented Jul 28, 2023

Closing in lieu of #321. I accidentally created the same PR twice 🫤

@davidjb davidjb closed this Jul 28, 2023
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