Ticket #5983: add_index_support_to_text_field_with_auto_complete.diff

actionpack/test/template/java_script_macros_helper_test.rb

@@ -60 +60 @@
-
-    assert_dom_equal %(<input id=\"message_recipient\" name=\"message[recipient]\" size=\"30\" type=\"text\" /><div class=\"auto_complete\" id=\"message_recipient_auto_complete\"></div><script type=\"text/javascript\">\n//<![CDATA[\nvar message_recipient_auto_completer = new Ajax.Autocompleter('message_recipient', 'message_recipient_auto_complete', 'http://www.example.com/auto_complete_for_message_recipient', {})\n//]]>\n</script>),
-      text_field_with_auto_complete(:message, :recipient, {}, :skip_style => true)
+
+    assert_dom_equal %(<input id=\"message_0_recipient\" name=\"message[0][recipient]\" size=\"30\" type=\"text\" /><div class=\"auto_complete\" id=\"message_0_recipient_auto_complete\"></div><script type=\"text/javascript\">\n//<![CDATA[\nvar message_0_recipient_auto_completer = new Ajax.Autocompleter('message_0_recipient', 'message_0_recipient_auto_complete', 'http://www.example.com/auto_complete_for_message_recipient', {})\n//]]>\n</script>),
+      text_field_with_auto_complete(:message, :recipient, {:index => 0}, :skip_style => true)
-  end
-  
-  def test_in_place_editor_external_control

actionpack/lib/action_view/helpers/java_script_macros_helper.rb

@@ -193 +193 @@
-      # 
-      # See the RDoc on ActionController::AutoComplete to learn more about this.
-      def text_field_with_auto_complete(object, method, tag_options = {}, completion_options = {})
+        if(tag_options[:index])
+          tag_name = "#{object}_#{tag_options[:index]}_#{method}"
+        else
+          tag_name = "#{object}_#{method}"
+        end
+
-        (completion_options[:skip_style] ? "" : auto_complete_stylesheet) +
-        text_field(object, method, tag_options) +
-"#{object}_#{method}
-"#{object}_#{method}"
+tag_name + "
+tag_name
-      end
-      
-      private