Skip to content

Update aliases of Environment classes in Container#1595

Merged
Quetzacoalt91 merged 1 commit intoPrestaShop:devfrom
Quetzacoalt91:update-env-aliases
Jan 27, 2026
Merged

Update aliases of Environment classes in Container#1595
Quetzacoalt91 merged 1 commit intoPrestaShop:devfrom
Quetzacoalt91:update-env-aliases

Conversation

@Quetzacoalt91
Copy link
Member

Questions Answers
Description? The previous integration of the Environment class broke the module on specific versions of PrestaShop, because of the proper Twig classes not being loaded.
Type? bug fix
BC breaks? Nope
Deprecations? Nope
Fixed ticket? /
Sponsor company @PrestaShopCorp
How to test? Nightly tests run properly. To check with @nesrineabdmouleh

@Quetzacoalt91 Quetzacoalt91 self-assigned this Jan 14, 2026
@Quetzacoalt91 Quetzacoalt91 added the bug Type: Bug fix label Jan 14, 2026
@github-project-automation github-project-automation bot moved this to Ready for review in PR Dashboard Jan 14, 2026
@sonarqubecloud
Copy link

Copy link
Contributor

@ga-devfront ga-devfront left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oupsy thank you for this quick fix

@nesrineabdmouleh
Copy link
Contributor

Hello @Quetzacoalt91

Thank you for the fix!

Tested with :
PS 1.7.2.1 php 7.1 ✔️
PS 1.7.6.9 php 7.2 ✔️
PS 1.7.8.10 php 7.4 ✔️
PS 1.7.8.11 php 7.2 ✔️
PS 8.2.3 php 8.1 ✔️
PS 9.0.0 php 8.2 ✔️

@nesrineabdmouleh nesrineabdmouleh added QA ✔️ Status: Check done, Code approved and removed waiting for QA labels Jan 27, 2026
@Quetzacoalt91 Quetzacoalt91 added this to the 7.6.0 milestone Jan 27, 2026
@Quetzacoalt91 Quetzacoalt91 merged commit 9a772cb into PrestaShop:dev Jan 27, 2026
64 of 65 checks passed
@github-project-automation github-project-automation bot moved this from Ready for review to Merged in PR Dashboard Jan 27, 2026
@Quetzacoalt91 Quetzacoalt91 deleted the update-env-aliases branch January 27, 2026 15:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Type: Bug fix QA ✔️ Status: Check done, Code approved

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

5 participants