? ? Pending

User tests: Successful: Unsuccessful:

avatar richard67
richard67
28 Mar 2020

Pull Request for Issue # .

Summary of Changes

This Pull Request (PR) updates the utf8mb4 conversion by tables which have been added to staging for action logs and the privacy suite but have been forgotten to add to the utf8mb4 conversion SQL scripts.

In addition it fixes a database error shown after a conversion which requires an additional use of the "Fix" button. The reason for this error is the same: It was forgotten to update one of the utf8mb4 conversion SQL scripts to changes in database structure.

The utf8mb4 conversion is needed for the migration or update of an old MySQL database (but with still supported version) which doesn't support utf8mb4 character set to a newer version which supports that. It runs on a Joomla update, if necessary, or when you go to the database checker page (Extensions -> Manage -> Database) and see a message about missing utf8 conversion and use the "Fix" button after having migrated or updated your database server.

Because there might be already installations out in the wild where the conversion has been done after action logs and the privacy suite have been added, we might have databases which are only partly converted, i.e. these new tables have still utf8 (without mb4) character sets.

Therefore this PR adds a 2nd conversion step, which runs the conversion for these tables only and not for all again in such a case.

Testing Instructions

This PR is only relevant for MySQL databases, not for MariaDB or PostgreSQL databases.

But MariaDB can be used for testing, too. In this case you have to patch the MySQLi database driver to simulate an old MySQL version which doesn't support utf8mb4 like described below in the testing instructions.

Please potential testers don't be discouraged by the long testing insctructions. They are detailled and step by step but at the end you will see testing did not require much time and was not much work.

Requirements

You need to have either a MySQL database server with a version older than 5.5.3 (which doesn't support utf8mb4 yet) and another MySQL database server with a newer version or a MariaDB, or if you have only a newer MySQL version than 5.5.3 or only have MariaDB you have to patch the utf8mb4 support detection in the database driver. The testing instructions will tell you when to do that.

For patching the MySQLi driver, edit file libraries/joomla/database/driver/mysqli.php and make function serverClaimsUtf8mb4Support() always return false by adding the code line marked with the green frame in following image:
j3-pr-28495-patch-db-driver-mysqli-for-utf8

For the MySQL (PDO) driver it would be more complicated. I recommend to test with the MySQLi driver, that's sufficient.

You should not test with MySQL 8.0.19 or later because you will run into another issue, see PR #28370 , which has nothing to do with this PR here but might be confusing when checking the database checker for errors.

Finally you should have error logging switched on in your PHP settings (log_errors = On; in php.ini or .user.ini) so that errors are logged into a log file (either webserver's log file or PHP error log).

Test 1: Reproduce issue = starting condition for Tests 2 and 3

Scenario: Reproduce the issue which results in a partly unconverted database if the database has been converted from utf8 to utf8mb4 with Joomla 3.9.16 after a migration or server update.

1.1 Prepare the installation of a 3.9.16 by unpacking the full package into an empty folder of your webserver.

1.2 If you don't have a MySQL database server with a version older than 5.5.3, you have to patch now the database driver as descibed above with the requirements. Before, make a backup of the file to be patched so you can later revert that patch.

1.3 Do the installation. If you not patched the database driver because you have an old database server, use a database on that server for the installation.

1.4 Check in your database e.g. with phpMyadmin collations of all Joomla core database tables.
Result: All tables have collations starting with "utf8_".

1.5 Now either revert the patch of the database driver made in step 1.2 or move the database to a new MySQL database server which supports utf8mb4 (5.5.3 or later).

1.6 Export your database (structure and content) e.g. with phpMyAdmin into an SQL file so you can later use it in Test 3, and backup your Joomla folder for the same purpose. Name the backups so that you can distinguish it from the backup made at the end of this step, e.g. something with "unconverted".

1.7 Login to backend and go to "Extensions -> Manage -> Database".

Result: The database check shows one error about missing UTF-8 Multibyte (utf8mb4) conversion.
j3-pr-28495-db-check-unconverted

1.8 Use the "Fix" button.

Result: The database check shows one error for column core_title of table #__ucm_content, see 1st screenshot in section "Actual result" below.

1.9 Use the "Fix" button again.

Result: The database is up to date, no errors are shown.

1.10 Check e.g. with phpMyAdmin collations of all Joomla core database tables.

Result: All tables have collations starting with "utf8mb4_", except of tables for action logs and the privacy suite, which still have collations starting with "utf8_", see 2nd screenshot in section "Actual result" below.

1.11 Export your database (structure and content) e.g. with phpMyAdmin into an SQL file so you can later use it in Test 5, and backup your Joomla folder for the same purpose. Name the backups so that you can distinguish it from the backup made at in step 1.6, e.g. something with "partly-converted".

Test 2: Update partly converted database from Test 1 to patched update package

Scenario: Fix a partly unconverted database due to this issue with updating to a future Joomla 3.9 version which contains the patch of this PR using the Joomla Update component.

2.1 Update the installation from previous Test 1 to a nightly build of today + the patch of this PR applied + debug log added for this test, either using the "Upload&Update" tab or the "Live Update" tab of the Joomla Update component.

2.2 After the update has finished, check e.g. with phpMyAdmin collations of all Joomla core database tables.

Result: All tables have collations starting with "utf8mb4_", see screenshot in section "Expected result" below.

2.3 Go to "Extensions -> Manage -> Database".

Result: The database is up to date, no errors are shown.

2.4 Check in your PHP error logfile (or the webserver log file) for logs like the following:

DEBUG 1: Running utf8mb4-conversion-01.sql
DEBUG 2: Running utf8mb4-conversion-02.sql
DEBUG 3: Running utf8mb4-conversion-03.sql

Result: Only the DEBUG 3: Running utf8mb4-conversion-03.sql can be found, i.e. the other conversion files which have already run before in Test 1 have not been run again.

Test 3: Update unconverted database to patched update package

Scenario: Convert an unconverted database with updating to a future Joomla 3.9 version which contains the patch of this PR using the Joomla Update component.

3.1 Restore to the 3.9.16 with unconverted database using the backups made in step 1.6 of Test 1.

3.2 Repeat all steps of Test 2.

Results:

  • All tables have collations starting with "utf8mb4_", see screenshot in section "Expected result" below.
  • In "Extensions -> Manage -> Database", the database is up to date, no errors are shown.
  • In PHP error log: All 3 DEBUG logs can be found, i.e. all 3 scripts have been run.

Test 4: Convert unconverted database with database fixer

Scenario: Completely convert an unconverted database on a future J 3.9 version which contains the patch of this PR using the "Fix" button in "Extensions -> Manage -> Database", like it would be after a migration or an update of the database server.

4.1 On a clean staging, apply the patch of this PR.

Then either add debug logs like shown in following two screenshots to file administrator/components/com_admin/script.php, or replace it with the patched version which you can download with following zip file and which contains the same debug logs: https://test5.richard-fath.de/script_pr-28495_debug.zip

j3-pr-28495-script-debug-log_1

j3-pr-28495-script-debug-log_2

If you don't have a Git clone, you can alternatively unpack the following patched installation package with debug log for testing into an empty folder on your webserver: https://test5.richard-fath.de/Joomla_3.9.17-dev-Development-Full_Package_2020-04-02_pr-28495_debug.zip

4.2 If you don't have a MySQL database server with a version older than 5.5.3, you have to patch now the database driver as descibed above with the requirements. Before, make a backup of the file to be patched so you can later revert that patch.

4.3 Do the installation. If you not patched the database driver because you have an old database server, use a database on that server for the installation.

4.4 Check in your database e.g. with phpMyadmin collations of all Joomla core database tables.
Result: All tables have collations starting with "utf8_".

4.5 Now either revert the patch of the database driver made in step 4.2 or move the database to a new MySQL database server which supports utf8mb4 (5.5.3 or later).

4.6 Login to backend and go to "Extensions -> Manage -> Database".

Result: The database check shows one error about missing UTF-8 Multibyte (utf8mb4) conversion.

4.7 Use the "Fix" button.

Result: The database is up to date, no errors are shown.

4.8 Check e.g. with phpMyAdmin collations of all Joomla core database tables.

Result: All tables have collations starting with "utf8mb4_", see screenshot in section "Expected result" below.

4.9 Check in your PHP error logfile (or the webserver log file) for logs like the following:

DEBUG 1: Running utf8mb4-conversion-01.sql
DEBUG 2: Running utf8mb4-conversion-02.sql
DEBUG 3: Running utf8mb4-conversion-03.sql

Result: All 3 DEBUG logs can be found, i.e. all 3 scripts have been run.

Test 5: Convert partly unconverted database with database fixer

Scenario: Fix a partly unconverted database using the "Fix" button in "Extensions -> Manage -> Database", like it would be after a migration or an update of the database server with Joomla 3.9.16 after having the patch of this PR applied in some other way instead of using Joomla Update.

5.1 Restore to the 3.9.16 with partly unconverted database using the backups made in step 1.11 of Test 1.

5.2 Apply the patch of this PR. Then add the debug logs or use the modified script.php as described in step 4.1 of Test 4.

5.3 Login to backend and go to "Extensions -> Manage -> Database".

Result: The database check shows one error about missing UTF-8 Multibyte (utf8mb4) conversion.

5.4 Use the "Fix" button.

Result: The database is up to date, no errors are shown.

5.5 Check e.g. with phpMyAdmin collations of all Joomla core database tables.

Result: All tables have collations starting with "utf8mb4_", see screenshot in section "Expected result" below.

5.6 Check in your PHP error logfile (or the webserver log file) for logs like the following:

DEBUG 1: Running utf8mb4-conversion-01.sql
DEBUG 2: Running utf8mb4-conversion-02.sql
DEBUG 3: Running utf8mb4-conversion-03.sql

Result: Only the DEBUG 3: Running utf8mb4-conversion-03.sql can be found, i.e. the other conversion files which have already run before in Test 1 have not been run again.

Expected result

Utf8mb4 conversion handles all core tables. After conversion with the database fixer or updating the CMS, all core tables show collations starting with "utf8mb4_" in phpMyAdmin.
j3-pr-28495-db-converted-yes

The database error about column core_title of table #__ucm_content is not shown anymore after conversion with the database fixer.

Actual result

When converting a utf8 database to utf8mb4 with current staging with the "Fix" button of the database checker, like it is necessary after a transfer of the database from an old server which doesn't suppoort utf8mb4 to a new server which does support it, or after an update of the old server to a new version, the checker shows after conversion a database error for column core_title of table #__ucm_content.
j3-pr-28495-db-check-fix-1

It needs to use the "Fix" button a 2nd time to finally fix the database.

After that, or when the utf8mb4 conversion has been run with a Joomla update, the tables for action logs and the privacy suite still have collations starting with "utf8_" in database.
j3-pr-28495-db-converted-partly

Documentation Changes Required

None.

avatar richard67 richard67 - open - 28 Mar 2020
avatar richard67 richard67 - change - 28 Mar 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 28 Mar 2020
Category Administration com_admin
avatar richard67
richard67 - comment - 28 Mar 2020

Ping @wilsonge : As agreed.

avatar richard67 richard67 - change - 28 Mar 2020
The description was changed
avatar richard67 richard67 - edited - 28 Mar 2020
avatar richard67 richard67 - change - 28 Mar 2020
The description was changed
avatar richard67 richard67 - edited - 28 Mar 2020
avatar richard67 richard67 - change - 28 Mar 2020
The description was changed
avatar richard67 richard67 - edited - 28 Mar 2020
avatar richard67 richard67 - change - 28 Mar 2020
Labels Added: ?
avatar richard67 richard67 - change - 28 Mar 2020
The description was changed
avatar richard67 richard67 - edited - 28 Mar 2020
avatar richard67 richard67 - change - 28 Mar 2020
The description was changed
avatar richard67 richard67 - edited - 28 Mar 2020
avatar richard67 richard67 - change - 28 Mar 2020
The description was changed
avatar richard67 richard67 - edited - 28 Mar 2020
avatar richard67 richard67 - change - 28 Mar 2020
The description was changed
avatar richard67 richard67 - edited - 28 Mar 2020
avatar richard67
richard67 - comment - 30 Mar 2020

@alikon Could you test this one? Review as described should be sufficient. And any idea who else I could ping for 2nd tester?

avatar wilsonge
wilsonge - comment - 30 Mar 2020

Does this run a second time for those users who have already upgraded the majority of their tables? I thought it only ran once as a one off and thus this would need to go into a new upgrade step?

avatar richard67
richard67 - comment - 30 Mar 2020

@wilsonge If one has already converted while these tables were missing in the sql, it will not run again for these tables.

avatar richard67
richard67 - comment - 30 Mar 2020

@wilsonge For the case that someone has had action logs and com_privacy already in J3 on a db which did not support utf8mb4 and then migrated to utf8mb4, this could indeed be a problem. Do we have to handle that?

avatar richard67
richard67 - comment - 30 Mar 2020

@wilsonge is right, it has to be done in a new step so people who have converted from utf8 to utf8mb4 while the tables were missing need to have these tables converted too.

I've set the "Updates requested" label on GitHub to show that this PR is not ready for tests and will leave a comment telling the same at the top of the tesdting instructions. Give me some time for the changes, especially for the testing instructions.

I will post back here and remove label and hint in testing instructions when ready for test.

avatar richard67 richard67 - change - 30 Mar 2020
The description was changed
avatar richard67 richard67 - edited - 30 Mar 2020
avatar richard67 richard67 - change - 30 Mar 2020
Labels Added: ?
avatar richard67 richard67 - change - 30 Mar 2020
Labels Added: ?
Removed: ?
avatar joomla-cms-bot joomla-cms-bot - change - 30 Mar 2020
Category Administration com_admin Administration com_admin Installation Libraries
avatar richard67 richard67 - change - 31 Mar 2020
Labels Added: ?
Removed: ?
avatar richard67 richard67 - change - 31 Mar 2020
Labels Added: ?
Removed: ?
avatar richard67 richard67 - change - 31 Mar 2020
Labels Added: ?
Removed: ?
avatar richard67 richard67 - change - 31 Mar 2020
The description was changed
avatar richard67 richard67 - edited - 31 Mar 2020
avatar richard67 richard67 - change - 31 Mar 2020
Labels Added: ?
Removed: ?
avatar richard67 richard67 - change - 1 Apr 2020
Labels Added: ?
Removed: ?
avatar richard67 richard67 - change - 1 Apr 2020
The description was changed
avatar richard67 richard67 - edited - 1 Apr 2020
avatar richard67 richard67 - change - 1 Apr 2020
Labels Added: ?
Removed: ?
avatar richard67 richard67 - change - 1 Apr 2020
The description was changed
avatar richard67 richard67 - edited - 1 Apr 2020
avatar richard67 richard67 - change - 2 Apr 2020
Labels Added: ?
Removed: ?
avatar richard67 richard67 - change - 2 Apr 2020
The description was changed
avatar richard67 richard67 - edited - 2 Apr 2020
avatar richard67 richard67 - change - 2 Apr 2020
The description was changed
avatar richard67 richard67 - edited - 2 Apr 2020
avatar richard67 richard67 - change - 2 Apr 2020
The description was changed
avatar richard67 richard67 - edited - 2 Apr 2020
avatar richard67 richard67 - change - 2 Apr 2020
The description was changed
avatar richard67 richard67 - edited - 2 Apr 2020
avatar richard67 richard67 - change - 2 Apr 2020
The description was changed
avatar richard67 richard67 - edited - 2 Apr 2020
avatar richard67 richard67 - change - 2 Apr 2020
The description was changed
avatar richard67 richard67 - edited - 2 Apr 2020
avatar richard67 richard67 - change - 2 Apr 2020
The description was changed
avatar richard67 richard67 - edited - 2 Apr 2020
avatar richard67 richard67 - change - 2 Apr 2020
The description was changed
avatar richard67 richard67 - edited - 2 Apr 2020
avatar richard67 richard67 - change - 2 Apr 2020
The description was changed
avatar richard67 richard67 - edited - 2 Apr 2020
avatar richard67 richard67 - change - 2 Apr 2020
The description was changed
avatar richard67 richard67 - edited - 2 Apr 2020
avatar richard67 richard67 - change - 2 Apr 2020
The description was changed
avatar richard67 richard67 - edited - 2 Apr 2020
avatar richard67 richard67 - change - 2 Apr 2020
The description was changed
avatar richard67 richard67 - edited - 2 Apr 2020
avatar richard67 richard67 - change - 2 Apr 2020
The description was changed
avatar richard67 richard67 - edited - 2 Apr 2020
avatar richard67 richard67 - change - 2 Apr 2020
The description was changed
avatar richard67 richard67 - edited - 2 Apr 2020
avatar richard67
richard67 - comment - 2 Apr 2020

@wilsonge The diff looks bad on GitHub for script.php. If you download it and check the differences to staging with a better comparison tool, it will be much easier to understand my changes. Please review again, and if you have enough time, test it too please.

avatar richard67
richard67 - comment - 2 Apr 2020

PR is ready for test. Please potential testers don't be discouraged by the long testing insctructions. They are detailled and step by step but at the end you will see testing did not require much time and was not much work.

avatar richard67 richard67 - change - 2 Apr 2020
The description was changed
avatar richard67 richard67 - edited - 2 Apr 2020
avatar richard67 richard67 - change - 2 Apr 2020
The description was changed
avatar richard67 richard67 - edited - 2 Apr 2020
avatar richard67 richard67 - change - 2 Apr 2020
The description was changed
avatar richard67 richard67 - edited - 2 Apr 2020
avatar richard67 richard67 - change - 2 Apr 2020
The description was changed
avatar richard67 richard67 - edited - 2 Apr 2020
avatar richard67 richard67 - change - 2 Apr 2020
The description was changed
avatar richard67 richard67 - edited - 2 Apr 2020
avatar richard67 richard67 - change - 2 Apr 2020
Labels Removed: ?
avatar richard67 richard67 - change - 2 Apr 2020
The description was changed
avatar richard67 richard67 - edited - 2 Apr 2020
avatar richard67 richard67 - change - 2 Apr 2020
The description was changed
avatar richard67 richard67 - edited - 2 Apr 2020
avatar alikon
alikon - comment - 3 Apr 2020

firstly i was scared about how much long the test info there was, and i've spent quite sometimes reading those, but lucky me for a mysql 8.0.19 or postgresql user they simply became a code review, i'm not the good one on those but...

avatar alikon
alikon - comment - 3 Apr 2020

I have tested this item successfully on 53cd5a7

code review


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/28495.

avatar alikon alikon - test_item - 3 Apr 2020 - Tested successfully
avatar richard67
richard67 - comment - 3 Apr 2020

@alikon Thanks for review, but I am not sure if code review really is enough or if it needs real tests.

avatar alikon
alikon - comment - 3 Apr 2020

at the moment i don't have the right differents environment needed to fully test this
task for automated tests CI/CD ..... if we really have those... ? working

avatar richard67
richard67 - comment - 3 Apr 2020

@alikon It needs only one environment, the rest is simulated by patching the database driver. Is all described in the instructions.

avatar richard67 richard67 - change - 4 Apr 2020
The description was changed
avatar richard67 richard67 - edited - 4 Apr 2020
avatar richard67 richard67 - change - 4 Apr 2020
The description was changed
avatar richard67 richard67 - edited - 4 Apr 2020
avatar richard67 richard67 - change - 4 Apr 2020
The description was changed
avatar richard67 richard67 - edited - 4 Apr 2020
avatar astridx
astridx - comment - 5 Apr 2020

Test 1: Reproduce issue = starting condition for Tests 2 and 3

1.1 I prepare the installation of a 3.9.16 by git merge version 3.9.17-dev into an empty folder of your webserver.

1.2 I patched the database driver.

Screenshot from 2020 04 05 00 58 39

1.3 I did the installation.

1.4 I checked in my database e.g. with phpMyadmin collations of all Joomla core database tables.
Result: All tables have collations starting with "utf8_".

joomla_cms.sql.txt

1.5 I revert the patch of the database driver made in step 1.2.

1.6 I exported my database and made a backup of my Joomla folder (I named it unconverted) so that I can use it in step 3.

1.7 I opened back end and went to "Extensions -> Manage -> Database".
Result: The database check shows one error about missing UTF-8 Multibyte (utf8mb4) conversion.
Extensions  Database   tes   Administration

1.8 I Used the "Fix" button.

Extensions  Database   tes   Administration(1)

1.9 I Used the "Fix" button again.

Extensions  Database   tes   Administration(2)

1.10 I Checked collations of all Joomla core database tables.

Result: All tables have collations starting with "utf8mb4_", except of tables for action logs and the privacy suite, which still have collations starting with "utf8_", see 2nd screenshot in section "Actual result" below.

joomla_cms.sql_with_error.txt

1.11 I exported my database and made a backup of my Joomla folder (I named it partly-converted) so that I can use it in step 5.

Test 2: Update partly converted database from Test 1 to patched update package

2.1.

  • I applied this patch via git fetch origin pull/28495/head:staging-fix-utf8mb4-converion and git checkout staging-fix-utf8mb4-converion
  • I made an update with your file https://test5.richard-fath.de/Joomla_3.9.17-dev-Development-Update_Package_2020-04-02_pr-28495_debug.zip.
    Joomla Update   tes   Administration
  • I was not sure what you mean with + debug log added for this test. I activated debug mode.
    Global Configuration   tes   Administration

2.2. Database is fine

[joomla_cms-2_3.sql.txt](https://github.com/joomla/joomla-cms/files/4433132/joomla_cms-
2_3.sql.txt)

2.3. "Extensions -> Manage -> Database" is fine.

Extensions  Database   tes   Administration(3)

2.4 While searching for utf8 I only found this entry in the error log of my web server /var/log/apache2/error.log

[Sun Apr 05 09:49:22.369797 2020] [php7:notice] [pid 21825] [client 127.0.0.1:44392] DEBUG 3: Running utf8mb4-conversion-03.sql, referer: http://localhost/joomla-cms3/administrator/index.php?option=com_joomlaupdate&task=update.install&1d1da3af05823a35234105b4789d902e=1
Cannot load Xdebug - it was already loaded

Test 3: Update unconverted database to patched update package

Scenario: Convert an unconverted database with updating to a future Joomla 3.9 version which contains the patch of this PR using the Joomla Update component.

3.1. I did it my way:

  • git stash
  • git checkout staging
  • git pull origin staging (optional)
  • I patched serverClaimsUtf8mb4Support again.
  • I deleted configuration.php and made a new installation
  • git fetch origin pull/28495/head:staging-fix-utf8mb4-converion and checkout
  • made update with file provided.

3.2 Repeat all steps of Test 2.

Results:

  • All tables have collations starting with "utf8mb4_", see screenshot in section "Expected result" below.
  • In "Extensions -> Manage -> Database", the database is up to date, no errors are shown.
  • In PHP error log: All 3 DEBUG logs can be found, i.e. all 3 scripts have been run.
    [Sun Apr 05 10:34:40.499841 2020] [php7:notice] [pid 28497] [client 127.0.0.1:44916] DEBUG 1: Running utf8mb4-conversion-01.sql, referer: http://localhost/joomla-cms3/administrator/index.php?option=com_joomlaupdate&task=update.install&aec17b1127f13aa29544f8659bfb6daa=1 [Sun Apr 05 10:34:40.794758 2020] [php7:notice] [pid 28497] [client 127.0.0.1:44916] DEBUG 2: Running utf8mb4-conversion-02.sql, referer: http://localhost/joomla-cms3/administrator/index.php?option=com_joomlaupdate&task=update.install&aec17b1127f13aa29544f8659bfb6daa=1 [Sun Apr 05 10:34:44.866093 2020] [php7:notice] [pid 28497] [client 127.0.0.1:44916] DEBUG 3: Running utf8mb4-conversion-03.sql, referer: http://localhost/joomla-cms3/administrator/index.php?option=com_joomlaupdate&task=update.install&aec17b1127f13aa29544f8659bfb6daa=1

Test 4: Convert unconverted database with database fixer

4.1. I did it my way:

  • git stash
  • git checkout staging
  • I patched serverClaimsUtf8mb4Support again.
  • I deleted configuration.php and made a new installation
  • git checkout staging-fix-utf8mb4-converion
  • I open "Extensions -> Manage -> Database".
    Extensions  Database   test   Administration
  • I click on fix - everything fine on first click.
    Extensions  Database   test   Administration(1)

Test 5: Convert partly unconverted database with database fixer

  • git stash
  • git checkout staging
  • I patched serverClaimsUtf8mb4Support again.
  • I deleted configuration.php and made a new installation
  • i reverted the patch serverClaimsUtf8mb4Support .
  • I went to "Extensions -> Manage -> Database" and fixed the database partially.
  • git checkout staging-fix-utf8mb4-converion
  • I open "Extensions -> Manage -> Database".
  • I click on fix - no all databases are with the correct collation.

I did not find an entry for Point 4 and 5 in error log, but I think I messed this up.

I followed the steps of the description. i haven't looked at the code. I actually wanted to do that now. But I needed more time than expected and now I have no more time.

avatar astridx
astridx - comment - 5 Apr 2020

I have tested this item successfully on 24a2697


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/28495.

avatar astridx astridx - test_item - 5 Apr 2020 - Tested successfully
avatar astridx astridx - test_item - 5 Apr 2020 - Tested successfully
avatar richard67
richard67 - comment - 5 Apr 2020

@astridx Thanks a lot for testing. Regarding the missing debug logs maybe you just have looked into the wrong file? Depending on the system configuration, it might be a separate PHP error log or the webserver's log file like e.g. here on my Linux "/home/richard/lamp/logs/error.log". Update: I just see for tests 1 to 3 you had the debug logs, so maybe in tests 4 and 5 you had forgotten to add the debug log? Or my description is wrong? Of course it needs to add those error_logs after having applied the patch of this PR, otherwise they might just be overwritten.

avatar richard67
richard67 - comment - 5 Apr 2020

@astridx P.S.: If you want to check my code, the diff on GitHub looks ugly for script.php. It looks better when comparing locally with the 4.0-dev file with a better comparison tool (BeyondCompare, TotalCommander, AraxisMerge, ..., or what your IDE provides, if you have some).

avatar chmst
chmst - comment - 5 Apr 2020

I have tested this item successfully on 24a2697

I've tested this following exactly the testing instructions. I had the same results as @astridx, so do not to repeat the screens.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/28495.

avatar chmst chmst - test_item - 5 Apr 2020 - Tested successfully
avatar richard67
richard67 - comment - 5 Apr 2020

@astridx @chmst Thanks a lot ladies for testing.

@wilsonge I would pefer you to have a final look on it and set RTC or request changes, that's why I don't set it RTC now.

avatar richard67 richard67 - change - 8 Apr 2020
Status Pending Ready to Commit
avatar richard67
richard67 - comment - 8 Apr 2020

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/28495.

avatar richard67 richard67 - change - 8 Apr 2020
Labels Added: ?
avatar richard67
richard67 - comment - 8 Apr 2020

Test results and so RTC are still valid, last merge was just an update to the latest changes in the staging branch.

avatar richard67 richard67 - change - 8 Apr 2020
The description was changed
avatar richard67 richard67 - edited - 8 Apr 2020
avatar HLeithner HLeithner - change - 13 Apr 2020
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2020-04-13 12:11:28
Closed_By HLeithner
avatar HLeithner HLeithner - close - 13 Apr 2020
avatar HLeithner HLeithner - merge - 13 Apr 2020
avatar HLeithner
HLeithner - comment - 13 Apr 2020

Thanks for bringing the rest of joomla to the age of 4 byte strings

Add a Comment

Login with GitHub to post a comment