Broken Trac Report

Jonathan "Duke" Leto jonathan at leto.net
Thu Apr 28 17:17:44 UTC 2011


Howdy,

Thanks! You guys rock.

Duke

On Thu, Apr 28, 2011 at 7:13 AM, Bruce Gray <bruce.gray at acm.org> wrote:
>> On Thu, Apr 28, 2011 at 1:33 AM, Jonathan "Duke" Leto <jonathan at leto.net>
>> wrote:
>>>
>>> Can somebody fix this report?
>>>
>>> http://trac.parrot.org/parrot/report/6
>
>
> On Apr 28, 2011, at 8:10 AM, Andrew Whitworth wrote:
>>
>> I removed the offending clause from the report query, and it appears
>> to work now. Whether it does the correct thing still, I can't say. I
>> don't know what it is supposed to do.
>
> I see what the clause was intended to do, and have replaced it, then fixed
> it.
>
> Fixed by changing this:
> ORDER BY
>    (milestone IS NULL),
>    milestone DESC,
>    (status = 'closed'),
>    (
>        CASE status
>        WHEN 'closed'
>        THEN modified
>        ELSE (-1)*p.value
>        END
>    ) DESC
> to this:
> ORDER BY
>    (milestone IS NULL),
>    milestone DESC,
>    (status = 'closed'), -- Within each milestone, closed tickets appear at
> bottom
>    (
>        -- Closed tickets sort youngest to oldest
>        -- Open   tickets sort most urgent to least urgent
>        CASE status
>        WHEN 'closed'
>        THEN changetime
>        ELSE (-1) * CAST( p.value AS INTEGER )
>        END
>    ) DESC
>
> 1. "modified" is an alias for "changetime", and aliases are not allowed in
> the ORDER BY clause.
> 2. "p.value" is a text field, even though it is always an int value when
> used for Priority, and so needs a CAST().
>
> --
> Hope this helps,
> Bruce Gray (Util of PerlMonks)
>
>



-- 
Jonathan "Duke" Leto <jonathan at leto.net>
209.691.DUKE // http://leto.net
NOTE: Personal email is only checked twice a day at 10am/2pm PST,
please call/text for time-sensitive matters.


More information about the parrot-dev mailing list