? ? Pending

User tests: Successful: Unsuccessful:

avatar csthomas
csthomas
17 Feb 2017

Pull Request for issue #13289
To help #7680 on mssql

Summary of Changes

  1. Add missing defaults for columns [all db]:
  • xreference
  • sortname1, sortname2, sortname3
  • metakey, metadesc - column type TEXT can not have default value for mysql
  1. Add missing default for sqlsrv only:
  • checked_out
  • created_by_alias
  • published
  1. Remove defaults because that columns can not have default '' for :
  • name
  • alias
  • language # for postgresql only
  1. Remove DEFAULT NULL or alone NULL from sql files because this is redundant.

  2. Clean up for postgresql:
    Replace DEFAULT 0 NOT NULL to NOT NULL DEFAULT 0

Info:
https://github.com/joomla/joomla-cms/blob/staging/installation/sql/postgresql/joomla.sql#L264
https://github.com/joomla/joomla-cms/blob/staging/installation/sql/mysql/joomla.sql#L287

Testing Instructions

Install joomla with applied changes.
Installation should pass on all databases.
Test create contact.

avatar csthomas csthomas - open - 17 Feb 2017
avatar csthomas csthomas - change - 17 Feb 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 17 Feb 2017
Category MS SQL SQL Installation
avatar csthomas csthomas - change - 17 Feb 2017
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - change - 17 Feb 2017
Category MS SQL SQL Installation SQL Installation Postgresql MS SQL
avatar csthomas csthomas - change - 17 Feb 2017
Title
Add missing defaults in contact_details table for sqlsrv
Sync columns in contact_details table, add missing defaults, clean up
avatar csthomas csthomas - edited - 17 Feb 2017
avatar csthomas csthomas - change - 17 Feb 2017
The description was changed
avatar csthomas csthomas - edited - 17 Feb 2017
avatar csthomas csthomas - change - 17 Feb 2017
The description was changed
avatar csthomas csthomas - edited - 17 Feb 2017
avatar csthomas csthomas - change - 17 Feb 2017
The description was changed
avatar csthomas csthomas - edited - 17 Feb 2017
avatar csthomas csthomas - change - 17 Feb 2017
The description was changed
avatar csthomas csthomas - edited - 17 Feb 2017
avatar csthomas csthomas - change - 17 Feb 2017
The description was changed
avatar csthomas csthomas - edited - 17 Feb 2017
avatar csthomas csthomas - edited - 17 Feb 2017
avatar csthomas csthomas - change - 17 Feb 2017
The description was changed
avatar csthomas csthomas - edited - 17 Feb 2017
avatar joomla-cms-bot joomla-cms-bot - change - 17 Feb 2017
Category MS SQL SQL Installation Postgresql SQL Administration com_admin Postgresql MS SQL Installation
avatar csthomas csthomas - change - 17 Feb 2017
The description was changed
avatar csthomas csthomas - edited - 17 Feb 2017
avatar csthomas csthomas - change - 17 Feb 2017
The description was changed
avatar csthomas csthomas - edited - 17 Feb 2017
avatar csthomas csthomas - edited - 17 Feb 2017
avatar joomdonation
joomdonation - comment - 17 Feb 2017

I have tested this item successfully on 308e1d3

Installation works well. Contacts can be created both on com_contact component and User - Contact Creator plugin

Please note that I only tested it on MySQL.


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

avatar joomdonation joomdonation - test_item - 17 Feb 2017 - Tested successfully
avatar waader waader - test_item - 19 Feb 2017 - Tested successfully
avatar waader
waader - comment - 19 Feb 2017

I have tested this item successfully on 308e1d3

I tested postgresql and mssql.


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

avatar zero-24 zero-24 - change - 19 Feb 2017
The description was changed
avatar zero-24 zero-24 - edited - 19 Feb 2017
avatar zero-24 zero-24 - change - 19 Feb 2017
Milestone Added:
Status Pending Ready to Commit
Labels Added: ?
avatar zero-24 zero-24 - change - 19 Feb 2017
The description was changed
avatar zero-24 zero-24 - edited - 19 Feb 2017
avatar rdeutz rdeutz - change - 19 Feb 2017
The description was changed
Labels
avatar rdeutz
rdeutz - comment - 19 Feb 2017

@csthomas could you have a look if I fixed the conflicts right

avatar csthomas
csthomas - comment - 19 Feb 2017

It is OK

avatar rdeutz rdeutz - change - 19 Feb 2017
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2017-02-19 19:52:09
Closed_By rdeutz
avatar rdeutz rdeutz - close - 19 Feb 2017
avatar rdeutz rdeutz - merge - 19 Feb 2017

Add a Comment

Login with GitHub to post a comment