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

Replace smarty condition "else if" by "elseif" #605

Merged
merged 3 commits into from
Mar 5, 2024

Conversation

Klemart3D
Copy link
Contributor

@Klemart3D Klemart3D commented Mar 1, 2024

Questions Answers
Description? Replace smarty condition "else if" by "elseif" following official doc
Type? refactor
BC breaks? no
Deprecations? no
Fixed ticket? Partially #604 (must be also applied into Classic theme repo & modules)
Sponsor company Your company or customer's name goes here (if applicable).
How to test? Search for "{else if" in whole .tpl files, there must be "0 results"

@Klemart3D
Copy link
Contributor Author

Klemart3D commented Mar 1, 2024

Pull request also made for Classic theme : PrestaShop/classic-theme#142

@kpodemski
Copy link
Contributor

Thank you @Klemart3D. It doesn't change anything from the functional side of things, so I'll merge it without QA.

@kpodemski kpodemski merged commit 898132d into PrestaShop:develop Mar 5, 2024
6 checks passed
@kpodemski kpodemski added this to the Beta milestone Mar 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

4 participants