Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

change sections field to blob #862

Closed
wants to merge 9 commits into from
Closed

change sections field to blob #862

wants to merge 9 commits into from

Conversation

fritzmg
Copy link
Contributor

@fritzmg fritzmg commented Jun 4, 2017

This fixes contao/core#8722. Due to the new section wizard the limit of 1022 characters (on MyISAM) for the sections field will be reached much sooner. Thus, depending on the length of section names, you would be limited to a small number of sections. The limit is probably even smaller on InnoDB. If you go past that limit, invalid serialized data will be saved into the field.

This PR changes the field type to blob, which is used for other fields with similar wizards as well.

Copy link
Member

@ausi ausi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When we merge this, contao/installation-bundle#55 should get merged too.

@leofeyer leofeyer added the bug label Jun 4, 2017
@leofeyer leofeyer added this to the 4.4.0 milestone Jun 4, 2017
@leofeyer leofeyer changed the base branch from master to release/4.4.0 June 5, 2017 13:05
@leofeyer leofeyer self-assigned this Jun 5, 2017
@leofeyer leofeyer changed the base branch from release/4.4.0 to master June 5, 2017 13:07
@leofeyer leofeyer changed the base branch from master to release/4.4.0 June 5, 2017 13:08
leofeyer pushed a commit that referenced this pull request Jun 5, 2017
@leofeyer
Copy link
Member

leofeyer commented Jun 5, 2017

Merged in 4cd899e.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants