[#9294] - [Bug fix] Correct issue 9292 syntax error in joomla.sql for PostrgeSQL
- Closed
- 3 Mar 2016
- Medium
- Build: staging
- # 9294
- Diff
- richard67:correct-postgresql-syntax-error-1
- Success Human Test Results: 2 Successful 0 Failed. Details
User tests: Successful: Unsuccessful:
Pull Request for Issue #9292 .
Summary of Changes
This PR corrects a silly syntax error in the installation sql script joomla.sql for PostgreSQL I've made with my PR #9269 .
When removing the unique key definition at the end of the create table, I forgot to remove also the comma (shame).
Testing Instructions
Only relevant for PostgreSQL database users.
Do a new installation either with Joomla! 3.5.0 Beta 3 or current staging, but before you start, replace the file installation/sql/postgresql/joomla.sql by the one from this PR.
Result: The syntax error described in issue #9292 does not happen anymore, and the table for the redirect links is created.
If no other errors, installation should succeed, otherwise if new erros let me know here.
If all OK, please mark your test result in the issue tracker for this PR.
Votes
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Labels |
Added:
?
|
||
| Title |
|
||||||
| Title |
|
||||||
not expert, but maybe "header" INTEGER should be integer (lower case)?
@andrepereiradasilva Thanks, this fixed it.
| Title |
|
||||||
@Dygear Could you mark your test result as success then at the issue tracker? Just go to https://issues.joomla.org/tracker/joomla-cms/9294, click the "Test this" button at the to left area and then select the test result. Thanks.
I have tested this item 
successfully on f57d30a
Now install under PostgreSQL 9.5.1 with PHP 7.0.3 without issue.
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/9294.
Thanks for testing. One more tester with a PostgreSQL database available, please. Maybe @alikon ?
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/9294.
I'll do this evening , now busy ;)
On 3 Mar 2016 2:22 pm, "Richard Fath" notifications@github.com wrote:
Thanks for testing. One more tester with a PostgreSQL database available,
please. Maybe @alikon https://github.com/alikon ?
This comment was created with the J!Tracker Application
https://github.com/joomla/jissues at issues.joomla.org/joomla-cms/9294
https://issues.joomla.org/tracker/joomla-cms/9294.—
Reply to this email directly or view it on GitHub
#9294 (comment).
Thanks in advance.
I have tested this item successfully on f57d30a
tested under
PHP 5.5.30
PostgreSQL 9.4.5
p.s.
very happy to see attention about PostgreSQL
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/9294.
Thanks for testing to all testers.
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/9294.
| Status | Pending | ⇒ | Ready to Commit |
wilsonge
- | Status | Ready to Commit | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2016-03-03 20:12:21 |
| Closed_By | ⇒ | wilsonge | |
| Labels |
Added:
?
|
||
| Labels |
Removed:
?
|
||
Changed title to being more descriptive
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/9294.