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()

Added by Steffen Ritter about 6 years ago. Updated over 1 year ago.

Status:
Closed
Priority:
Must have
Category:
File Abstraction Layer (FAL)
Target version:
-
Start date:
2013-12-06
Due date:
2013-12-31
% Done:

100%

Estimated time:
1.00 h
TYPO3 Version:
6.2
PHP Version:
Tags:
Complexity:
easy
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

Revision a8c54798 (diff)
Added by Steffen Ritter about 6 years ago

[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

Revision 9d4ff3cf (diff)
Added by Benni Mack about 6 years ago

[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 about 6 years ago

  • Parent task set to #54261

#2 Updated by Gerrit Code Review about 6 years ago

  • Status changed from Accepted to Under Review
  • translation missing: en.field_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 about 6 years ago

  • % Done changed from 0 to 70

#4 Updated by Gerrit Code Review about 6 years 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 about 6 years 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 about 6 years 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 about 6 years 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 about 6 years ago

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

#9 Updated by Gerrit Code Review about 6 years 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 Benni Mack about 6 years ago

  • Status changed from Under Review to Resolved

#11 Updated by Benni Mack over 1 year ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF