? Success

User tests: Successful: Unsuccessful:

avatar andrepereiradasilva
andrepereiradasilva
22 Aug 2016

Summary of Changes

Normalizes all sql azure joomla install file.

  • always use SET IDENTITY_INSERT [#__xxxx] ON/OFF;
  • INSERT to INSERT INTO
  • Quote all table names and db field names
  • Always use ' to quote string values.
  • Code style

Testing Instructions

Mainly code review. (easier to check code diff without white spaces changes) https://github.com/joomla/joomla-cms/pull/11725/files?w=1
If anyone could test an install 3.6.x would be great. Don't have sql azure to test,

Documentation Changes Required

None.

avatar joomla-cms-bot joomla-cms-bot - change - 22 Aug 2016
Category Installation SQL MS SQL
avatar andrepereiradasilva andrepereiradasilva - open - 22 Aug 2016
avatar andrepereiradasilva andrepereiradasilva - change - 22 Aug 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 22 Aug 2016
Labels Added: ?
avatar andrepereiradasilva andrepereiradasilva - change - 22 Aug 2016
The description was changed
avatar andrepereiradasilva andrepereiradasilva - edited - 22 Aug 2016
avatar alikon alikon - test_item - 23 Aug 2016 - Tested successfully
avatar alikon
alikon - comment - 23 Aug 2016

I have tested this item successfully on d4b33d0

on code review still unable to test on MSSQL


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

avatar waader waader - test_item - 23 Aug 2016 - Not tested
avatar waader
waader - comment - 23 Aug 2016

I have not tested this item.

I get this error message when installing additional languages:

[Microsoft][SQL Server Native Client 11.0][SQL Server]Cannot insert the value NULL into column 'custom_data', table 'joomla35.dbo.#__extensions'; column does not allow nulls. INSERT fails. SQL=INSERT INTO #__extensions VALUES ('EnglishAustralia','language','en-AU','','1','1','0','0','{"name":"English (Australia)","type":"language","creationDate":"August 2016","author":"stevec4","copyright":"Copyright (C) 2005 - 2016 Open Source Matters. All rights reserved.","authorEmail":"stevec4@comcast.net","authorUrl":"www.joomla.org","version":"3.6.2.2","description":"en-AU administrator language","group":"","filename":"install"}','{}')
[Microsoft][SQL Server Native Client 11.0][SQL Server]Cannot insert the value NULL into column 'custom_data', table 'joomla35.dbo.#__extensions'; column does not allow nulls. INSERT fails. SQL=INSERT INTO #__extensions VALUES ('Belarusian','language','be-BY','','0','1','0','0','{"name":"Belarusian","type":"language","creationDate":"2012-11-20","author":"Dennis Hermatski","copyright":"Copyright (C) 2005 - 2012 Open Source Matters. All rights reserved","authorEmail":"sirdionisiy@gmail.com","authorUrl":"www.joomla.com","version":"3.0.2","description":"be-BY site language","group":"","filename":"install"}','{}')
[Microsoft][SQL Server Native Client 11.0][SQL Server]Cannot insert the value NULL into column 'custom_data', table 'joomla35.dbo.#__extensions'; column does not allow nulls. INSERT fails. SQL=INSERT INTO #__extensions VALUES ('Basque','language','eu-ES','','0','1','0','0','{"name":"Basque","type":"language","creationDate":"August 2016","author":"Joomla! Basque Translation Team","copyright":"Copyright (C) 2005 - 2016 Open Source Matters. All rights reserved.","authorEmail":"joomlaeuskaraz@gmail.com","authorUrl":"","version":"3.6.2.1","description":"eu-ES site language","group":"","filename":"install"}','{}')


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

avatar andrepereiradasilva
andrepereiradasilva - comment - 23 Aug 2016

This does not seems related to this PR.
This PR does not changes that, you probably already have that issue before thsi PR.

So you probably can't install any language right? even in backend "Install languages" right?

[Microsoft][SQL Server Native Client 11.0][SQL Server]
Cannot insert the value NULL into column 'custom_data', table 'joomla35.dbo.#__extensions';
column does not allow nulls.

INTO #__extensions VALUES ('EnglishAustralia','language','en-AU','','1','1','0','0','{"name":"English (Australia)","type":"language","creationDate":"August 2016","author":"stevec4","copyright":"Copyright (C) 2005 - 2016 Open Source Matters. All rights reserved.","authorEmail":"stevec4@comcast.net","authorUrl":"www.joomla.org","version":"3.6.2.2","description":"en-AU administrator language","group":"","filename":"install"}','{}')

@alikon any idea why this happens?

avatar waader waader - test_item - 23 Aug 2016 - Tested successfully
avatar waader
waader - comment - 23 Aug 2016

I have tested this item successfully on d4b33d0

You are right! This problem existed already.


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

avatar brianteeman brianteeman - change - 23 Aug 2016
Status Pending Ready to Commit
avatar brianteeman
brianteeman - comment - 23 Aug 2016

RTC


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

avatar joomla-cms-bot joomla-cms-bot - change - 23 Aug 2016
Labels Added: ?
avatar alikon
alikon - comment - 23 Aug 2016

@andrepereiradasilva MSSQL is right
the extensions table field
[custom_data] [nvarchar](max) NOT NULL,
is declared NOT NULL but without default

so we need to provide a value to that field as it is now in sql script nothing is provided for that field

avatar andrepereiradasilva
andrepereiradasilva - comment - 23 Aug 2016

ok so i added the default value
b875501

because it exists in joomla.sql in postgresql
https://github.com/joomla/joomla-cms/blob/staging/installation/sql/postgresql/joomla.sql#L460

@waader @alikon please check

avatar alikon
alikon - comment - 23 Aug 2016

You should add the usual alter table for the update too...

On 23 Aug 2016 3:34 pm, "andrepereiradasilva" notifications@github.com
wrote:

ok so i added the default value
b875501
b875501

because it exists in joomla.sql in postgresql
https://github.com/joomla/joomla-cms/blob/staging/
installation/sql/postgresql/joomla.sql#L460

@waader https://github.com/waader @alikon https://github.com/alikon
please check


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#11725 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AALFsfoD5ULt4H7cK7LnCUoSs1xO0CXdks5qivbbgaJpZM4JpxqF
.

avatar andrepereiradasilva
andrepereiradasilva - comment - 23 Aug 2016

ok so i reverted that change will wait for this to et merged and then will make a PR to put that right.

avatar wilsonge wilsonge - change - 25 Aug 2016
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2016-08-25 11:57:53
Closed_By wilsonge
avatar wilsonge wilsonge - close - 25 Aug 2016
avatar wilsonge wilsonge - merge - 25 Aug 2016
avatar joomla-cms-bot joomla-cms-bot - close - 25 Aug 2016
avatar joomla-cms-bot joomla-cms-bot - change - 25 Aug 2016
Labels Removed: ?
avatar csthomas
csthomas - comment - 31 Jan 2017

I only remind
[custom_data] [nvarchar](max) NOT NULL, still does not have DEFAULT ''

Add a Comment

Login with GitHub to post a comment