? ? Pending

User tests: Successful: Unsuccessful:

avatar richard67
richard67
21 Mar 2020

Pull Request for Issue # .

Summary of Changes

This Pull Request changes the joomla.sql files for new installations for both kinds of databases.

MySQL (and MariaDB):

  • Code style: Removed redundant spaces and added missing names quoting and a missing line break.
  • Fixed alphabetical order of extensions inserted for webservices plugins.

PostgreSQL:

Fix unneeded differences in data between PostgreSQL and MySQL and comments style.

The differences between the databases fixed by this PR are:

  • Vote plugin is by default disabled on MySQL, on PostgreSQL it is enabled.
  • On MySQL, the default workflow stages and transitions are inserted with the right asset ID's, on PostgreSQL they are inserted with asset ID zero.
  • On MySQL, the new record in the extensions table for plg_system_webauthn is inserted with order number zero like it is for all the other extensions, on PostgreSQL it is order number 8.
  • The order in which new records for the diverse webservices plugins are inserted into the extensions table differs between the 2 database types.

In the comments for the diverse sections in posgresql/joomla.sql, table names are in most cases quoted like on MySQL, only for a few recently added ones it was done differently. This PR unifies that so a comparison between the two joomla.sql files will not show difference in quotes in these comments.

Finally, fixed alphabetical order of extensions inserted for webservices plugins in the same way as for MySQL.

Testing Instructions

The test consists of 2 parts:

  1. Code review.
  2. Test if new installation still works.

Instructions for part 1 - code review:

  1. Check if the differences shown in the "Files changed" view on GitHub are equal to what is described in the summary of changes above and check that they don't contain any typos or errors.
  2. Using a good comparison tool, e.g. BeyondCompare, Araxis Merge, TotalCommander, ..., compare the two joomla.sql files for MySQL and PostgreSQL. Check for all insert statements that they insert the same data and don't contain any differences which are not expected due to the different database type (like e.g. CURRENT_TIMESTAMP being a function or a session variable, that's expected).

Instructions for part 2 - test new installation:

  1. Apply the patch of this PR to a current, clean 4.0-dev branch or recent nightly. If necessary remove configuration.php and clear database to prepare a new installation.
  2. Make a new installation on whatever kind of database you have available. If you have more than 1, test as many as you feel comfortable with. Verify that the installation works as well as it works without this PR.
  3. Login to backend and check it works as well as it works without this PR.
  4. When reporting back your test result, please note which kind of database you have tested.

Expected result

After a new installation, the initial data inserted with joomla.sql is the same for MySQL and PostgreSQL databases.

Actual result

After a new installation, the initial data inserted with joomla.sql is different between MySQL and PostgreSQL databases. The differences are as described in the summary of changes.

Documentation Changes Required

None.

avatar richard67 richard67 - open - 21 Mar 2020
avatar richard67 richard67 - change - 21 Mar 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 21 Mar 2020
Category SQL Installation Postgresql
avatar richard67 richard67 - change - 21 Mar 2020
The description was changed
avatar richard67 richard67 - edited - 21 Mar 2020
avatar richard67 richard67 - change - 21 Mar 2020
Labels Added: ?
avatar richard67 richard67 - change - 21 Mar 2020
The description was changed
avatar richard67 richard67 - edited - 21 Mar 2020
avatar richard67 richard67 - change - 21 Mar 2020
The description was changed
avatar richard67 richard67 - edited - 21 Mar 2020
avatar richard67 richard67 - change - 21 Mar 2020
Title
[4.0] [WiP] Fix initial core data for PostgreSQL and code style for MySQL in joomla.sql files
[4.0] Fix initial core data for PostgreSQL and code style for MySQL in joomla.sql files
avatar richard67 richard67 - edited - 21 Mar 2020
avatar richard67 richard67 - change - 21 Mar 2020
The description was changed
avatar richard67 richard67 - edited - 21 Mar 2020
avatar richard67 richard67 - change - 21 Mar 2020
The description was changed
avatar richard67 richard67 - edited - 21 Mar 2020
avatar richard67 richard67 - edited - 21 Mar 2020
avatar richard67 richard67 - change - 21 Mar 2020
The description was changed
avatar alikon
alikon - comment - 22 Mar 2020

I have tested this item successfully on 7171556

tested o postgresql
review for mysql/mariadb


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

avatar alikon alikon - test_item - 22 Mar 2020 - Tested successfully
avatar richard67 richard67 - change - 22 Mar 2020
The description was changed
avatar richard67 richard67 - edited - 22 Mar 2020
avatar richard67 richard67 - change - 22 Mar 2020
Title
[4.0] Fix initial core data for PostgreSQL and code style for MySQL in joomla.sql files
[4.0] Fix initial core data and code style in joomla.sql files
avatar richard67 richard67 - edited - 22 Mar 2020
avatar richard67
richard67 - comment - 22 Mar 2020

@Quy I've decided to follow your suggestion and fix alphabetical order. You are right, it is like that elsewhere, too.

@alikon Sorry for that, but could you test again? Thanks in advance.

avatar richard67
richard67 - comment - 22 Mar 2020

@Quy Could you check again if all ok now? I hope I haven't missed one.

avatar Quy
Quy - comment - 22 Mar 2020

I have tested this item successfully on 8ae9896

Tested with MySQL.


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

avatar Quy Quy - test_item - 22 Mar 2020 - Tested successfully
avatar richard67
richard67 - comment - 22 Mar 2020

@alikon Could you review? Last changes since you've tested were for alphabetical ordering of the records for the webservices plugins in the extensions table, so I think review should be sufficient.

The changes are in these 2 commits:

avatar chmst
chmst - comment - 23 Mar 2020

I have tested this item successfully on 8ae9896

Tested with mySQL


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

avatar chmst chmst - test_item - 23 Mar 2020 - Tested successfully
avatar richard67
richard67 - comment - 23 Mar 2020

@alikon Could you test again or review the PostgreSQL part? Thanks in advance.

avatar alikon
alikon - comment - 23 Mar 2020

I have tested this item successfully on 8ae9896

code review


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

avatar alikon alikon - test_item - 23 Mar 2020 - Tested successfully
avatar alikon alikon - change - 23 Mar 2020
Status Pending Ready to Commit
avatar alikon
alikon - comment - 23 Mar 2020

RTC


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

avatar richard67
richard67 - comment - 23 Mar 2020

Thanks to all testers and reviewers.

avatar wilsonge wilsonge - change - 24 Mar 2020
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2020-03-24 12:27:45
Closed_By wilsonge
Labels Added: ?
avatar wilsonge wilsonge - close - 24 Mar 2020
avatar wilsonge wilsonge - merge - 24 Mar 2020
avatar wilsonge
wilsonge - comment - 24 Mar 2020

Thanks for spending the time on this @richard67 !

Add a Comment

Login with GitHub to post a comment