Ruby on Rails | Screencasts | Download | Documentation | Weblog | Community | Source

Ticket #10035 (new defect)

Opened 7 months ago

Last modified 1 month ago

[PATCH] Form.Element.Serializers.select(element, index) "index" parameter name is a misnomer, should be "value"

Reported by: prust Assigned to: sam
Priority: low Milestone: 2.x
Component: Prototype Version: edge
Severity: minor Keywords:
Cc:

Description

This isn't a huge deal, but the mis-named parameter makes it look like you can pass an index to this function and to the setValue method which calls this function. Passing an index will not work, this function takes a value (or an array of values), just like the other serializers.

Attachments

change_index_param_to_value.diff (1.1 kB) - added by jdalton on 04/02/08 16:05:54.

Change History

04/02/08 16:05:25 changed by jdalton

  • summary changed from Form.Element.Serializers.select(element, index) "index" parameter name is a misnomer, should be "value" to [PATCH] Form.Element.Serializers.select(element, index) "index" parameter name is a misnomer, should be "value".

Good point. :)

04/02/08 16:05:54 changed by jdalton

  • attachment change_index_param_to_value.diff added.