#2626 default_scope for select option builds invalid SQL. - 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.

#2626 ✓stale

default_scope for select option builds invalid SQL.

Reported by bill | May 8th, 2009 @ 09:44 PM | in 3.x

This query failed in my app due to the fact that you can't declare multiple columns in the "SELECT count" fragment:

SELECT count(id,title) AS count_id_title FROM resources INNER JOIN resources_tags ON resources.id = resources_tags.resource_id WHERE (((resources.approved = 1) AND (resources_tags.tag_id = 109 )) AND (resources_tags.tag_id = 109 )) )

The query was invoked by "m = Model.approved" in the controller and then calling "m.size" in the view.

This is declared in the model: default_scope :select => 'id,title' named_scope :approved, :conditions => { :approved => true }

Rails 2.3.2 MySQL 5.0 Win XP

Comments and changes to this ticket

You flagged this item as spam.
Emili Parreño May 13th, 2009 @ 10:41 AM
- Tag set to “activerecord, count, mysql, patch”
Is an error in construct_count_options_from_args(*args) method when build the sql sentence

Fix for this bug in the pacth file
Pratik May 21st, 2009 @ 09:34 AM
Hi Emili,

The patch needs failing tests.

Thanks!
You flagged this item as spam.
Emili Parreño May 21st, 2009 @ 11:30 AM
I attach the patch with tests
- fix_bug_2626_2.diff 2.3 KB
You flagged this item as spam.
Emili Parreño June 26th, 2009 @ 09:13 AM
What about this patch? I sent the patch with tests.
Jeremy Kemper May 4th, 2010 @ 06:48 PM
- Milestone changed from “2.x” to “3.x”
[bulk edit]
Rohit Arondekar October 9th, 2010 @ 04:18 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.