This project is archived and is in readonly mode.

#620 ✓invalid
Marcelo Barbudas

include field ignored

Reported by Marcelo Barbudas | July 15th, 2008 @ 01:46 AM | in 2.x

I have a query like this:

item = Item.find(:all, order => [ "field(users.login, 'automated_user'), DESC" ], :include => :user)

This fails in Rails 2.1 because the include is ignored.

The only way to get it working was to add an

:conditions => [ ' IS NOT NULL' ] or in general anything involving a check of the user table.

Comments and changes to this ticket

  • Stephanie Collett

    Stephanie Collett July 23rd, 2008 @ 12:31 AM

    This just got me today. Thanks for the work around.

  • Frederick Cheung

    Frederick Cheung August 9th, 2008 @ 04:07 PM

    The order option should be a string (you get away with passing a one element array because of Array's default to_s implementation. If you use a string then rails should notice that you are referencing the users table in your order option

  • Marcelo Barbudas

    Marcelo Barbudas August 12th, 2008 @ 10:45 AM

    Indeed making it a string fixes the problem.

    After using conditions => [] for some time it feels naturally to use arrays for options in find methods.

  • Tarmo Tänav

    Tarmo Tänav September 18th, 2008 @ 05:26 AM

    • State changed from “new” to “invalid”
    • Tag changed from activerecord to activerecord

    The docs do say that :order should be an sql fragment. This working with anything else is just a coincidence, though indeed that could be a surprise, but there isn't much that could be done here.

Create your profile

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

<h2 style="font-size: 14px">Tickets have moved to Github</h2>

The new ticket tracker is available at <a href=""></a>

Referenced by