User tests: Successful: Unsuccessful:
Pull Request for Issue #30823 (comment).
This pull request (PR) fixes a wrong line break in the params value of an insert statement which came into the installation SQL script "base.sql" for PostgreSQL with the merge of PR #30823 .
See the review comment referred above.
As you can see, system tests were passing with that erroneous line break, so it did not cause an SQL error on installation, but it is at least a code style error so to say.
Check that system tests on PostgreSQL are passing in Drone.
Code review: Compare the part of code changed by this PR here with the same SQL statement in the same script for MySQL.
https://github.com/joomla/joomla-cms/blob/4.0-dev/installation/sql/mysql/base.sql#L163
In the script for PostgreSQL there is a line break:
https://github.com/joomla/joomla-cms/blob/4.0-dev/installation/sql/postgresql/base.sql#L169-L170
In the script for PostgreSQL there is no erroneous line break, so it looks the same as the script for MySQL:
joomla-cms/installation/sql/postgresql/base.sql
Lines 169 to 170 in 2bc9e22
None.
Status | New | ⇒ | Pending |
Category | ⇒ | Postgresql SQL Installation |
I have tested this item
obvious fix - by code review
I have tested this item
Reviewed no line break present.
passes drone.
Status | Pending | ⇒ | Ready to Commit |
RTC
Labels |
Added:
?
?
|
Status | Ready to Commit | ⇒ | Fixed in Code Base |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2020-10-19 21:31:13 |
Closed_By | ⇒ | Quy | |
Labels |
Thanks.
Thanks.
@alikon As you are an SQL expert: Could you test by review?
@brianteeman Could you give it a test, too?
Thanks in advance.