[Tickets #10346] Re: Syntax error in Horde_ActiveSync_State_History->listDevices()

[bugs at horde.org]

DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: http://bugs.horde.org/ticket/10346
------------------------------------------------------------------------------
  Ticket             | 10346
  Updated By         | Michael Rubinsky <mrubinsk at horde.org>
  Summary            | Syntax error in
                     | Horde_ActiveSync_State_History->listDevices()
  Queue              | Horde Framework Packages
  Version            | Git master
  Type               | Bug
-State              | Feedback
+State              | Assigned
  Priority           | 1. Low
  Milestone          |
  Patch              |
  Owners             | Michael Rubinsky
------------------------------------------------------------------------------


Michael Rubinsky <mrubinsk at horde.org> (2011-07-19 14:20) wrote:

> It seems there is a comma missing and the column reference  
> "device_id" is ambiguous, something like this works:

No, there is no comma missing and it's not ambiguous. "device_id" is a  
column alias for "d.device_id". We don't want two columns of device_id  
returned (they are the same value). It's used so the returned data is  
keyed as "device_id" and not "d.device_id". However, maybe postgres  
doesn't like the alias to be named the same as a column name.

> - SELECT d.device_id device_id, device_type, device_agent,  
> device_policykey,  ...
> + SELECT d.device_id, u.device_id, device_type, device_agent,  
> device_policykey,  ...
>