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
| 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: | 4.00 hours | |
| TYPO3 Version: | 6.2 | Complexity: | medium | |
| PHP Version: | Sprint Focus: |
Associated revisions
[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
[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 over 1 year 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 over 1 year ago
- Tracker changed from Bug to Task
- Status changed from New to Accepted
- Assigned To set to Steffen Ritter
- Target version set to 6.2.0
- Complexity set to medium
#3 Updated by Steffen Ritter over 1 year ago
- Due date set to 2013-12-31
- Estimated time set to 4.00
- Remaining (hours) set to 4.0
#4 Updated by Steffen Ritter over 1 year ago
- Parent task set to #54266
#5 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/27193
#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/27193
#7 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/27193
#8 Updated by Steffen Ritter over 1 year ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 71eb69ed230a7a024c5e15c8334525ca229dd784.
#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/27223
#10 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/27223
#11 Updated by Frans Saris over 1 year ago
- Status changed from Under Review to Resolved
Applied in changeset ebc957e99b3f26d35e4e8d472bb0044b6a8ab58e.