Feeling Lucky
?
[#8255] - PostgreSQL - wrong group by com_contacts
- Closed
- 5 Nov 2015
- Medium
- Build: staging
- Easy Test
- # 8255
- Diff
- alikon:patch-49
- Pending Human Test Results: 1 Successful 0 Failed. Details
User tests: Successful: Unsuccessful:
Steps to reproduce the issue
on a (empty) multilanguage site
administration->contacts->contacts
Actual result
sql error
After patch
work as expected
Comments
wrong group by clause for postgresql,mssql
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Labels |
Added:
?
|
||
alikon
- comment
- 3 Nov 2015
unfortunately for strict sql we cannot
cause they are listed in the select clause
..."uc"."name" AS "editor","ag"."title" AS "access_level"...
zero-24
- comment
- 3 Nov 2015
ok. Also nothing critical as the id (i have not see that bevor) is still on the top ;) I can try testing this later.
| Category | ⇒ | Postgresql |
| Easy | No | ⇒ | Yes |
alikon
- comment
- 3 Nov 2015
i've followed the order defined in the SELECT clause
SELECT
"a"."id","a"."name","a"."alias","a"."checked_out","a"."checked_out_time","a"."catid","a"."user_id","a"."published","a"."access","a"."created","a"."created_by","a"."ordering","a"."featured","a"."language","a"."publish_up","a"."publish_down",
"ul"."name" AS "linked_user",
"ul"."email",
"l"."title" AS "language_title",
"uc"."name" AS "editor",
"ag"."title" AS "access_level","c"."title" AS "category_title"
waader
- comment
- 5 Nov 2015
I have tested this item 
successfully on 17da637
Thanks alikon!
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8255.
| Status | Pending | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2015-11-05 14:32:22 |
| Closed_By | ⇒ | wilsonge |
wilsonge
- comment
- 5 Nov 2015
Merged on code review (having done a bunch of these PR's for 3.4 myself :P)
| Milestone |
Added: |
||
@alikon is it required to have this ordering or can we move this:
to the top of the group by? (as it was bevor) and than the others above this lines?