Prestashop: Remove smarty from engine template for PrestaShop and use twig

Created on 25 Oct 2019  路  5Comments  路  Source: PrestaShop/PrestaShop

Is your feature request related to a problem? Please describe.
please remove smarty from engine template for Prestashop and use twig

Describe the solution you'd like
A clear and concise description of what you want to happen.

Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.

Additional context
Add any other context or screenshots about the feature request here.

CO Improvement TBS

Most helpful comment

This is already happening in the back-office 馃槈, we are migrating BO pages one by one from legacy prestashop framework to Symfony, and each new page uses Twig instead of Smarty.

However we cannot remove Smarty from front-office now as it would be a major BC break that would make all current FO themes totally useless. This would be a very harsh move for the community and a violation of SemVer :/ .

When we release next major version (= BC breaks allowed) we might drop smarty for twig, this has not been decided yet. Most of us would be happy to perform this move, however we need to consider the community and make sure this is in the best interest of both the software and its users.

All 5 comments

Hi @heithemmoumni,

Ping @PrestaShop/prestashop-core-developers what do you think?

Thanks!

This is already happening in the back-office 馃槈, we are migrating BO pages one by one from legacy prestashop framework to Symfony, and each new page uses Twig instead of Smarty.

However we cannot remove Smarty from front-office now as it would be a major BC break that would make all current FO themes totally useless. This would be a very harsh move for the community and a violation of SemVer :/ .

When we release next major version (= BC breaks allowed) we might drop smarty for twig, this has not been decided yet. Most of us would be happy to perform this move, however we need to consider the community and make sure this is in the best interest of both the software and its users.

Hi @heithemmoumni,

So, can we close this ticket?

Thanks!

@khouloudbelguith If there is no other issue for Twig in Front office, we can't keep this one but the title needs to be changed.

@khouloudbelguith, @matks for you response.

Was this page helpful?
0 / 5 - 0 ratings