Task #59110
Epic #58282: Workspaces Workpackage #2
Story #58284: Bug fixes & Behaviour
Field size of t3ver_label too low
| Status: | Resolved | Start date: | 2014-05-26 | |
|---|---|---|---|---|
| Priority: | Should have | Due date: | ||
| Assigned To: | Oliver Hader | % Done: | 100% |
|
| Category: | Workspaces | Spent time: | - | |
| Target version: | - | |||
| TYPO3 Version: | 6.2 | Complexity: | easy | |
| PHP Version: | 5.3 | 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
[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 about 1 year 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 about 1 year 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 about 1 year ago
- Assigned To set to Oliver Hader
#4 Updated by Oliver Hader about 1 year ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 7f21712f3a54035bd6d1580668cb94df47fa0af5.