?
avatar joeforjoomla
joeforjoomla
15 Oct 2016

Steps to reproduce the issue

Check the field type 'folderlist' in a generic extension parameters:

Expected result

All dropdown values are the names of the folders as always been

Actual result

All dropdown values includes the full path to the folders on Windows

System information (as much as possible)

Joomla 3.6 and later is affected, due to changes in the file root/libraries/joomla/form/fields/folderlist.php

The line 189:
$path = JPATH_ROOT . '/' . $path;

works only for Unix paths, fails on Windows. It should be:

$path = JPATH_ROOT . '\\' . $path;

but should be fixed as:

$path = JPATH_ROOT . DIRECTORY_SEPARATOR . $path;

Additional comments

correct_path
wrong_paths

avatar joeforjoomla joeforjoomla - open - 15 Oct 2016
avatar joeforjoomla joeforjoomla - change - 15 Oct 2016
The description was changed
avatar joeforjoomla joeforjoomla - edited - 15 Oct 2016
avatar joeforjoomla joeforjoomla - change - 15 Oct 2016
The description was changed
avatar joeforjoomla joeforjoomla - edited - 15 Oct 2016
avatar joeforjoomla joeforjoomla - change - 15 Oct 2016
The description was changed
avatar joeforjoomla joeforjoomla - edited - 15 Oct 2016
avatar joeforjoomla joeforjoomla - change - 15 Oct 2016
The description was changed
avatar joeforjoomla joeforjoomla - edited - 15 Oct 2016
avatar joeforjoomla
joeforjoomla - comment - 15 Oct 2016

Assign to 3.6.3 branch and fix it before the release

avatar brianteeman
brianteeman - comment - 15 Oct 2016

sent from my phone so cant search but
99% certain there is at least one if not two Pr for this already

avatar mbabker
mbabker - comment - 15 Oct 2016

SIR YES SIR!

On Saturday, October 15, 2016, JoeforJoomla Boy notifications@github.com
wrote:

Assign to 3.6.3 branch and fix it before the release


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#12423 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAWfoZOUOJhtSui9nRq5hBvY7yvKAKe7ks5q0TCngaJpZM4KXy2N
.

avatar brianteeman brianteeman - close - 15 Oct 2016
avatar brianteeman brianteeman - change - 15 Oct 2016
The description was changed
Status New Closed
Closed_Date 0000-00-00 00:00:00 2016-10-15 20:02:18
Closed_By brianteeman
avatar brianteeman brianteeman - close - 15 Oct 2016
avatar ggppdk
ggppdk - comment - 16 Oct 2016

@joeforjoomla
Searching for "folderlist" in PRs (from my desktop):
https://github.com/joomla/joomla-cms/pulls?utf8=%E2%9C%93&q=folderlist

You can find a relative recent PR #11288,
which has several responces and title:

folderlist form field returning full paths on Windows (issue #11102)

avatar brianteeman brianteeman - change - 19 Oct 2016
Labels Added: ?

Add a Comment

Login with GitHub to post a comment