Bug #56400

Finding a storage by storage type fails

Added by Marc Bastian Heinrichs over 1 year ago. Updated over 1 year ago.

Status:Resolved Start date:2014-02-27
Priority:Must have Due date:
Assigned To:Marc Bastian Heinrichs % Done:

100%

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

Description

Results in failing to build a correct ResourceFactory->localDriverStorageCache
results in failing to find a bestMatchingStorageByLocalPath results failing in getting the right storage for a fileIdentifier results in dupes in sys_file with storage 0.


Related issues

related to Core - Task #55186: Prevent Storages to be queried over and over again Resolved 2014-01-20

Associated revisions

Revision 3d4de84e
Added by Marc Bastian Heinrichs over 1 year ago

[BUGFIX] Fix StorageRepository::findByStorageType

Since optimizing the retrieval of Storages the
findByStorageType() doesn't find any (also right ones)
storage, because of a wrong comparison.
This fixes also the localDriverStorageCache in
ResourceFactory, finding a bestMatchingStorageByLocalPath
in ResourceFactory, getting the right storage for a local
path as fileIdentifier and creation of duplicate entries
in sys_file with storage 0.

Resolves: #56400
Releases: 6.2
Change-Id: I75ac357dff498f1a209d4c42896bdeddab3641ad
Reviewed-on: https://review.typo3.org/27915
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
Reviewed-by: Helmut Hummel
Tested-by: Helmut Hummel

History

#1 Updated by Gerrit Code Review over 1 year ago

  • Status changed from Accepted 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/27915

#2 Updated by Marc Bastian Heinrichs over 1 year ago

  • Is Regression changed from No to Yes

#3 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/27915

#4 Updated by Marc Bastian Heinrichs over 1 year ago

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

Also available in: Atom PDF