User tests: Successful: Unsuccessful:
Pull Request for new issue.
Correct length of column "subject" of table "#__privacy_consents" in schema update for mysql
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.
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.
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
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.
None.
Status | New | ⇒ | Pending |
Category | ⇒ | SQL Administration com_admin |
I have tested this item
Code review.
Status | Pending | ⇒ | Ready to Commit |
Ready to Commit after two successful tests.
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:
?
|
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.