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

Fix directory listing #8755

Merged
merged 1 commit into from Feb 20, 2018

Conversation

Projects
None yet
3 participants
@alegout
Copy link
Contributor

alegout commented Feb 13, 2018

Questions Answers
Branch? 1.7.2.x
Description? Very long work, fix directory listing.
Type? bug fix
Category? BO
BC breaks? no
Deprecations? no
Fixed ticket? BOOM-4803
How to test? admin-dev/filemanager/dialog.php?type=1&lang=fr&popup=0&field_id=0&fldr=.. should not list parent dir (which is the img projet_root/img)

Many thanks to Robin Peraglie from RIPS Technologies.

Important guidelines


This change is Reviewable

@eternoendless eternoendless changed the title BO: Fix directory listing Fix directory listing Feb 15, 2018

@eternoendless eternoendless added this to the 1.7.2.5 milestone Feb 15, 2018

@marionf marionf added QA ✔️ and removed waiting for QA labels Feb 19, 2018

@eternoendless

This comment has been minimized.

Copy link
Member

eternoendless commented Feb 20, 2018

Thank you @alegout

@eternoendless eternoendless merged commit 74877e4 into PrestaShop:1.7.2.x Feb 20, 2018

2 checks passed

Codacy/PR Quality Review Good work! A positive pull request.
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment