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

Correcting order of queries in update sql + wrong alias char in joomla.sql #8376

Merged
merged 1 commit into from Nov 11, 2015

Conversation

infograf768
Copy link
Member

See #8374

When updating, in 3.5.0-2015-07-01.sql, the alias collation to utf8mb4_bin has to be modified AFTER the table changes to utf8mb4_general_ci

To test: install a clean 3.4.5, then download https://github.com/joomla/joomla-cms/releases/download/3.5.0-beta/Joomla_3.5.0-beta-Beta-Update_Package.zip , unzip and replace ROOT/administrator/components/com_admin/sql/updates/mysql/3.5.0-2015-07-01.sql by the modified file. Zip again.
Then install the update via Extensions=>Update
There will be one issue to fix (related to user_profile) in Database Fix (see below)

Then in PHPMyadmin, export your database.
The alias definitions will now correctly be:

`alias` varchar(191) CHARACTER SET utf8mb4 COLLATE utf8mb4_bin NOT NULL DEFAULT '',

The PR also corrects some alias varchar when installing a clean full 3.5.0. They should be 191 and not 255.

This PR does NOT solve the other issue reported in #8374
i.e.

Table 'mytable_user_profiles' does not have column 'profile_value' with type 'TEXT'. 

(From file 3.3.4-2014-08-03.sql.)

@Kubik-Rubik
Copy link
Member

Tested successfully, thank you @infograf768! Merged on code review.

Kubik-Rubik added a commit that referenced this pull request Nov 11, 2015
Correcting order of queries in update sql + wrong alias char in joomla.sql
@Kubik-Rubik Kubik-Rubik merged commit 8cc49c0 into joomla:staging Nov 11, 2015
@Kubik-Rubik Kubik-Rubik added this to the Joomla! 3.5.0 milestone Nov 11, 2015
@infograf768 infograf768 deleted the sql350 branch November 11, 2015 17:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants