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

Ticket #8120 (closed defect: fixed)

Opened 2 years ago

Last modified 2 years ago

[PATCH][TINY] auto_complete_field lacks :method option

Reported by: zapnap Assigned to: core
Priority: normal Milestone: 1.x
Component: ActionPack Version: edge
Severity: normal Keywords: tiny patch
Cc:

Description

In order to support a RESTful approach to auto-completion, destination URL for the request should be a filter on a resource's #index ("give me all the tag names via the /tags resource starting with the string 'abc'.."). This means submitting the request as a GET instead of a POST. Currently, there is no way to specify a :method option for the auto_complete_field helper.

Easy fix by adding support for the parameter to the helper in java_script_macro_helpers.rb. Including a patch (w/ tests).

Attachments

auto_complete_field_method_option_patch.diff (2.5 kB) - added by zapnap on 04/19/07 16:59:57.
patch to add :method option to auto_complete_field

Change History

04/19/07 16:59:57 changed by zapnap

  • attachment auto_complete_field_method_option_patch.diff added.

patch to add :method option to auto_complete_field

04/21/07 17:34:18 changed by zapnap

  • type changed from enhancement to defect.

05/20/07 19:09:56 changed by zapnap

  • keywords changed from rest autocompletion auto_complete_field to tiny patch rest autocompletion auto_complete_field.
  • summary changed from auto_complete_field lacks :method option to [PATCH] [TINY] auto_complete_field lacks :method option.

05/20/07 19:12:45 changed by zapnap

  • keywords changed from tiny patch rest autocompletion auto_complete_field to tiny patch.
  • summary changed from [PATCH] [TINY] auto_complete_field lacks :method option to [PATCH][TINY] auto_complete_field lacks :method option.

05/26/07 00:32:56 changed by bitsweat

  • status changed from new to closed.
  • resolution set to fixed.

(In [6851]) auto_complete_field takes a :method option so you can GET or POST. Closes #8120.