#1037 :selected option is ignored for collection_select - 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.

#1037 ✓resolved

:selected option is ignored for collection_select

Reported by YoNoSoyTu | September 12th, 2008 @ 11:07 PM

(Copied from #7043 in old bug tracker)

The :selected option in collection_select is ignored. This patch uses the value of :selected option if it is provided, otherwise normal Rails behaviour apply.

Test included.

0001--selected-option-no-longer-ignored-for-collection_.patch 2.8 KB

Comments and changes to this ticket

You flagged this item as spam.
billno September 30th, 2008 @ 03:41 AM
This patch is two years old!!! See #7043 in the old bug tracker

What does it take to get it committed? I just checked github and it does not seem to be in the sources.

This stuff up wasted a lot of my time. collection_select does NOT work as documented and this patch fixes it.
You flagged this item as spam.
Matt Buck October 23rd, 2008 @ 09:02 PM
I second that. What needs to be done to get this into core? I'm volunteering.
You flagged this item as spam.
YoNoSoyTu October 23rd, 2008 @ 09:28 PM
Read http://rails.lighthouseapp.com/p...

You only have to apply the patch to the Rails edge, test it and if it works give this ticket a comment with "plus 1" (with the symbol and without the space), and a short comment why this patch should be considered for inclusion.

By the way, I have updated the patch, removing whitespace, as source style document say.
- 0001--selected-option-no-longer-ignored-for-collection_.patch 2.8 KB
You flagged this item as spam.
YoNoSoyTu October 23rd, 2008 @ 09:29 PM
I did not know who to merge both patches into one (sorry).
- 0002-Whitespace-removed.patch 1.7 KB
You flagged this item as spam.
Gabriel Medina October 31st, 2008 @ 11:05 AM
- Title changed from “:selected option is ignored for collection_select” to “[PATCH] :selected option is ignored for collection_select”
- Tag changed from “actionpack, bug, edge, helper, patch, tested, tiny” to “actionpack, bug, collection, collection_select, edge, helper, ignored, patch, selected, tested, tiny”
Single file patch.

Gabriel Medina.
- selected_option_no_longer_ignored_for_collections_and_whitespace_removed.patch 4.5 KB
You flagged this item as spam.
Shane Vitarana November 3rd, 2008 @ 05:02 AM
+1, works as expected and tests look good.
Pratik January 19th, 2009 @ 08:15 PM
- Milestone cleared.
- Assigned user set to “Pratik”
- Title changed from “[PATCH] :selected option is ignored for collection_select” to “:selected option is ignored for collection_select”
Repository January 28th, 2009 @ 08:01 PM
- State changed from “new” to “resolved”
(from [9a8e2a059c823498e0fb318f25b926a6d015ef87]) Ensure selected option is not ignored for collection_select. [#1037 state:resolved]

Signed-off-by: Pratik Naik pratiknaik@gmail.com http://github.com/rails/rails/co...
You flagged this item as spam.
bingbing March 26th, 2011 @ 02:15 AM
louis vuitton watches