[frontend] Refactor Event::Request#payload_with_diff

openSUSE · Apr 5, 2018 · d0cd921 · d0cd921
1 parent 32aaa07
commit d0cd921
Showing 1 changed file with 4 additions and 5 deletions.
diff --git a/src/api/app/models/event/request.rb b/src/api/app/models/event/request.rb
@@ -81,13 +81,12 @@ def payload_with_diff
 
       ret = payload
       payload['actions'].each do |a|
-        diff = calculate_diff(a)
+        diff = calculate_diff(a).try(:lines)
         next unless diff
-        diff = diff.lines
-        dl = diff.length
-        if dl > DIFF_LIMIT
+        diff_length = diff.length
+        if diff_length > DIFF_LIMIT
           diff = diff[0..DIFF_LIMIT]
-          diff << "[cut #{dl - DIFF_LIMIT} lines to limit mail size]"
+          diff << "[cut #{diff_length - DIFF_LIMIT} lines to limit mail size]"
         end
         a['diff'] = diff.join
       end