Task #53690

Epic #55070: Workpackages

Epic #54260: WP: FAL Missing Issues / Features / API

Story #54268: As an TYPO3-Contributor I want to have a clean CodeBase in order maintain the product

Remove calls to FileRepository when retrieving FileObjects

Added by Steffen Ritter over 1 year ago. Updated over 1 year ago.

Status:Resolved Start date:2013-11-16
Priority:Should have Due date:2013-12-31
Assigned To:Steffen Ritter % Done:

100%

Category:File Abstraction Layer (FAL) Spent time: -
Target version:6.2.0 Estimated time:2.00 hours
TYPO3 Version:6.2 Complexity:
PHP Version: Sprint Focus:

Associated revisions

Revision 3756116a
Added by Steffen Ritter over 1 year ago

[TASK] Fix remaining calls to FileRepository for retrieving Files

In previous efforts the calls to the FileRepository, when retrieving
File objects have been relocated to the ResourceFactory, since the
coexistence causes problems when it comes to metadata and multiple
instances of the same entity.

As findByUid is defined in the AbstractRepository it was overlooked
until now. This patch overrides the base method, deprecated it and
calls the ResourceFactory, too.
In addition all calls in core are replaced with the direct call.

Resolves: #53690
Releases: 6.2
Change-Id: I89a665211bd7d400f339e27bd2b155958fe5daf1
Reviewed-on: https://review.typo3.org/26850
Reviewed-by: Ingo Schmitt
Tested-by: Ingo Schmitt
Reviewed-by: Frans Saris
Tested-by: Frans Saris

History

#1 Updated by Steffen Ritter over 1 year ago

  • Assigned To set to Steffen Ritter

#2 Updated by Steffen Ritter over 1 year ago

  • Tracker changed from Bug to Task
  • Due date set to 2013-12-31
  • Estimated time set to 2.00
  • Remaining (hours) set to 2.0

#3 Updated by Steffen Ritter over 1 year ago

  • Parent task set to #54268

#4 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/26850

#5 Updated by Steffen Ritter over 1 year ago

  • % Done changed from 0 to 70

#6 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/26850

#7 Updated by Steffen Ritter over 1 year ago

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

Also available in: Atom PDF