? Pending

User tests: Successful: Unsuccessful:

avatar richard67
richard67
22 Jun 2019

Pull Request for issue described below.

Summary of Changes

This is the same as PR #24704 but for the installation.

Testing Instructions

  1. Make a new, clean installation on 4.0-dev or with latest nightly 4.0.* build on a MySQL database using the PDO driver.
  2. Goto system panel in backend and check if some database error.
  3. Check database view and check which database error it is.
  4. Check with PhpMyAdmin table charsets and collations.

Expected result

No database error, all tables are created with utf8mb4 charset and collations.

Actual result

One database error about tables not having been converted to utf8mb4, all tables are created with utf8 charset and collations.

What is funny: When connecting to the backend after installation, connection shows utf8mb4 support.

But using the "fix" button to run the utf8mb4 conversion does not help (which makes sense because 4.0 should have all converted already), so you have to live with that database error and wrong collation in db until the end of your life.

When using the MySQLi driver, everything is ok, i.e. this issue is only for the MySQL PDO driver.

But this might be just lucky circumstance, or to be more precise, inconsistency between the drivers regarding initial value for utf8mb4 support, and so this PR also adds the right default for the MySQLi driver.

At the end it is just the same for the installation as PR #24704 was for the installed J4.

With staging this issue does not exist, it exists only in 4.0-dev.

Documentation Changes Required

None.

avatar richard67 richard67 - open - 22 Jun 2019
avatar richard67 richard67 - change - 22 Jun 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 22 Jun 2019
Category Installation
avatar richard67 richard67 - change - 22 Jun 2019
Labels Added: ?
avatar richard67
richard67 - comment - 22 Jun 2019

Ping @wilsonge ?

avatar richard67
richard67 - comment - 22 Jun 2019

@mbabker If you have a bit time, could you check if this PR is an acceptable way to solve the issue we recently discussed? It works, but if you think it's an ugly hack, I will close it of course.

avatar richard67 richard67 - change - 22 Jun 2019
The description was changed
avatar richard67 richard67 - edited - 22 Jun 2019
avatar richard67 richard67 - change - 22 Jun 2019
The description was changed
avatar richard67 richard67 - edited - 22 Jun 2019
avatar mbabker
mbabker - comment - 22 Jun 2019

That's how it should be. The key part is in making sure the right options always get passed into the drivers, which this PR fixes.

avatar wilsonge wilsonge - change - 22 Jun 2019
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2019-06-22 18:12:26
Closed_By wilsonge
avatar wilsonge wilsonge - close - 22 Jun 2019
avatar wilsonge wilsonge - merge - 22 Jun 2019
avatar wilsonge
wilsonge - comment - 22 Jun 2019

Thanks!

avatar richard67
richard67 - comment - 22 Jun 2019

Thanks too.

Add a Comment

Login with GitHub to post a comment