From d780c2a6b3e2765d655e925ae1cbafabdb3b7777 Mon Sep 17 00:00:00 2001
From: CactusPuppy <puppycactus@gmail.com>
Date: Thu, 1 Apr 2021 23:10:43 -0700
Subject: [PATCH] Make body argument for query_api more clear

---
 app/services/mailbluster_manager.rb | 30 ++++++++++++++---------------
 1 file changed, 15 insertions(+), 15 deletions(-)

diff --git a/app/services/mailbluster_manager.rb b/app/services/mailbluster_manager.rb
index ace60586..3925d1dd 100644
--- a/app/services/mailbluster_manager.rb
+++ b/app/services/mailbluster_manager.rb
@@ -8,33 +8,33 @@ class MailblusterManager
     }
   }
 
-  def self.query_api(method, path, body)
+  def self.query_api(method, path, body: {})
     options = @auth_headers.merge(body: body.to_json)
     send(method, path, options).parsed_response
   end
 
   def self.create_lead(user)
-    query_api(:post, '/',
-              'email'            => user.email,
-              'firstName'        => user.name,
-              'overrideExisting' => true,
-              'subscribed'       => true,
-              'tags'             => [ENV['OSEM_NAME'] || 'snapcon']
-    )
+    query_api(:post, '/', body: {
+                'email'            => user.email,
+                'firstName'        => user.name,
+                'overrideExisting' => true,
+                'subscribed'       => true,
+                'tags'             => [ENV['OSEM_NAME'] || 'snapcon']
+              })
   end
 
   def self.edit_lead(user, add_tags: [], remove_tags: [], old_email: nil)
     email_hash = Digest::MD5.hexdigest(old_email.presence || user.email)
-    query_api(:put, "/#{email_hash}",
-              'email'      => user.email,
-              'firstName'  => user.name,
-              'addTags'    => add_tags,
-              'removeTags' => remove_tags
-    )
+    query_api(:put, "/#{email_hash}", body: {
+                'email'      => user.email,
+                'firstName'  => user.name,
+                'addTags'    => add_tags,
+                'removeTags' => remove_tags
+              })
   end
 
   def self.delete_lead(user)
     email_hash = Digest::MD5.hexdigest user.email
-    query_api(:delete, "/#{email_hash}", {})
+    query_api(:delete, "/#{email_hash}")
   end
 end