Skip to content
This repository has been archived by the owner on Nov 3, 2023. It is now read-only.

[3.5.0 RC] file manage broken on xampp #7847

Closed
Zeromax opened this issue May 27, 2015 · 11 comments
Closed

[3.5.0 RC] file manage broken on xampp #7847

Zeromax opened this issue May 27, 2015 · 11 comments
Assignees
Labels
Milestone

Comments

@Zeromax
Copy link

Zeromax commented May 27, 2015

On Windows the file Manager is broken.

$currentFolder = str_replace(TL_ROOT . DIRECTORY_SEPARATOR, '', $folders[$f]);

$currentFolder = str_replace(TL_ROOT . DIRECTORY_SEPARATOR, '', $folders[$f]);

This will not work, cause the first slash from a file is not a backslash.
$folders[$f] = D:\www\contao-core/files/tiny_templates
TL_ROOT . DIRECTORY_SEPARATOR = D:\www\contao-core\

I think there should be a path normalization. In the File.php all path in this Clas are returned with a slash and not normalized for the correct DIRECTORY_SEPARATOR.

Warning: filesize(): stat failed for D:\www\contao-core/D:\www\contao-core/files/IMG_1275.jpg in system\modules\core\library\Contao\File.php on line 184 #0 [internal function]: __error(2, 'filesize(): sta...', 'D:\\www\\contao-c...', 184, Array)
#1 system\modules\core\library\Contao\File.php(184): filesize('D:\\www\\contao-c...')
#2 system\modules\core\drivers\DC_Folder.php(2406): Contao\File->__get('filesize')
#3 system\modules\core\drivers\DC_Folder.php(308): Contao\DC_Folder->generateTree('D:\\www\\contao-c...', 0, false, false, false)
#4 system\modules\core\classes\Backend.php(639): Contao\DC_Folder->showAll()
#5 system\modules\core\controllers\BackendMain.php(131): Contao\Backend->getBackendModule('files')
#6 contao\main.php(20): Contao\BackendMain->run()
#7 {main}


Warning: filesize(): stat failed for D:\www\contao-core/D:\www\contao-core/files/P1140966_back 4zu3.JPG in system\modules\core\library\Contao\File.php on line 184 #0 [internal function]: __error(2, 'filesize(): sta...', 'D:\\www\\contao-c...', 184, Array)
#1 system\modules\core\library\Contao\File.php(184): filesize('D:\\www\\contao-c...')
#2 system\modules\core\drivers\DC_Folder.php(2406): Contao\File->__get('filesize')
#3 system\modules\core\drivers\DC_Folder.php(308): Contao\DC_Folder->generateTree('D:\\www\\contao-c...', 0, false, false, false)
#4 system\modules\core\classes\Backend.php(639): Contao\DC_Folder->showAll()
#5 system\modules\core\controllers\BackendMain.php(131): Contao\Backend->getBackendModule('files')
#6 contao\main.php(20): Contao\BackendMain->run()
#7 {main}


Warning: filesize(): stat failed for D:\www\contao-core/D:\www\contao-core/files/tinymce.css in system\modules\core\library\Contao\File.php on line 184 #0 [internal function]: __error(2, 'filesize(): sta...', 'D:\\www\\contao-c...', 184, Array)
#1 system\modules\core\library\Contao\File.php(184): filesize('D:\\www\\contao-c...')
#2 system\modules\core\drivers\DC_Folder.php(2406): Contao\File->__get('filesize')
#3 system\modules\core\drivers\DC_Folder.php(308): Contao\DC_Folder->generateTree('D:\\www\\contao-c...', 0, false, false, false)
#4 system\modules\core\classes\Backend.php(639): Contao\DC_Folder->showAll()
#5 system\modules\core\controllers\BackendMain.php(131): Contao\Backend->getBackendModule('files')
#6 contao\main.php(20): Contao\BackendMain->run()
#7 {main}
@leofeyer leofeyer added the defect label Jun 2, 2015
@leofeyer leofeyer added this to the 3.5.0 milestone Jun 2, 2015
@leofeyer leofeyer self-assigned this Jun 2, 2015
leofeyer added a commit that referenced this issue Jun 2, 2015
@leofeyer
Copy link
Member

leofeyer commented Jun 2, 2015

Fixed in eb131db. Can you please checkout the latest release/4.0.0 branch and check if it works for you?

@Zeromax
Copy link
Author

Zeromax commented Jun 2, 2015

@leofeyer Yes on the way...

@Zeromax
Copy link
Author

Zeromax commented Jun 2, 2015

@leofeyer I will put the issue reports to the contao/standard-edition

@leofeyer
Copy link
Member

leofeyer commented Jun 3, 2015

Why? The ticket is assigned to Contao 3.5!

leofeyer added a commit that referenced this issue Jun 3, 2015
@Zeromax
Copy link
Author

Zeromax commented Jun 3, 2015

ah... You have written

Can you please checkout the latest release/4.0.0 branch and check if it works for you?

Will check 3.5 now ;)

@leofeyer
Copy link
Member

leofeyer commented Jun 3, 2015

My bad :(

@Zeromax
Copy link
Author

Zeromax commented Jun 3, 2015

No Problem ;) You have to manage lots of branches at the Moment.

So I checked out: release/3.5.0 and everything works fine.

Btw: The part overlay is not added yet?

@leofeyer
Copy link
Member

leofeyer commented Jun 3, 2015

The part overlay?

@Zeromax
Copy link
Author

Zeromax commented Jun 3, 2015

When I hover the Image in the file tree, then it swaps -> #7814

@leofeyer
Copy link
Member

leofeyer commented Jun 3, 2015

Nope, haven't got any feedback of one of the other core developers yet.

@Zeromax
Copy link
Author

Zeromax commented Jun 3, 2015

Ah ok. Thanks for the info.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants