Prestashop: Deleting languague fails with open_basedir restriction in effect

[rozwell]

Describe the bug

Unable to delete a language because it fails on .htaccess and index.php files from PS_MODULE_DIR due to lack of testing whenever the path is actually a directory or not with open_basedir restriction in effect.

Expected behavior

Error should not be thrown, language shall be deleted.

Steps to Reproduce

Steps to reproduce the behavior:

1. Go to International => Localization => Languages
2. Click on 'delete' on a chosen language
3. Click OK to confirm
4. See error

Additional information

• PrestaShop version: develop/1.7.7.x
• PHP version: 7.2.32 with open_basedir restriction

[prestashop-issue-bot[bot]]

Thanks for opening this issue! We will help you to keep its state consistent

[NeOMakinG]

@PrestaShop/prestashop-product-team should this be in the 177 scope or develop ? Feel free to change the PR description with the right answer!

[khouloudbelguith]

Hi @rozwell,

Have you this exception when deleting a language?

Thanks!

[marionf]

@NeOMakinG We need to reproduce the issue before, make sure it's not a regression and qualify it
The version mentionned here is the affected version, not the version where the issue will be fixed

[khouloudbelguith]

@marionf, @NeOMakinG,

I followed these steps:
set the open_basedir value (ex open_basedir = '/var/www/html/Prestashop')
Try to delete a language
I have this exception

I just reload the page
I have this error

I have the same issue with PS1.7.6.7 and 177x branch.
But I'm not sure if it is a PrestaShop core issue or not.

Thanks!

[rozwell]

Hi @rozwell,

Have you this exception when deleting a language?

Hi @khouloudbelguith,
Yes, that's the issue, just in my case it was failing on .htaccess file first, then index.php.

[khouloudbelguith]

@rozwell, thanks for your feedback.
I manage to reproduce the issue with PS1.7.6.7 and 177x branch.
I’ll add this to the debug roadmap so that it’s fixed. If you have already fixed it on your end or if you think you can do it, please do send us a pull request!
Thanks!

[rozwell]

@khouloudbelguith, I've already sent a pull request for it #20681 :)