Some refactoring

2 files changed, 64 insertions, 59 deletions

diff --git a/billing/app/helpers/braintree_form_helper.rb b/billing/app/helpers/braintree_form_helper.rb
new file mode 100644
index 0000000..cb322fa
--- /dev/null
+++ b/billing/app/helpers/braintree_form_helper.rb
@@ -0,0 +1,64 @@
+module BraintreeFormHelper
+  class BraintreeFormBuilder < ActionView::Helpers::FormBuilder
+    include ActionView::Helpers::AssetTagHelper
+    include ActionView::Helpers::TagHelper
+
+    def initialize(object_name, object, template, options, proc)
+      super
+      @braintree_params = @options[:params]
+      @braintree_errors = @options[:errors]
+      @braintree_existing = @options[:existing]
+    end
+
+    def fields_for(record_name, *args, &block)
+      options = args.extract_options!
+      options[:builder] = BraintreeFormBuilder
+      options[:params] = @braintree_params && @braintree_params[record_name]
+      options[:errors] = @braintree_errors && @braintree_errors.for(record_name)
+      new_args = args + [options]
+      super record_name, *new_args, &block
+    end
+
+    def text_field(method, options = {})
+      has_errors = @braintree_errors && @braintree_errors.on(method).any?
+      field = super(method, options.merge(:value => determine_value(method)))
+      result = content_tag("div", field, :class => has_errors ? "fieldWithErrors" : "")
+      result.safe_concat validation_errors(method)
+      result
+    end
+
+    protected
+
+    def determine_value(method)
+      if @braintree_params
+        @braintree_params[method]
+      elsif @braintree_existing
+
+        if @braintree_existing.kind_of?(Braintree::CreditCard)
+
+          case method
+          when :number
+            method = :masked_number
+          when :cvv
+            return nil
+          end
+        end
+
+        @braintree_existing.send(method)
+      else
+        nil
+      end
+    end
+
+    def validation_errors(method)
+      if @braintree_errors && @braintree_errors.on(method).any?
+        @braintree_errors.on(method).map do |error|
+          content_tag("div", ERB::Util.h(error.message), {:style => "color: red;"})
+        end.join
+      else
+        ""
+      end
+    end
+  end
+end
+
diff --git a/billing/app/helpers/braintree_helper.rb b/billing/app/helpers/braintree_helper.rb
index bc78c02..2d18b6c 100644
--- a/billing/app/helpers/braintree_helper.rb
+++ b/billing/app/helpers/braintree_helper.rb
@@ -1,64 +1,5 @@
 module BraintreeHelper
-  class BraintreeFormBuilder < ActionView::Helpers::FormBuilder
-    include ActionView::Helpers::AssetTagHelper
-    include ActionView::Helpers::TagHelper
 
-    def initialize(object_name, object, template, options, proc)
-      super
-      @braintree_params = @options[:params]
-      @braintree_errors = @options[:errors]
-      @braintree_existing = @options[:existing]
-    end
 
-    def fields_for(record_name, *args, &block)
-      options = args.extract_options!
-      options[:builder] = BraintreeFormBuilder
-      options[:params] = @braintree_params && @braintree_params[record_name]
-      options[:errors] = @braintree_errors && @braintree_errors.for(record_name)
-      new_args = args + [options]
-      super record_name, *new_args, &block
-    end
-
-    def text_field(method, options = {})
-      has_errors = @braintree_errors && @braintree_errors.on(method).any?
-      field = super(method, options.merge(:value => determine_value(method)))
-      result = content_tag("div", field, :class => has_errors ? "fieldWithErrors" : "")
-      result.safe_concat validation_errors(method)
-      result
-    end
-
-    protected
-
-    def determine_value(method)
-      if @braintree_params
-        @braintree_params[method]
-      elsif @braintree_existing
-
-        if @braintree_existing.kind_of?(Braintree::CreditCard)
-
-          case method
-          when :number
-            method = :masked_number
-          when :cvv
-            return nil
-          end
-        end
-
-        @braintree_existing.send(method)
-      else
-        nil
-      end
-    end
-
-    def validation_errors(method)
-      if @braintree_errors && @braintree_errors.on(method).any?
-        @braintree_errors.on(method).map do |error|
-          content_tag("div", ERB::Util.h(error.message), {:style => "color: red;"})
-        end.join
-      else
-        ""
-      end
-    end
-  end
 end