Bug #77323

Enlarge 'field' in table 'sys_refindex' from 40 to 64

Added by Kasper Ligaard over 4 years ago. Updated over 2 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
-
Start date:
2016-08-01
Due date:
% Done:

100%

Estimated time:
1.00 h
TYPO3 Version:
7
PHP Version:
7.0
Tags:
Complexity:
easy
Is Regression:
No
Sprint Focus:

Description

In the standard table definitions in typo3/sysext/core/ext_tables.sql, the schema definition for table sys_refindex has the size of the field called field set to varchar(40), but MySQL allows 64 characters (at least since v. 5.5).

We have field-names in our tables, which are longer than 40 characters, and thus have met this problem. It might be that the field was simply truncated previously, but now that we switch to MySQL in strict mode, we get a warning.

We also think that 40 is rather short; 64 would be more appropriate.

The change we propose is to enlarge the field varchar from 40 to 64 in ext_tables.sql:

CREATE TABLE sys_refindex (
        ...
        field varchar(64) DEFAULT '' NOT NULL,
        ...
);

#2

Updated by Gerrit Code Review over 4 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/49369

#3

Updated by Gerrit Code Review over 4 years 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/49369

#4

Updated by Gerrit Code Review over 4 years ago

Patch set 1 for branch TYPO3_7-6 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/49371

#5

Updated by Joerg Boesche over 4 years ago

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

Updated by Benni Mack over 2 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF