lib/ragoon/services/notification.rb in ragoon-0.3.0 vs lib/ragoon/services/notification.rb in ragoon-0.3.1

@@ -21,11 +21,13 @@
         )
       end
     end
 
     notifications = notification_get_notifications_by_id(items)
-    keys = notifications.xpath('//notification').first.attributes.keys.map(&:to_sym)
+    first_item = notifications.xpath('//notification').first
+    return [] if first_item.nil?
+    keys = first_item.attributes.keys.map(&:to_sym)
 
     notifications.xpath('//notification').each_with_object([]) do |notification, result|
       result.push(
         keys.each_with_object({}) { |key, hash| hash[key] = notification[key] }
       )
@@ -58,28 +60,18 @@
     parameter_node = Ragoon::XML.create_node('parameters')
 
     notification_items.each do |notification_item|
       parameter_node.add_child(
         Ragoon::XML.create_node(
-        'notification_id',
-        module_id: notification_item[:module_id],
-        item:      notification_item[:item],
+          'notification_id',
+          module_id: notification_item[:module_id],
+          item:      notification_item[:item]
+        )
       )
-      )
     end
 
     body_node.add_child(parameter_node)
 
     client.request(action_name, body_node)
     client.result_set
-  end
-
-
-  def default_options(action_name)
-    case action_name
-    when 'ScheduleGetEvents'
-      Ragoon::Services.start_and_end
-    else
-      {}
-    end
   end
 end