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

feature(db): new installations use utf8mb4 and longtext columns #10864

Merged
merged 1 commit into from Apr 19, 2017

Conversation

mrclay
Copy link
Member

@mrclay mrclay commented Apr 5, 2017

This allows existing sites to upgrade even with older MySQL versions, but new installs get a better schema and require MySQL 5.5.3+.

Fixes #10863

Pros: Full unicode support, practically no more column truncation issues
Cons: Installations will vary a bit more (though site owners do these kinds of things anyway)

@mrclay mrclay changed the title chore(db): new installations use utf8mb4 and longtext columns feature(db): new installations use utf8mb4 and longtext columns Apr 7, 2017
@@ -122,7 +122,7 @@
'install:error:settings' => 'Unable to create the settings file',
'install:error:databasesettings' => 'Unable to connect to the database with these settings.',
'install:error:database_prefix' => 'Invalid characters in database prefix',
'install:error:oldmysql' => 'MySQL must be version 5.0 or above. Your server is using %s.',
'install:error:oldmysql' => 'MySQL must be version 5.5.3 or above. Your server is using %s.',
Copy link
Member Author

Choose a reason for hiding this comment

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

I should use a new key.

@mrclay
Copy link
Member Author

mrclay commented Apr 17, 2017

Installation went smooth. I had a leftover cookie from another install (same domain) that prevented login, but worked fine after clearing them.

emoji

Clearly we need this.

This allows existing sites to upgrade even with older MySQL versions, but
new installs get a better schema and require MySQL 5.5.3+.

Fixes Elgg#10863
@mrclay mrclay merged commit c2d22de into Elgg:master Apr 19, 2017
@mrclay mrclay deleted the 10863_db_up branch April 19, 2017 21:37
@hypeJunction
Copy link
Contributor

Oh my. I wish this never saw the day of light. Emoji hell 💩

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

Successfully merging this pull request may close these issues.

None yet

3 participants