Task #53691

Epic #55070: Workpackages

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

Story #54266: As an User I want FAL to be performant

Files are not able to be created anymore without beeing indexed - remove all this fallback logic

Added by Steffen Ritter almost 6 years ago. Updated almost 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:
4.00 h
TYPO3 Version:
6.2
PHP Version:
Tags:
Complexity:
medium
Sprint Focus:

Associated revisions

Revision 71eb69ed (diff)
Added by Steffen Ritter over 5 years ago

[TASK] Remove fallback for not-indexed files

Due to all the refactorings all file-objects are created centralized.
In addition it is not possible anymore to create a file object without
an existing index record. Therefore all the fallback-logic for lazy
loading the index record can be removed from the file object.

In addition the unit tests for these parts of the file object have been
removed, too.

Releases: 6.2
Resolves: #53691
Change-Id: I0a01c12e2caaf8554a5b606b7a85d50b44e0e97c
Reviewed-on: https://review.typo3.org/27193
Reviewed-by: Benjamin Mack
Tested-by: Benjamin Mack

Revision ebc957e9 (diff)
Added by Frans Saris over 5 years ago

[BUGFIX] Followup to #53691 Failing unit test

Remove a duplicate mocking in test fileIndexStatusIsTrueIfUidIsSet()
and setUp() triggered as side effect after patch 71eb69ed230a.

Resolves: #53691
Releases: 6.2
Change-Id: I8ac96d9157bf946bba3c9561369f3553d459f275
Reviewed-on: https://review.typo3.org/27223
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
Reviewed-by: Helmut Hummel
Tested-by: Helmut Hummel

History

#1 Updated by Steffen Ritter almost 6 years ago

  • mergeIndexRecord not needed anymore
  • fileProperties could be summarized in actual properties as they are fixed
  • getter and Setter for the hard existing properties
  • remove loadMetaData and throw exception on file creation without uid

#2 Updated by Steffen Ritter almost 6 years ago

  • Tracker changed from Bug to Task
  • Status changed from New to Accepted
  • Assignee set to Steffen Ritter
  • Target version set to 6.2.0
  • Complexity set to medium

#3 Updated by Steffen Ritter almost 6 years ago

  • Due date set to 2013-12-31
  • Estimated time set to 4.00 h
  • translation missing: en.field_remaining_hours set to 4.0

#4 Updated by Steffen Ritter almost 6 years ago

  • Parent task set to #54266

#5 Updated by Gerrit Code Review over 5 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/27193

#6 Updated by Gerrit Code Review over 5 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/27193

#7 Updated by Gerrit Code Review over 5 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/27193

#8 Updated by Steffen Ritter over 5 years ago

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

#9 Updated by Gerrit Code Review over 5 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/27223

#10 Updated by Gerrit Code Review over 5 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/27223

#11 Updated by Frans Saris over 5 years ago

  • Status changed from Under Review to Resolved

#12 Updated by Riccardo De Contardi almost 2 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF