Skip to content

Fix the paths of the deleted files, which were absolute but invalid paths#1543

Merged
Quetzacoalt91 merged 1 commit intoPrestaShop:devfrom
Quetzacoalt91:file-paths-of-deleted-files
Jan 7, 2026
Merged

Fix the paths of the deleted files, which were absolute but invalid paths#1543
Quetzacoalt91 merged 1 commit intoPrestaShop:devfrom
Quetzacoalt91:file-paths-of-deleted-files

Conversation

@Quetzacoalt91
Copy link
Member

Questions Answers
Description? Fix the build of the backlog of files updates, where the deleted files were not a valid absolute path to the unzipped archive of PrestaShop
Type? bug fix
BC breaks? Nope
Deprecations? Nope
Fixed ticket? Fixes #1552
Sponsor company @PrestaShopCorp
How to test? Files to delete are not triggering anymore errors during an update. To reproduce this issue, the root path of the webserver files must be added to the basedir. For instance, this is /var/www/html on a Docker environment, so in the php.ini you would get:
open_basedir = "/var/www/html/:/tmp"

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

sonarqubecloud bot commented Dec 4, 2025

@AureRita
Copy link
Contributor

AureRita commented Dec 8, 2025

Hi @Quetzacoalt91

Thank you for your PR, I tested it and it seems to works as you can see :

Capture.video.du.2025-12-08.16-29-43.mp4

Tested from :
8.2.1 to 9.0.1

Currently, there is no other Approval on this PR, so we'll wait for other Approve to put the QA Approved.

Thank you

@Quetzacoalt91 Quetzacoalt91 added the QA ✔️ Status: Check done, Code approved label Dec 10, 2025
@Quetzacoalt91 Quetzacoalt91 added this to the 7.6.0 milestone Dec 10, 2025
@Quetzacoalt91 Quetzacoalt91 added the Blocked Status: The issue is blocked by another task label Dec 10, 2025
@Quetzacoalt91 Quetzacoalt91 removed the Blocked Status: The issue is blocked by another task label Jan 7, 2026
@Quetzacoalt91 Quetzacoalt91 merged commit 2f8b784 into PrestaShop:dev Jan 7, 2026
56 checks passed
@github-project-automation github-project-automation bot moved this from Ready for review to Merged in PR Dashboard Jan 7, 2026
@Quetzacoalt91 Quetzacoalt91 deleted the file-paths-of-deleted-files branch January 7, 2026 11:20
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.

Hundreds of is_link(): open_basedir restriction in effect warnings during upgrade

4 participants