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 smarty deprecation #32967
fix smarty deprecation #32967
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hello @lmeyer1 ,
This PR is on the develop branch, while from the beginning the issue related to this PR exists only on 8.0.x
So could you change the PR branch !
@lmeyer1 it's weird we already handled this modifier here PrestaShop/config/smarty.config.inc.php Line 114 in 5339457
And it was also fixed on 8.0.x theoretically, but not released yet as a patch I believe https://github.com/PrestaShop/PrestaShop/blob/8.0.x/config/smarty.config.inc.php#L116 So same question did you test with the latest 8.0.x version from the branch? |
@jolelievre well, this changes from using a modifier to using a function directly, not sure if it will be allowed in the future tho 🤔 |
@jolelievre |
@@ -82,7 +82,7 @@ | |||
$("#{$id|escape:'html':'UTF-8'}").tree("collapseAll"); | |||
|
|||
{if isset($selected_categories)} | |||
{assign var=imploded_selected_categories value='","'|implode:$selected_categories} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The code is not equivalent before and after. We must use implode('","', $selected_categories);
Nobody noticed it 😄
22c8ff0
Hasn't this been fixed already in #33136 ? |
@SharakPL Yes, it is. |
|implode
as modifier