Task #59110

Epic #58282: Workspaces Workpackage #2

Story #58284: Bug fixes & Behaviour

Field size of t3ver_label too low

Added by Oliver Hader over 5 years ago. Updated about 1 year ago.

Status:
Closed
Priority:
Should have
Assignee:
Category:
Workspaces
Target version:
-
Start date:
2014-05-26
Due date:
% Done:

100%

TYPO3 Version:
6.2
PHP Version:
5.3
Tags:
Complexity:
easy
Sprint Focus:

Description

During workspace move operations sometimes a warning like this can be found:

102: These fields are not properly updated in database: (t3ver_label) Probably value mismatch with fieldtype.

The reason is, that the t3ver_label field in almost every record is limited to 30 characters.
The DataHandlerHooks tries to set t3ver_label to "Placeholder version for moving record" (37 characters) which leads to the mentioned mismatch.

Associated revisions

Revision 7f21712f (diff)
Added by Oliver Hader over 5 years ago

[BUGFIX] Field size of t3ver_label too low

During workspace move operations sometimes a warning like
this can be found:

102: These fields are not properly updated in database:
(t3ver_label) Probably value mismatch with fieldtype.

The reason is, that the t3ver_label field in almost every
record is limited to 30 characters in the SQL definition.
Besides that the error code "102" of the mentioned warning
is invalid and should just be "1" to be recognized by the
accordant functional tests as well.

Resolves: #59110
Releases: 6.2
Change-Id: Ia97a32dbefd1d27e33f186c19c5a6a495dcc194a
Reviewed-on: https://review.typo3.org/30466
Reviewed-by: Wouter Wolters
Reviewed-by: Krzysztof Adamczyk
Reviewed-by: Marcin SÄ…gol
Reviewed-by: Oliver Hader
Tested-by: Oliver Hader

History

#1 Updated by Oliver Hader over 5 years ago

Since the "varchar(30)" definition has been copied and pasted for ages now, I guess it might be easier to reduce the information that TYPO3 is putting into that field and maybe to normalize the structure.

#2 Updated by Gerrit Code Review over 5 years ago

  • Status changed from New 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/30466

#3 Updated by Oliver Hader over 5 years ago

  • Assignee set to Oliver Hader

#4 Updated by Oliver Hader over 5 years ago

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

#5 Updated by Benni Mack about 1 year ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF