active_spy 1.3.2 → 1.3.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 56617f3765e3e7026148c98c1285bec0a7e8d672
-  data.tar.gz: 8ec5dd20bda1b57a6ccfbdb504fe55902b11f5e1
+  metadata.gz: 9eddbc6ab774adc7bef8293f674e5edb6543a136
+  data.tar.gz: 97362082bf6dcea5a8e019a659ad3431056404d2
 SHA512:
-  metadata.gz: 57d0d785bfa03abedaff77350567b59948338a59b7f2efd05752075127671c81d1b3dedd583759c0c37430949a3de4b15f58d424ec15a1cc15e68ec7018f6e78
-  data.tar.gz: 784ec1190721e1b8e235a7a6469bca8676758ee317cc43133e424f73a03f01576bfd9407f3c41d25edd1f5a17314bf0f024b54dd74f4b7e5e47933240155ad16
+  metadata.gz: d2887a5140377f86276206b62c0750aafbf13906ddfe1657c3a51dd6dc19a9b65d7895a015cc8536171b14bb92cbd7b8781ecd1b98fecd28bfa6d872af3b8a52
+  data.tar.gz: 3c5ef3b2fdc8a046adfccfe11aa3104719a8fcb0d7e333be1957eb28fa4f55a02150198288a84805d6f61e3a0040c14fb08640b14abc74fe3659c60382ea03e1

data/VERSION

@@ -1 +1 @@
-1.3.2
+1.3.3

data/active_spy.gemspec

@@ -2,11 +2,11 @@
 # DO NOT EDIT THIS FILE DIRECTLY
 # Instead, edit Jeweler::Tasks in Rakefile, and run 'rake gemspec'
 # -*- encoding: utf-8 -*-
-# stub: active_spy 1.3.2 ruby lib
+# stub: active_spy 1.3.3 ruby lib
 
 Gem::Specification.new do |s|
   s.name = "active_spy"
-  s.version = "1.3.2"
+  s.version = "1.3.3"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.require_paths = ["lib"]

data/app/controllers/active_spy/notifications_controller.rb

@@ -9,17 +9,50 @@ module ActiveSpy
       result = nil
       hooks.each do |hook|
         if hook['post_class'].downcase == params['class']
-          listener = "#{hook['post_class']}Listener".constantize
-          result = listener.new.handle(params['event'])
-          if result.errors.present?
-            Rails.logger.warn("[EVENT][#{hook['post_class']}] Error on save #{result}: #{result.errors}")
-            render json: result.errors
-          else
-            render json: result
-          end and return
+           handle_result and return
         end
       end
       render nothing: true, status: :not_found
     end
   end
+
+  private
+  def handle_result(hook, params)
+    result = get_result(hook, params)
+    if result.is_a? Array
+      handle_array_result(result, params)
+    else
+      handle_model_result(result, params)
+    end
+  end
+
+
+  def get_result(hook, params)
+    listener = "#{hook['post_class']}Listener".constantize
+    result = listener.new.handle(params['event'])
+  end
+
+  def handle_model_result(result, params)
+    if result.errors.present?
+      Rails.logger.warn("[EVENT][#{hook['post_class']}] Error receiving event #{params}")
+      Rails.logger.warn("[EVENT][#{hook['post_class']}] Listener result: #{result}")
+      Rails.logger.warn("[EVENT][#{hook['post_class']}] Result errors: #{result.errors}")
+      render json: result.errors
+    else
+      render nothing: true
+    end
+  end
+
+  def handle_array_result(result, params)
+    model_with_errors = result.select { |m| m.errors.present? }
+    if model_with_errors.any?
+      Rails.logger.warn("[EVENT][#{hook['post_class']}] Error receiving event #{params}")
+      Rails.logger.warn("[EVENT][#{hook['post_class']}] Listener result: #{result}")
+      model_with_errors.each do |model|
+        Rails.logger.warn("[EVENT][#{hook['post_class']}] #{model} errors: #{model.errors}")
+      end
+    else
+      render nothing: true
+    end
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: active_spy
 version: !ruby/object:Gem::Version
-  version: 1.3.2
+  version: 1.3.3
 platform: ruby
 authors:
 - Douglas Camata