Task #54226
Epic #55070: Workpackages
Epic #54260: WP: FAL Missing Issues / Features / API
Story #54261: As an Extension-Developer I want to use FAL assets in my custom development
Extend File/Folder API for getContainingFolder()
| Status: | Resolved | Start date: | 2013-12-06 | |
|---|---|---|---|---|
| Priority: | Must have | Due date: | 2013-12-31 | |
| Assigned To: | Steffen Ritter | % Done: | 100% |
|
| Category: | File Abstraction Layer (FAL) | Spent time: | - | |
| Target version: | - | Estimated time: | 1.00 hour | |
| TYPO3 Version: | 6.2 | Complexity: | easy | |
| PHP Version: | Sprint Focus: |
Description
No official way to retrieve the parent container, but this is useful quite often...
therefore --> add it and remove the hacks in the file module
Associated revisions
[TASK] Add API to retrieve the parent folder of a resource
Currently if some code needs the parent directory or folder
object of some files or folders it either needs to query the
storage object for the identifier of the parent folder and then
create the folder object manually or needs to fiddle around with
the identifier manually, assuming that all identifiers are a
representation of a hierarchical path as it is for the local
driver.
The latter not only will fail for all storages having non-path
identifiers it is more overhead to create the folder objects
manually anyhow.
This patch adds "getParentFolder" to the ResourceInterface and
implements the method in AbstractFile and Folder as well as it
replaces the manual resolval in the file list module.
Releases: 6.2
Resolves: #54226
Change-Id: I651d62340186dd9ac57277b498f8f98ee160dfd2
Reviewed-on: https://review.typo3.org/26724
Reviewed-by: Alexander Opitz
Reviewed-by: Marcin Sągol
Reviewed-by: Sebastian Fischer
Tested-by: Sebastian Fischer
Reviewed-by: Ernesto Baschny
Tested-by: Ernesto Baschny
[BUGFIX] Followup to #54226
The fix for issue #54226 introduced a bug
with FileReferences as it did not implement the
change of the interface.
Resolves: #54226
Releases: 6.2
Change-Id: I9f67f3f23b96569cdfadba1505e4881103a86ccb
Reviewed-on: https://review.typo3.org/27170
Reviewed-by: Benjamin Mack
Tested-by: Benjamin Mack
History
#1 Updated by Steffen Ritter over 1 year ago
- Parent task set to #54261
#2 Updated by Gerrit Code Review over 1 year ago
- Status changed from Accepted to Under Review
- Remaining (hours) set to 1.0
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/26724
#3 Updated by Steffen Ritter over 1 year ago
- % Done changed from 0 to 70
#4 Updated by Gerrit Code Review over 1 year ago
Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/26724
#5 Updated by Gerrit Code Review over 1 year ago
Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/26724
#6 Updated by Gerrit Code Review over 1 year ago
Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/26724
#7 Updated by Gerrit Code Review over 1 year ago
Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/26724
#8 Updated by Steffen Ritter over 1 year ago
- Status changed from Under Review to Resolved
- % Done changed from 70 to 100
Applied in changeset a8c5479803eec44326b4a07bb2270aecaaf29c20.
#9 Updated by Gerrit Code Review over 1 year ago
- Status changed from Resolved 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/27170
#10 Updated by Benjamin Mack over 1 year ago
- Status changed from Under Review to Resolved
Applied in changeset 9d4ff3cf98480d2d765450fd4965da9c2dd5318d.