Project

General

Profile

Actions

Bug #77323

closed

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

Added by Kasper Ligaard over 7 years ago. Updated over 5 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,
        ...
);

Actions #2

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

Actions #3

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

Actions #4

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

Actions #5

Updated by Joerg Boesche over 7 years ago

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

Updated by Benni Mack over 5 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF