소스 검색

Fix for serialization error

Andrew Kane 9 년 전
부모
커밋
8bb651ac04
3개의 변경된 파일5개의 추가작업 그리고 5개의 파일을 삭제
  1. 2 2
      app/mailers/blazer/check_mailer.rb
  2. 1 1
      app/models/blazer/check.rb
  3. 2 2
      app/views/blazer/check_mailer/state_change.html.erb

+ 2 - 2
app/mailers/blazer/check_mailer.rb

@@ -4,11 +4,11 @@ module Blazer
 
     default from: Blazer.from_email if Blazer.from_email
 
-    def state_change(check, state, state_was, rows, error)
+    def state_change(check, state, state_was, rows_count, error)
       @check = check
       @state = state
       @state_was = state_was
-      @rows = rows
+      @rows_count = rows_count
       @error = error
       mail to: check.emails, subject: "Check #{state.titleize}: #{check.blazer_query.name}"
     end

+ 1 - 1
app/models/blazer/check.rb

@@ -14,7 +14,7 @@ module Blazer
 
       # do not notify on creation, except when not passing
       if (state_was || state != "passing") && state != state_was && emails.present?
-        Blazer::CheckMailer.state_change(self, state, state_was, rows, error).deliver_later
+        Blazer::CheckMailer.state_change(self, state, state_was, rows.size, error).deliver_later
       end
       save!
     end

+ 2 - 2
app/views/blazer/check_mailer/state_change.html.erb

@@ -1,6 +1,6 @@
 <p><%= link_to "View", query_url(@check.blazer_query_id) %></p>
 <% if @error %>
   <p><%= @error %></p>
-<% elsif @rows.any? %>
-  <p><%= pluralize(@rows.size, "row") %></p>
+<% elsif @rows_count > 0 %>
+  <p><%= pluralize(@rows_count, "row") %></p>
 <% end %>