Merge pull request bootstrap-ruby#229 from Stellenticket/select-block

Allow to pass block to select helper in Rails >= 4.1.0

Author: carloslopes

CHANGELOG.md

@@ -7,6 +7,7 @@ Bugfixes:
 Features:
 
   - Allow primary button classes to be overridden (#183, @tanimichi)
+  - Allow to pass a block to select helper in Rails >= 4.1.0 (#229, @doits)
 
 ## 2.3.0 (2015-02-17)
 

lib/bootstrap_form/form_builder.rb

@@ -66,9 +66,17 @@ def file_field_with_bootstrap(name, options = {})
 
     alias_method_chain :file_field, :bootstrap
 
-    def select_with_bootstrap(method, choices, options = {}, html_options = {})
-      form_group_builder(method, options, html_options) do
-        select_without_bootstrap(method, choices, options, html_options)
+    if Gem::Version.new(Rails::VERSION::STRING) >= Gem::Version.new("4.1.0")
+      def select_with_bootstrap(method, choices = nil, options = {}, html_options = {}, &block)
+        form_group_builder(method, options, html_options) do
+          select_without_bootstrap(method, choices, options, html_options, &block)
+        end
+      end
+    else
+      def select_with_bootstrap(method, choices, options = {}, html_options = {})
+        form_group_builder(method, options, html_options) do
+          select_without_bootstrap(method, choices, options, html_options)
+        end
       end
     end
 

test/bootstrap_selects_test.rb

@@ -32,6 +32,17 @@ def setup
     assert_equal expected, @builder.select(:status, [['activated', 1], ['blocked', 2]], { prompt: "Please Select" }, class: "my-select")
   end
 
+  if Gem::Version.new(Rails::VERSION::STRING) >= Gem::Version.new("4.1.0")
+    test "selects with block use block as content" do
+      expected = %{<div class="form-group"><label class="control-label" for="user_status">Status</label><select class="form-control" name="user[status]" id="user_status"><option>Option 1</option><option>Option 2</option></select></div>}
+      select = @builder.select(:status) do
+        content_tag(:option) { 'Option 1' } +
+        content_tag(:option) { 'Option 2' }
+      end
+      assert_equal expected, select
+    end
+  end
+
   test "selects render labels properly" do
     expected = %{<div class="form-group"><label class="control-label" for="user_status">User Status</label><select class="form-control" id="user_status" name="user[status]"><option value="1">activated</option>\n<option value="2">blocked</option></select></div>}
     assert_equal expected, @builder.select(:status, [['activated', 1], ['blocked', 2]], label: "User Status")