#262 ✓resolved
Tony Pitale

dm-aggregates #count bug

Reported by Tony Pitale | May 8th, 2008 @ 03:08 PM

#count doesn't handle the new triplet form of a condition e.g. [operator, property, value].

Comments and changes to this ticket

  • Sam Smoot

    Sam Smoot May 8th, 2008 @ 05:26 PM

    triplet is actually the original form. Discussed on IRC. Raw-SQL conditions like this:

    Zoo.all(:conditions => ['name = ?', 'bob'])
    

    Translate to this:

    ['name = ?', ['bob']]
    

    But should change to this:

    [:raw, 'name = ?', ['bob']]
    

    To be handled by adapters consistently.

    We did discuss adding an actual Condition class, but that'll slow things down a tad, and I'm not sure it's really any simpler so let's hold off on that for now.

  • Sam Smoot

    Sam Smoot May 8th, 2008 @ 11:59 PM

    • Milestone cleared.
  • Dan Kubb (dkubb)

    Dan Kubb (dkubb) May 13th, 2008 @ 11:01 PM

    • State changed from “new” to “resolved”
    • Assigned user set to “Dan Kubb (dkubb)”

Please Sign in or create a free account to add a new ticket.

With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.

New-ticket Create new ticket

Create your profile

Help contribute to this project by taking a few moments to create your personal profile. Create your profile »

People watching this ticket

Pages