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 6 years ago. Updated over 2 years ago.

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

100%

Estimated time:
2.00 h
TYPO3 Version:
6.2
PHP Version:
Tags:
Complexity:
Sprint Focus:

Associated revisions

Revision 3756116a (diff)
Added by Steffen Ritter about 6 years 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 about 6 years ago

  • Assignee set to Steffen Ritter

#2 Updated by Steffen Ritter about 6 years ago

  • Tracker changed from Bug to Task
  • Due date set to 2013-12-31
  • Estimated time set to 2.00 h
  • translation missing: en.field_remaining_hours set to 2.0

#3 Updated by Steffen Ritter about 6 years ago

  • Parent task set to #54268

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

  • % Done changed from 0 to 70

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

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

#8 Updated by Riccardo De Contardi over 2 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF