? ? Pending

User tests: Successful: Unsuccessful:

avatar andrepereiradasilva
andrepereiradasilva
8 Jan 2017

Pull Request for Issue #13510 and probably other issues relating ucm_content table in more strict db systems (which in joomla 4.0 mysql will be also).

Summary of Changes

Normalize the #__ucm_content table across db systems.
Only tested in mysql.
@waader please check mssql
@alikon please also check postgresql and mssql.

Testing Instructions

  1. SQL code review
  2. Test update: Apply patch, manually run the sql query for your Db system, confirm db table, confirm all fine
  3. Test install: apply patch, delete configuration.php do a normal install, confirm db table, confirm all fine

Documentation Changes Required

None.

avatar andrepereiradasilva andrepereiradasilva - open - 8 Jan 2017
avatar andrepereiradasilva andrepereiradasilva - change - 8 Jan 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 8 Jan 2017
Category SQL Administration com_admin Postgresql MS SQL Installation
avatar waader
waader - comment - 8 Jan 2017

Works for mssql and postgres apart from the update file for postgres. I deleted "TEXT" in the following lines and then it worked:

ALTER TABLE "#__ucm_content" ALTER COLUMN "core_body" SET DEFAULT '';
ALTER TABLE "#__ucm_content" ALTER COLUMN "core_params" SET DEFAULT '';
ALTER TABLE "#__ucm_content" ALTER COLUMN "core_metadata" SET DEFAULT '';
ALTER TABLE "#__ucm_content" ALTER COLUMN "core_images" SET DEFAULT '';
ALTER TABLE "#__ucm_content" ALTER COLUMN "core_urls" SET DEFAULT '';
ALTER TABLE "#__ucm_content" ALTER COLUMN "core_metakey" SET DEFAULT '';
ALTER TABLE "#__ucm_content" ALTER COLUMN "core_metadesc" SET DEFAULT '';

avatar andrepereiradasilva andrepereiradasilva - change - 8 Jan 2017
Labels Added: ?
avatar andrepereiradasilva
andrepereiradasilva - comment - 8 Jan 2017

"text" removed - copy paste error . sorry

avatar waader
waader - comment - 8 Jan 2017

I have tested this item successfully on 6f9254b

Works with mssql and postgres.

Thanks!


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

avatar waader waader - test_item - 8 Jan 2017 - Tested successfully
avatar alikon
alikon - comment - 9 Jan 2017

I have tested this item successfully on 6f9254b


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

avatar alikon alikon - test_item - 9 Jan 2017 - Tested successfully
avatar zero-24 zero-24 - change - 9 Jan 2017
Milestone Added:
Status Pending Ready to Commit
Labels Added: ?
avatar rdeutz rdeutz - change - 10 Jan 2017
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2017-01-10 13:58:41
Closed_By rdeutz
Labels
avatar rdeutz rdeutz - close - 10 Jan 2017
avatar rdeutz rdeutz - merge - 10 Jan 2017

Add a Comment

Login with GitHub to post a comment