? Pending

User tests: Successful: Unsuccessful:

avatar richard67
richard67
10 Aug 2020

Pull Request for Issue # .

Summary of Changes

Since the merge of PR #30192 , the installation on PostgreSQL databases is broken.

You can see that by the system-tests-postgres constantly failing on the 4.0-dev branch.

Testing Instructions

Code review and check that system-tests-postgres succeeds in Drone, or make a new installation using a PostgreSQL database.

Actual result BEFORE applying this Pull Request

Installation fails with SQL error when using a PostgreSQL database.

Expected result AFTER applying this Pull Request

Installation succeeds when using a PostgreSQL database.

Documentation Changes Required

None.

avatar richard67 richard67 - open - 10 Aug 2020
avatar richard67 richard67 - change - 10 Aug 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 10 Aug 2020
Category Postgresql SQL Installation
avatar richard67 richard67 - change - 10 Aug 2020
Title
[4.0] Fix wrong names quoting in PostgreSQL installation SQL script
[4.0] Fix broken installation with SQL error on PostgreSQL databases
avatar richard67 richard67 - edited - 10 Aug 2020
avatar alikon alikon - test_item - 10 Aug 2020 - Tested successfully
avatar alikon
alikon - comment - 10 Aug 2020

I have tested this item successfully on ffd1881

tested on postgresql 11.7


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

avatar alikon
alikon - comment - 10 Aug 2020

after drone finish this can be merged @richard67

avatar richard67
richard67 - comment - 10 Aug 2020

Yes we can count drone as 2nd tester, but I'd prefer if someone else merges it because I'm the author.

avatar HLeithner HLeithner - change - 10 Aug 2020
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2020-08-10 06:44:27
Closed_By HLeithner
Labels Added: ?
avatar HLeithner HLeithner - close - 10 Aug 2020
avatar HLeithner HLeithner - merge - 10 Aug 2020
avatar HLeithner
HLeithner - comment - 10 Aug 2020

Thanks

avatar richard67
richard67 - comment - 10 Aug 2020

Welcome.

Add a Comment

Login with GitHub to post a comment