? Pending

User tests: Successful: Unsuccessful:

avatar richard67
richard67
15 Sep 2018

Pull Request for new issue.

Summary of Changes

Correct length of column "subject" of table "#__privacy_consents" in schema update for mysql

Testing Instructions

Code review: Check length of column "subject" of table "#__privacy_consents" in files "installation/joomla.sql" and in schema updates "3.9.0-2018-05-24.sql", all for all database types. Check there is no other schema update later beside "3.9.0-2018-05-24.sql" which handles that database column.

Or real life test: Using MySQL database, install a new 3.9-beta2-dev (nightly build from today) on one site, install a 3.8.12 on another site and then update that site to nightly build from today. Then export both databases in sql and compare the sql files regarding table structures.

Expected result

Code review: Column "subject" of table "#__privacy_consents" has the same length in all sql files.

Real life test: Column "subject" of table "#__privacy_consents" has the same length on both sites.

Actual result

Code review: Column "subject" of table "#__privacy_consents" has in all sql files a lenght of 255 except in schema update "mysql/3.9.0-2018-05-24.sql", where it has a lenght of 25.

See here:
https://github.com/joomla/joomla-cms/blob/staging/installation/sql/mysql/joomla.sql#L1704

https://github.com/joomla/joomla-cms/blob/staging/installation/sql/postgresql/joomla.sql#L1688

https://github.com/joomla/joomla-cms/blob/staging/installation/sql/sqlazure/joomla.sql#L2408

https://github.com/joomla/joomla-cms/blob/staging/administrator/components/com_admin/sql/updates/mysql/3.9.0-2018-05-24.sql#L8

https://github.com/joomla/joomla-cms/blob/staging/administrator/components/com_admin/sql/updates/postgresql/3.9.0-2018-05-24.sql#L12

https://github.com/joomla/joomla-cms/blob/staging/administrator/components/com_admin/sql/updates/sqlazure/3.9.0-2018-05-24.sql#L16

Real life test: On the new installation, column "subject" of table "#__privacy_consents" has a length of 255, on the updated site it has a length of 25.

Documentation Changes Required

None.

avatar richard67 richard67 - open - 15 Sep 2018
avatar richard67 richard67 - change - 15 Sep 2018
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 15 Sep 2018
Category SQL Administration com_admin
avatar alikon
alikon - comment - 15 Sep 2018

I have tested this item successfully on 0fa5ed1

good find


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

avatar alikon alikon - test_item - 15 Sep 2018 - Tested successfully
avatar richard67 richard67 - change - 15 Sep 2018
The description was changed
avatar richard67 richard67 - edited - 15 Sep 2018
avatar richard67 richard67 - change - 15 Sep 2018
The description was changed
avatar richard67 richard67 - edited - 15 Sep 2018
avatar richard67
richard67 - comment - 15 Sep 2018

@alikon I found that doing the real life test, comparing an updated site with a new install. I was surprised to find only this one difference in structure ;-) Maybe someone should do such a test for postgresql, too. Maybe I will find some time for that later.

avatar SharkyKZ
SharkyKZ - comment - 15 Sep 2018

I have tested this item successfully on 0fa5ed1

Code review.


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

avatar SharkyKZ SharkyKZ - test_item - 15 Sep 2018 - Tested successfully
avatar richard67
richard67 - comment - 15 Sep 2018

@alikon @SharkyKZ I've just found another difference between new installation and update, see PR #22200 .
But this was the last one then.
Can you test it, too?

avatar franz-wohlkoenig franz-wohlkoenig - change - 15 Sep 2018
Status Pending Ready to Commit
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 15 Sep 2018

Ready to Commit after two successful tests.

avatar mbabker mbabker - close - 15 Sep 2018
avatar mbabker mbabker - merge - 15 Sep 2018
avatar mbabker mbabker - change - 15 Sep 2018
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2018-09-15 16:27:31
Closed_By mbabker
Labels Added: ?

Add a Comment

Login with GitHub to post a comment