[#3292] - PostgreSQL - query fix in mod_articles_archive
- Closed
- 17 Oct 2014
- Medium
- Build: staging
- # 3292
- Diff
- vdespa:patch-5
- Success The Travis CI build passed Details
User tests: Successful: Unsuccessful:
Steps to reproduce the issue
Activate the module mod_articles_archive (Archived Articles)
Expected result
No errors
Actual result
Syntax error. PostgreSQL does not accept GROUP BY followed by DESC, like MySQL DOES
System information (as much as possible)
PostgreSQL 9.3
Additional comments
| Status | New | ⇒ | Pending |
nicksavov
- | Status | Pending | ⇒ | New |
| Labels |
Removed:
?
|
||
| Status | New | ⇒ | Pending |
| Category | ⇒ | Postgresql |
alikon
- | Labels |
Added:
?
|
||
Works for me!
Just a small remark: when I tried to apply the patch via com_patchtester I get this error message:
Database query failed (error # %s): %s SQL=INSERT INTO "d4m21_patchtester_tests" ("pull_id","data","patched_by","applied","applied_version") VALUES (3292,'[{"action":"modified","index":"18d5a53..10bd3b9 100644","old":"modules\/mod_articles_archive\/helper.php","new":"modules\/mod_articles_archive\/helper.php","body":"\n\n
On PostgreSQL? Report an issue at https://github.com/joomla-extensions/patchtester/issues and we can look to see if something in that extension isn't written correctly.
We have two good tests on this PR now - Great - setting to RTC
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/3292.
| Status | Pending | ⇒ | Ready to Commit |
| Status | Ready to Commit | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2014-10-17 11:18:42 |
Merged into staging. Thanks!
Added the information from the issue #3291 into this PR