[CLEANUP] Enhance TCA - FAL migration for tt_content 21/25621/21
authorBenjamin Mack <benni@typo3.org>
Mon, 16 Dec 2013 15:50:56 +0000 (16:50 +0100)
committerErnesto Baschny <ernst@cron-it.de>
Tue, 4 Mar 2014 17:28:26 +0000 (18:28 +0100)
commitf064c5c0159843bd210bb372801900e5491bff27
treec547a30fd945ebb0c4e0861e9163080de7ce2f6c
parentd12424f163a6890b4ce078fcbe903c6bd1b6fa70
[CLEANUP] Enhance TCA - FAL migration for tt_content

The upgrade wizard to migrate the fields like e.g.
tt_content->image and pages->media fetches all records
of each table and loops over them. This is basic, and not
very clever, especially when the max_execution_time is
less than the upgrade wizards needs to process all fields
or if the memory_limit is reached because ALL of the
records are fetched.

Thus, the patch modifies the behavior in the following ways:
* As all TCA value are switched from text to integer
 (the value itself, not the DB field yet) the SQL is done to
only fetch records that are not empty, not integer
(and not deleted). This reduces the memory footprint
massively.
* The check for a record is now done for each table and
then for each field of the table (as the SQL has been changed).
* The field is only marked as "done" if no more records were
found in the migration run.
* Also, the redudant myfile_05.jpg are not moved if the
first file with that name (myfile.jpg) was moved already.

The migration wizard can now be run multiple times
(and the counter shows how many records are left).

Furthermore the wizard hides itself now once all migrations
are done.

Resolves: #53845
Resolves: #53891
Releases: 6.2
Change-Id: I835a07158e6869d80b4426d9774754421963ef81
Reviewed-on: https://review.typo3.org/25621
Reviewed-by: Jigal van Hemert
Reviewed-by: Markus Klein
Tested-by: Markus Klein
Reviewed-by: Ernesto Baschny
Tested-by: Ernesto Baschny
typo3/sysext/install/Classes/Updates/TceformsUpdateWizard.php