Bug #48643: home directories of users and groups are not mounted in file list - TYPO3 Core - TYPO3 Forge

Actions

Copy link

Bug #48643

closed

home directories of users and groups are not mounted in file list

Added by Steffen Weber almost 11 years ago. Updated over 5 years ago.

Status:

Closed

Priority:

Must have

Assignee:

Category:

File Abstraction Layer (FAL)

Target version:

Start date:

2013-05-29

Due date:

% Done:

Estimated time:

TYPO3 Version:

6.1

PHP Version:

5.3

Tags:

Complexity:

medium

Is Regression:

Sprint Focus:

Description

There are several errors in the initializeFileStorages() function of file

sysext/core/Classes/Authentication/BackendUserAuthentication.php

line 1466 should read
$userHomeFilterIdentifier = $userHomeFilter . $this->user['uid'] . $GLOBALS['TYPO3_CONF_VARS']['BE']['userUploadDir'];

in lines 1457-8 $userHomeStorageUid is (falsely?) set to $GLOBALS['TYPO3_CONF_VARS']['BE']['userHomePath'] and then intval($userHomeStorageUid) is called, which sets it to 0, so that the rest of the mouning code is not executed

analogue error in lines 1475-6 for the groups

after fixing this, the folder still are not mounted, so there seem to be some deeper errors

Actions

Copy link

Updated by Andreas Wolf almost 11 years ago

Category set to File Abstraction Layer (FAL)
Complexity set to medium

I would guess that this feature is so little used (I have only used it once in ten years of TYPO3 experience) that it simply broke during introduction of the File Abstraction Layer and nobody noticed it until now.

Actions

Copy link

Updated by TO_Webmaster no-lastname-given almost 11 years ago

This is a very annoying bug. I did some research. One problem is that the upgrade wizard does not change the paths in the configuration according to the new FAL. This can be fixed by hand. But then, it still does not work. addFileMount does not have a boolean return value, it throws errors. I did a quick and dirty fix for this, but you should create an appropriate fix asap.

Actions

Copy link

Updated by Gerrit Code Review almost 11 years ago

Status changed from New to Under Review

Patch set 1 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/22000

Actions

Copy link

Updated by Tim Lochmüller almost 11 years ago

Hey guys, I add a patch to solve the problem. I don't review the behavior of the install tool. Please check the patch and check the configuration of the to TYPO3_CONF_VARS options. They have to start with a storageUid. E.g.:

$GLOBALS['TYPO3_CONF_VARS']['BE']['userHomePath'] = '1:/users/';
$GLOBALS['TYPO3_CONF_VARS']['BE']['groupHomePath'] = '1:/groups/';

Actions

Copy link