default phrase should not force a blank option tag, now it is optional

1 files changed, 4 insertions, 2 deletions

diff --git a/lib/sms_fu_helper.rb b/lib/sms_fu_helper.rb
index 5730444..51b337e 100644
--- a/lib/sms_fu_helper.rb
+++ b/lib/sms_fu_helper.rb
@@ -10,8 +10,10 @@ module SMSFuHelper
   # - name => name of the method in which you want to store the carrier name
   # - phrase => default selected blank option in select box 
   # - selected => carrier to pre-select
-  def carrier_select(name = :mobile_carrier, phrase = "Select a Carrier", selected = nil)
-    select_tag name, options_for_select([phrase,nil]+carrier_collection, selected || phrase)
+  # - include_blank => Do you want the blank option tag (in ADDITION to the value-less 'phrase' option above)?
+  def carrier_select(name = :mobile_carrier, phrase = "Select a Carrier", selected = nil, include_blank = true)
+    options = phrase.nil? ? carrier_collection : include_blank ? [phrase,nil] + carrier_collection : [phrase] + carrier_collection
+    select_tag name, options_for_select(options, selected || phrase)
   end
 
 end