[TYPO3-core] Review of #47211 - ProcessedFile is persisted in sys_file as well

[Fabien Udriot]

Hi,

Currently FAL has a nasty bug affecting all 6.* versions which is quiet urgent to fix. To make it
short the problem is that processed files are currently persisted in "sys_file" instead of being in
"sys_file_processedfile".

I have left a snippet on the ticket, if you would like reproducing in a programming way:
http://forge.typo3.org/issues/47211#note-5. The bug can also be observed by installing the
introduction package and seeing "sys_file" being filled in with processed files.

In short, can I ask for feedback regarding https://review.typo3.org/#/c/20437/.

The patch add a way telling FAL not to index a file (= persist a file). As described in the commit
message, FAL will try by all means, indexing the file which is not always desired.

It could be some more general approaches but it would need more time, like fixing the fact that
ProcessedFile is returning a File and not a FileProcessed (around Line 194 in file
EXT:core/Classes/Resource/ProcessedFile.php). Or disabling auto-indexation and forcing the
persistence through the File Repository (like recently Flow / Extbase does). However, all that needs
to be discussed and I don't get much echo from the FAL team currently.

If nothing is done, can we considering reverting the regression (at least in 6.0 and 6.1) from commit:

https://review.typo3.org/#/c/19809/
https://review.typo3.org/#/c/19682/

What do you think?

Fb.