Task #64196

Make it possible to re index files from storage 0

Added by Frans Saris 7 months ago. Updated 17 days ago.

Status:Needs Feedback Start date:2015-01-08
Priority:Should have Due date:
Assigned To:- % Done:

0%
Category:File Abstraction Layer (FAL) Spent time: -
Target version:7.4 (Backend)
TYPO3 Version:6.2 Complexity:
PHP Version: Sprint Focus:Remote Sprint

Description

Currently all files outside of the available storages that are handled by the FAL api are indexed under pseudo storage 0. For example images in the uploads folder but also resources found in extension folders.

Problem with this is that for images the dimensions etc are stored in db and these do not get updated if a certain file gets replaced by another.

It should be possible to reindex these files just like other storages. With the side note that only files that are already are indexed should be re indexed and not that the whole typo3conf folder should get indexed.

For every updated indexed all linked processed files should be removed so a new processed file is created on next hit.

Related issues

related to Core - Bug #66340: Processed files for storage 0 do not get updated when the... Closed 2015-04-09
related to Core - Bug #50392: Specifying size in ImageViewHelper does nothing Closed 2013-07-24

History

#1 Updated by Philipp Gampe 6 months ago

We will need an own scheduler task for this.

#2 Updated by Tymoteusz Motylewski 5 months ago

  • Parent task set to #65815

#3 Updated by Xavier Perseguers 4 months ago

  • Parent task deleted (#65815)

#4 Updated by Benjamin Mack about 1 month ago

  • Target version changed from 7.1 (Cleanup) to 7.4 (Backend)

#5 Updated by Pierrick Caillon about 1 month ago

  • Status changed from New to In Progress
  • Assigned To set to Pierrick Caillon

#6 Updated by Pierrick Caillon about 1 month ago

Frans Saris wrote:

For every updated indexed all linked processed files should be removed so a new processed file is created on next hit.

I am worrying about cache in this case. If we remove the file but do not clear the cache after replacing the file and running the task, a new client will not see the image because the page content is from cache with the no more existing file path. I don't think we should explicitly remove files from this task as it is not done on other storages.

#7 Updated by Gerrit Code Review about 1 month ago

  • Status changed from In Progress 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 http://review.typo3.org/40316

#8 Updated by Gerrit Code Review about 1 month ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/40316

#9 Updated by Gerrit Code Review about 1 month ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/40316

#10 Updated by Gerrit Code Review about 1 month ago

Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/40316

#11 Updated by Helmut Hummel about 1 month ago

  • Status changed from Under Review to Needs Feedback

Do we really, really want this?

Honestly I wished we could get rid of the fallback storage. Making indexing work again here works in a different direction

#12 Updated by Pierrick Caillon about 1 month ago

  • Assigned To deleted (Pierrick Caillon)

#13 Updated by Susanne Moog 17 days ago

  • Sprint Focus changed from On Location Sprint to Remote Sprint

Also available in: Atom PDF