Fixed label for bug with specified id - #213

CHANGELOG.md

@@ -3,6 +3,7 @@
 Bugfixes:
 
   - Minor README corrections (#184, @msmithstubbs)
+  - Fixed label for bug with specified id (#213, #221, @datWav)
 
 Features:
 

lib/bootstrap_form/form_builder.rb

@@ -334,7 +334,7 @@ def convert_form_tag_options(method, options = {})
     end
 
     def generate_label(id, name, options, custom_label_col, group_layout)
-      options[:for] = id if acts_like_form_tag
+      options[:for] = id if acts_like_form_tag || !id.nil?
       classes = [options[:class], label_class]
       classes << (custom_label_col || label_col) if get_group_layout(group_layout) == :horizontal
       classes << "required" if required_attribute?(object, name)

test/bootstrap_form_group_test.rb

@@ -27,6 +27,11 @@ def setup
     assert_equal expected, @builder.text_field(:email, skip_label: true)
   end
 
+  test "change label for attribute with specified id" do
+    expected = %{<div class="form-group"><label class="control-label required" for="blabla_blub">Email</label><input class="form-control" id="blabla_blub" name="user[email]" type="text" value="steve@example.com" /></div>}
+    assert_equal expected, @builder.text_field(:email, id: 'blabla_blub')
+  end
+
   test "adding prepend text" do
     expected = %{<div class="form-group"><label class="control-label required" for="user_email">Email</label><div class="input-group"><span class="input-group-addon">@</span><input class="form-control" id="user_email" name="user[email]" type="text" value="steve@example.com" /></div></div>}
     assert_equal expected, @builder.text_field(:email, prepend: '@')
@@ -252,5 +257,5 @@ def setup
 
     expected = %{<div class="form-group"><div class="col-sm-9 col-sm-offset-3"><p class="form-control-static">Bar</p></div></div>}
     assert_equal expected, output
-  end    
+  end
 end