Project

General

Profile

Actions

Bug #67284

closed

Some storages cannot be marked as not public again

Added by Andreas Wolf almost 9 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
File Abstraction Layer (FAL)
Target version:
-
Start date:
2015-06-03
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
7
PHP Version:
Tags:
Complexity:
Is Regression:
No
Sprint Focus:

Description

When using the fal_webdav extension, a storage cannot be marked as not public again. The error is "The storage has been marked to be "publicly available" but is not detected as such by the driver. The setting has been reverted."
This is due to a broken check in UserStorageCapabilityService::renderIsPublic().

The check should only check if the record says the storage is public, but the driver does not report so. Instead, the method checks if the storage’s public setting and the record do not differ. This fails if the storage was marked as not public by the user (i.e. the record says "0"), as the object is still created from the old record (which said "1").

How to reproduce:

  1. Install fal_webdav current master (7351fe2)
  2. Create a storage, save it
  3. Edit it again, uncheck the "Is publicly available?" box
  4. Save, see the message once the form reloads
Actions #1

Updated by Andreas Wolf almost 9 years ago

  • Description updated (diff)
Actions #2

Updated by Gerrit Code Review almost 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/39927

Actions #3

Updated by Gerrit Code Review almost 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/39927

Actions #4

Updated by Gerrit Code Review almost 9 years ago

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/39927

Actions #5

Updated by Gerrit Code Review over 8 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/39927

Actions #6

Updated by Anonymous over 8 years ago

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

Updated by Gerrit Code Review over 8 years ago

  • Status changed from Resolved to Under Review

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/42547

Actions #8

Updated by Benni Mack over 8 years ago

  • Status changed from Under Review to Resolved
Actions #9

Updated by Benni Mack over 5 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF