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(Translation): Objects in context caused error #3

Merged
merged 1 commit into from
May 4, 2021
Merged

fix(Translation): Objects in context caused error #3

merged 1 commit into from
May 4, 2021

Conversation

derikb
Copy link
Contributor

@derikb derikb commented May 3, 2021

Closes #2

Fixes the error when there is an object in the $context. Also checks for brackets before even bothering to run all those str_replace (which could be substantial if your context has a lot of vars in it.

Hope I did this ok? We were just starting to try this out and it was all working fine until I loaded a template with a few objects in it.

Not sure if master is the appropriate target, but since its the only branch...

@JBlond
Copy link
Owner

JBlond commented May 4, 2021

Thank you for your contribution.

@JBlond JBlond merged commit eeb7356 into JBlond:master May 4, 2021
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.

Bug: Objects in $context cause error
2 participants