Bug #57492

Upgrading 6.2: Do not run FileTableSplittingUpdate if we migrate from TYPO3 before 6.0

Added by Alexander Opitz over 1 year ago. Updated over 1 year ago.

Status:Resolved Start date:2014-04-01
Priority:Should have Due date:
Assigned To:Alexander Opitz % Done:

100%

Category:Install Tool Spent time: -
Target version:next-patchlevel
TYPO3 Version:6.2 Is Regression:No
PHP Version: Sprint Focus:
Complexity:

Description

If no sys_file table exists we try to migrate the fields pid and tstamp from sys_file to sys_file_metadata which leads to image width/height = 0


Related issues

related to Core - Bug #46020: Image size is 0 when not scaled Closed 2013-03-04

Associated revisions

Revision a5d16741
Added by Alexander Opitz over 1 year ago

[BUGFIX] Limit FileTableSplittingUpdate to TYPO3 6.0/6.1

If the database field does not exist it may be that we have an
installation before 6.0 or after 6.1 so we do not need to migrate
the data to sys_file_metadata. This prevents empty metadata
information which may lead to other issues.
(width/height=0 for images)

Resolves: #57492
Related: #46020
Releases: 6.2
Change-Id: Id5826ea480d4139f2e232587fc2b119928b28bc8
Reviewed-on: https://review.typo3.org/29024
Reviewed-by: Markus Klein
Tested-by: Markus Klein
Reviewed-by: Wouter Wolters
Reviewed-by: Nicole Cordes
Reviewed-by: Anja Leichsenring
Reviewed-by: Jan Runte
Reviewed-by: Marc Bastian Heinrichs
Tested-by: Marc Bastian Heinrichs

History

#1 Updated by Alexander Opitz over 1 year ago

This seams only one issue with width/height=0 while migration.

#2 Updated by Alexander Opitz over 1 year ago

Another issue with this Upgrade Wizard:
If you have more files then metadata information, this Wizard will show up again, if you run this wizard again, all files wich have no metainformation will now get empty fields.

#3 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/29024

#4 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/29024

#5 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/29024

#6 Updated by Gerrit Code Review over 1 year ago

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

#7 Updated by Gerrit Code Review over 1 year ago

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

#8 Updated by Alexander Opitz over 1 year ago

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

Also available in: Atom PDF