-
Notifications
You must be signed in to change notification settings - Fork 704
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
new forced unclosed block compiler exception #397
Comments
uwetews
added a commit
to smarty-php/smarty-phpunit
that referenced
this issue
Oct 18, 2017
uwetews
added a commit
that referenced
this issue
Oct 18, 2017
This is fixed now. |
👍 works, thanks! |
wisskid
pushed a commit
that referenced
this issue
Apr 12, 2020
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
in latest dev 3.1.32-25 version brought in some issues with elder working code.
Hi Uwe
I don't say its an issue or bug, and do only want to make this noticable for your further work,
We had to 2 occurences of exceptions using code like
{$link="{if $file.hotlink}{$file.path}{else}{$file.full_file}{/if}"}
throwing an
unclosed "{else}" in doubled quoted string <--
and now easy to solve by using bracketsand a special one in a modifier
{$smarty.const.UNMET_REQUIREMENTS|sprintf:"{if $requirements_failures.{$plug.class_name}.s9y}s9y $plug.requirements.serendipity,{/if} {if $requirements_failures.{$plug.class_name}.php}PHP $plug.requirements.php,{/if} {if $requirements_failures.{$plug.class_name}.smarty}Smarty $plug.requirements.smarty{/if}"}
throwing
unclosed "{if}" in doubled quoted string <--
which also is avoidable using brackets,{$smarty.const.UNMET_REQUIREMENTS|sprintf:"({if ($requirements_failures.{$plug.class_name}.s9y)}s9y {$plug.requirements.serendipity},{/if}{if ($requirements_failures.{$plug.class_name}.php)} PHP {$plug.requirements.php},{/if}{if ($requirements_failures.{$plug.class_name}.smarty)} Smarty {$plug.requirements.smarty}{/if})"}
but outputs the brackets. (That is ok for me this special case, but might be an unwanted behaviour in other cases.)
Feel free to close,
The text was updated successfully, but these errors were encountered: