#2775 ActiveRecord::Base#find ... let :order param accept a Hash or an Array - Ruby on Rails

Type	To find
responsible:me	tickets assigned to you
tagged:"@high"	tickets tagged @high
milestone:next	tickets in the upcoming milestone
state:invalid	tickets with the state invalid
created:"last week"	tickets created last week
sort:number, importance, updated	tickets sorted by #, importance or updated
Combine keywords for powerful searching.
Use advanced searching »

This project is archived and is in readonly mode.

#2775 ✓stale

ActiveRecord::Base#find ... let :order param accept a Hash or an Array

Reported by Paul Campbell | June 7th, 2009 @ 10:48 AM | in 3.x

I've been using the :order param on finders quite a lot recently, and what with being able to specify a :conditions => {} hash, I've really felt that :order should be as well endowed.

I took a look at a request here: http://www.ruby-forum.com/topic/122335 which was shot down because hashes aren't sorted ... but it was never raised that :order could take an array.

This patch allows you to pass a hash to :order for sorting by one column only in a particular direction, eg:

Developer.find(:all, :order => {:name => :desc})

In addition, if you want to specify multiple sort columns, you can do so by passing in an array of hashes, strings, or symbols:

Developer.find(:all, :order => [{:salary => :asc}, :name, 'id'])

Paul

order-allow-hash-array.diff 6.1 KB

Comments and changes to this ticket

You flagged this item as spam.
Paul Campbell June 7th, 2009 @ 10:49 AM
- Tag set to “:order, activerecord”
You flagged this item as spam.
Yaroslav Markin August 14th, 2009 @ 10:14 PM
- Tag changed from “:order, activerecord” to “:order, activerecord, patch”
Jeremy Kemper May 4th, 2010 @ 06:48 PM
- Milestone changed from “2.x” to “3.x”
[bulk edit]
Rohit Arondekar October 9th, 2010 @ 04:05 AM
- State changed from “new” to “stale”
- Importance changed from “” to “”
Marking ticket as stale. If this is still an issue please leave a comment with suggested changes, creating a patch with tests, rebasing an existing patch or just confirming the issue on a latest release or master/branches.