move AR related code for alert to record class; reorg files

Author: Austin Schneider

app/controllers/alerts_controller.rb

@@ -5,17 +5,13 @@ class AlertsController < ApplicationController
   helper_method :alert
 
   def create
-    current_user.save_alert alert
+    alert.save
     respond_with alert, location: alerts_url
   end
 
   # TODO: test
   def alert
-    @alert ||= begin
-      alert_attrs = params[:alert] || {}
-      alert_attrs.reverse_merge! :location_id => params[:location_id]
-      Alert.new alert_attrs
-    end
+    @alert ||= current_user.build_alert_from_params(params)
   end
 
 end

app/helpers/application_helper.rb

@@ -2,8 +2,11 @@ module ApplicationHelper
 
   # TODO: test
   def present(object, klass = nil, &block)
-    klass ||= "#{object.class}Presenter".constantize
-    presenter = klass.new object, self
+    presenter = if !klass && object.respond_to?(:to_presenter)
+      object.to_presenter(self)
+    else
+      "#{object.class}Presenter".constantize.new object, self
+    end
     if block_given?
       block.arity > 0 ? yield(presenter) : presenter.instance_eval(&block)
     end

app/models/alert.rb

@@ -8,4 +8,8 @@ class EmailCallback < ActiveRecord::Base
     uniqueness: true,
     length: {maximum: 255}
 
+  def to_presenter(view)
+    EmailCallbackPresenter.new self, view
+  end
+
 end

app/models/email_callback.rb

@@ -0,0 +1,16 @@
+require 'alert/record'
+
+class Alert
+
+  class << self
+    def build(attrs)
+      Record.new attrs
+    end
+
+    delegate :joins, :scoped, :unscoped, :table_name,
+      :to => Record
+  end
+
+end
+
+require 'alert/presenter'

app/presenters/alert_presenter.rb

@@ -0,0 +1,48 @@
+class Alert
+  class Presenter < Poser::Presenter
+
+    presents :alert
+
+    def location
+      @location ||= LocationAttribute.new(self)
+    end
+
+    def code_is_not
+      @code_is_not ||= CodeIsNotAttribute.new(self)
+    end
+
+    def times
+      @times ||= TimesAttribute.new(self)
+    end
+
+    def email_callback
+      @email_callback ||= EmailCallbackAttribute.new(self)
+    end
+
+    def location_options_for_select
+      locations.map { |l| [l.title, l.id] }
+    end
+
+    def email_callback_options_for_select
+      EmailCallback.order { label.asc }.map { |ec| [ec.label, ec.id] }
+    end
+
+    # TODO: test
+    attr_accessor :form_builder
+
+    # TODO: test
+    def form
+      form_for alert do |f|
+        self.form_builder = f
+        yield
+        self.form_builder = nil
+      end
+    end
+
+  end
+end
+
+require 'alert/presenter/code_is_not_attribute'
+require 'alert/presenter/email_callback_attribute'
+require 'alert/presenter/location_attribute'
+require 'alert/presenter/times_attribute'

app/presenters/alert_presenter/attribute.rb

@@ -0,0 +1,9 @@
+class Alert
+  class Presenter
+    class Attribute < Poser::Presenter::Attribute
+
+      delegate :alert, to: :presenter
+
+    end
+  end
+end

app/presenters/alert_presenter/code_is_not.rb

@@ -0,0 +1,29 @@
+require 'alert/presenter/attribute'
+
+class Alert
+  class Presenter
+    class CodeIsNotAttribute < Attribute
+
+      def label
+        form_builder.label :code_is_not, 'Response status code is not'
+      end
+
+      def field
+        text_field :code_is_not, max_length: 255, size: 20
+      end
+
+      def errors
+        errors = alert.errors[:code_is_not]
+        if errors.present?
+          render 'shared/form_errors', errors: errors.map(&:capitalize)
+        end
+      end
+
+      # TODO: test
+      def to_s
+        alert.code_is_not.to_s
+      end
+
+    end
+  end
+end

app/presenters/alert_presenter/email_callback_attribute.rb

@@ -0,0 +1,37 @@
+require 'alert/presenter/attribute'
+
+class Alert
+  class Presenter
+    class EmailCallbackAttribute < Attribute
+
+      # TODO: test
+      delegate :link_to, :new_email_callback_path, :email_callback_options_for_select, to: :presenter
+
+      def label
+        form_builder.label :email_callback_id, 'Alert via'
+      end
+
+      # TODO: test
+      def field
+        if email_callback_options_for_select.empty?
+          "There are no email callbacks. #{link_to "Add one", new_email_callback_path}".html_safe
+        else
+          select :email_callback_id, email_callback_options_for_select
+        end
+      end
+
+      def errors
+        errors = alert.errors[:email_callback_id]
+        if errors.present?
+          render 'shared/form_errors', errors: errors.map(&:capitalize)
+        end
+      end
+
+      # TODO: test
+      def to_s
+        alert.email_callback.try(:label).to_s
+      end
+
+    end
+  end
+end