Adding in some view helpers for mobile carrier select boxes and updating README

3 files changed, 16 insertions, 2 deletions

diff --git a/lib/sms_fu.rb b/lib/sms_fu.rb
index 2aa2dea..6e22079 100644
--- a/lib/sms_fu.rb
+++ b/lib/sms_fu.rb
@@ -58,7 +58,7 @@ module SMSFu
   end
   
   def is_valid?(number)
-    return (number.length >= 10 && number[/^.\d+$/]) ? true : false
+    number.length >= 10 && number[/^.\d+$/]
   end  
   
   def determine_sms_email(phone_number, carrier)
diff --git a/lib/sms_fu_helper.rb b/lib/sms_fu_helper.rb
new file mode 100644
index 0000000..fdc67c1
--- /dev/null
+++ b/lib/sms_fu_helper.rb
@@ -0,0 +1,15 @@
+module SMSFuHelper  
+  # Returns a collection of carriers to be used in your own select tag
+  # e.g., <%= f.select :mobile_carrier, carrier_collection %>
+  def carrier_collection
+    SMSFu.carriers.sort.collect{ |carrier| [carrier[0].titleize, carrier[0]] }
+  end
+  
+  # Returns a formatted select box filled with carriers
+  # e.g., <%= carrier_select %>
+  # - name => name of the method in which you want to store the carrier name
+  # - phrase => default selected blank option in select box 
+  def carrier_select(name = :mobile_carrier, phrase = "Select a Carrier")
+    select_tag name, options_for_select([phrase,nil]+carrier_collection, phrase)
+  end
+end
\ No newline at end of file
diff --git a/lib/sms_notifier.rb b/lib/sms_notifier.rb
index 5d960c6..644408a 100644
--- a/lib/sms_notifier.rb
+++ b/lib/sms_notifier.rb
@@ -27,7 +27,6 @@ class SmsNotifier < ActionMailer::Base
     content_type      "text/plain"
     recipients        recipient
     from              sender_email
-    
     body['message'] = message
   end