Closed (fixed)
Project:
Project
Version:
5.x-1.x-dev
Component:
Projects
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
24 Aug 2007 at 00:55 UTC
Updated:
7 Sep 2007 at 07:34 UTC
title says it all. needs testing on mysql.
| Comment | File | Size | Author |
|---|---|---|---|
| pg_overview.patch | 564 bytes | hunmonk |
Comments
Comment #1
aclight commentedPatch applies cleanly and appears to work fine with MySQL.
The only thing is that it's not clear to me how this patch makes the query pgsql compliant. My best guess is that n.sticky is one of the fields in the ORDER BY statement in
project_project_sort_methods()and pgsql gives an error because that isn't one of the fields that is actually returned by the query because it's not in the SELECT statement?If that's the case then I would say go with it. If not, it might be helpful to write a quick note here as to why this change makes the query pgsql compliant, in case someone looks back at this issue in the future for guidance or for an explanation.
Comment #2
hunmonk commentedcorrect, n.sticky must also be in the select b/c of ORDER BY.
fixed in HEAD, 4.7.x-2.x, 4.7.x-1.x
Comment #3
(not verified) commented