diff options
Diffstat (limited to 'lib')
| -rw-r--r-- | lib/sms_fu.rb | 2 | ||||
| -rw-r--r-- | lib/sms_fu_helper.rb | 15 | ||||
| -rw-r--r-- | lib/sms_notifier.rb | 1 |
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 |