Add deletion by ID and User ID (#200)

Author: thewheat

README.md

@@ -201,6 +201,14 @@ contact = Contact.findByUserID("e1a7d875-d83a-46f7-86f4-73be98a98584");
 contact.setName("Stitch Hessian");
 Contact updated = Contact.update(contact);
 
+// Update a contact by ID
+Contact contact = new Contact().setID("541a144b201ebf2ec5000002").setName("Stitch Hessian");
+Contact updated = Contact.update(contact);
+
+// Update a contact by User ID
+Contact contact = new Contact().setUserID("e1a7d875-d83a-46f7-86f4-73be98a98584").setName("Stitch Hessian");
+Contact updated = Contact.update(contact);
+
 // Read a contact list by email
 ContactCollection contacts = Contact.listByEmail("[email protected]");
 while(contacts.hasNext()) {
@@ -218,10 +226,21 @@ ScrollableContactCollection contactsScroll = Contact.scroll();
 List<Contact> contacts = contactsScroll.getPage();
 contactsScroll = contactsScroll.scroll();
 
+// List contacts (sorting)
+Map<String, String> params = Maps.newHashMap();
+params.put("sort", "created_at");
+params.put("order", "asc");
+ContactCollection contacts = Contact.list(params);
 
 // Remove a contact
 Contact.delete(contact);
 
+// Remove a contact by id
+Contact.delete(contact.getID());
+
+// Remove a contact by user_id
+Contact.deleteByUserID(contact.getUserID());
+
 // Convert a contact
 User converted = Contact.convert(contact, user);
 ```

intercom-java/src/main/java/io/intercom/api/Contact.java

@@ -6,6 +6,7 @@
 import com.fasterxml.jackson.annotation.JsonInclude;
 import com.fasterxml.jackson.annotation.JsonProperty;
 import com.google.common.annotations.VisibleForTesting;
+import com.google.common.base.Strings;
 import com.google.common.collect.Maps;
 
 import java.net.URI;
@@ -66,7 +67,27 @@ public static Contact update(Contact c)
 
     public static Contact delete(Contact c)
             throws AuthorizationException, ClientException, ServerException, InvalidException, RateLimitException {
-        return DataResource.delete(c.getID(), "contacts", Contact.class);
+        if(!Strings.isNullOrEmpty(c.getID())) {
+            return delete(c.getID());
+        }
+        else if(!Strings.isNullOrEmpty(c.getUserID())) {
+            return deleteByUserID(c.getUserID());
+        }
+        else {
+            throw new InvalidException("to delete a contact you must provide a id or user_id value");
+        }
+    }
+
+    public static Contact delete(String id)
+            throws AuthorizationException, ClientException, ServerException, InvalidException, RateLimitException {
+        return DataResource.delete(id, "contacts", Contact.class);
+    }
+
+    public static Contact deleteByUserID(String user_id)
+            throws AuthorizationException, ClientException, ServerException, InvalidException, RateLimitException {
+        Map<String, String> params = new HashMap<String,String>();
+        params.put("user_id", user_id);
+        return DataResource.delete(params, "contacts", Contact.class);
     }
 
     public static User convert(Contact c, User u)
@@ -416,8 +437,7 @@ public String getID() {
         return id;
     }
 
-    @VisibleForTesting
-    Contact setID(String id) {
+    public Contact setID(String id) {
         this.id = id;
         return this;
     }
@@ -457,6 +477,11 @@ public String getUserID() {
         return userID;
     }
 
+    public Contact setUserID(String userID) {
+        this.userID = userID;
+        return this;
+    }
+
     public Avatar getAvatar() {
         return avatar;
     }