Bug #55439

Default upload folder not accessible for editors

Added by Alexander Stehlik over 1 year ago. Updated over 1 year ago.

Status:Resolved Start date:2014-01-29
Priority:Should have Due date:
Assigned To:- % Done:

100%

Category:File Abstraction Layer (FAL) Spent time: -
Target version:6.2.0
TYPO3 Version:6.2 Is Regression:No
PHP Version:5.3 Sprint Focus:
Complexity:

Description

The default folder of a storage is currently not available for normal editors by default.

A filemount to the folder is required, otherwise the user will get an error that the folder is not accessible.

Is this desired behavior. Or should the default folder automatically be added to the isWithinFileMountBoundaries check in the ResourceStorage?


Related issues

duplicated by Core - Bug #55466: Don't show upload form if user has no rights to defaultUp... Closed 2014-01-30

Associated revisions

Revision dc1a8865
Added by Alexander Stehlik over 1 year ago

[BUGFIX] Prevent permission Exception in ElementBrowser

To prevent Exceptions in the ElementBrowser resource Exceptions are
catched in two places:

The Exception is catched in the ElementBrowser itsself when the
previously selected folder is not accessible.

Additionally the Exception is cached in the getDefaultUploadFolder()
method in BackendUserAuthentication when the default folder of a
storage is not accessible.

Resolves: #55439
Releases: 6.2
Change-Id: I64848c3a88cc2cc571c0bf692101647b95666972
Reviewed-on: https://review.typo3.org/27145
Reviewed-by: Anja Leichsenring
Tested-by: Anja Leichsenring
Reviewed-by: Stefan Froemken
Tested-by: Stefan Froemken
Reviewed-by: Frans Saris
Tested-by: Frans Saris

History

#1 Updated by Alexander Stehlik over 1 year ago

This is specially a problem in the ElementBrowser. If the user did not access a folder before, the default folder is used as initial expanded folder.

If the user does not have access to it, the ElementBrowser will show an Exception "You are not allowed to access the given folder".

#2 Updated by Frans Saris over 1 year ago

Imo this should be handled. Maybe just catch the exeption so the element browser is still shown.

#3 Updated by Gerrit Code Review over 1 year ago

  • Status changed from New to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/27145

#4 Updated by Gerrit Code Review over 1 year ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/27145

#5 Updated by Alexander Stehlik over 1 year ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100

Also available in: Atom PDF