? Pending

User tests: Successful: Unsuccessful:

avatar richard67
richard67
28 Aug 2018

Remove copy and paste artifacts with MySQL syntax in create table statement for table "#__csp" from PostgreSQL installation and schema update script.

Can be merged on review.

@alikon please review.

avatar richard67 richard67 - open - 28 Aug 2018
avatar richard67 richard67 - change - 28 Aug 2018
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 28 Aug 2018
Category Postgresql SQL Administration com_admin Installation
avatar richard67 richard67 - edited - 28 Aug 2018
avatar richard67 richard67 - change - 28 Aug 2018
Title
[4.0] Fix SQL for creating table "#__csp" in installation and schema update
[4.0] Fix SQL for creating table "#__csp" in installation and schema update for PostgreSQL
avatar alikon
alikon - comment - 28 Aug 2018

I have tested this item successfully on a767a4b

code review


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

avatar alikon alikon - test_item - 28 Aug 2018 - Tested successfully
avatar richard67
richard67 - comment - 28 Aug 2018

@alikon Thanks.

avatar richard67 richard67 - change - 28 Aug 2018
Labels Added: ?
avatar richard67
richard67 - comment - 28 Aug 2018

@alikon Sorry, I updated my branch with 4.0-dev because GitHub was showing that my branch was not up to date. Now your test result was reset. Can you set it again? I will not use that button on my other 2 PRs now so you don't have to do that 3 times.


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

avatar richard67
richard67 - comment - 28 Aug 2018

@laoneo Can this one be merged?


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

avatar laoneo
laoneo - comment - 28 Aug 2018

Thanks!

avatar laoneo laoneo - change - 28 Aug 2018
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2018-08-28 12:23:57
Closed_By laoneo
avatar laoneo laoneo - close - 28 Aug 2018
avatar laoneo laoneo - merge - 28 Aug 2018

Add a Comment

Login with GitHub to post a comment