Project

General

Profile

Actions

Bug #63364

closed

ReadOnly filemounts check does not respect evaluatePermissions setting

Added by Alexander Stehlik over 9 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
File Abstraction Layer (FAL)
Target version:
-
Start date:
2014-11-27
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
6.2
PHP Version:
Tags:
Complexity:
Is Regression:
Yes
Sprint Focus:

Description

In the \TYPO3\CMS\Core\Resource\ResourceStorage it is possible to set a the class property $evaluatePermissions.

If this is FALSE no check for filemount boundaries should be performed.

The new feature for read only filemounts has introduced a regression here because the check for this new flag does not respect this setting.

Another problem is that this check in the checkFileActionPermission() method can also fail when the user has multiple filemounts, one which allows writing and one which is read only because it breaks on the first read only filemount.


Related issues 1 (0 open1 closed)

Related to TYPO3 Core - Bug #49391: options.folderTree.altElementBrowserMountPoints (readonly filemounts) not working with FALClosed2013-06-24

Actions
Actions #1

Updated by Gerrit Code Review over 9 years 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 http://review.typo3.org/34694

Actions #2

Updated by Gerrit Code Review over 9 years ago

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

Actions #3

Updated by Helmut Hummel over 9 years ago

  • Description updated (diff)
  • Status changed from Under Review to Needs Feedback

can you eleborate in non technical speak what usecase you have, what your expectations are and what the current behavior is?

Actions #4

Updated by Helmut Hummel over 9 years ago

  • Status changed from Needs Feedback to Accepted

Ok, I think I got it ;)

Actions #5

Updated by Gerrit Code Review over 9 years ago

  • Status changed from Accepted to Under Review

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

Actions #6

Updated by Gerrit Code Review over 9 years ago

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

Actions #7

Updated by Gerrit Code Review over 9 years ago

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

Actions #8

Updated by Gerrit Code Review over 9 years ago

Patch set 1 for branch TYPO3_6-2 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/34964

Actions #9

Updated by Helmut Hummel over 9 years ago

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

Updated by Gerrit Code Review over 9 years ago

  • Status changed from Resolved to Under Review

Patch set 2 for branch TYPO3_6-2 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/34964

Actions #11

Updated by Helmut Hummel over 9 years ago

  • Status changed from Under Review to Resolved
Actions #12

Updated by Benni Mack over 5 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF