HTTPS support and explicit update failure reporting

lib/pipedrive/base.rb

@@ -15,8 +15,8 @@ module Pipedrive
   class Base < OpenStruct
 
     include HTTParty
-    
-    base_uri 'api.pipedrive.com/v1'
+
+    base_uri 'https://api.pipedrive.com/v1'
     headers HEADERS
     format :json
 
@@ -49,16 +49,24 @@ def initialize(attrs = {})
     #
     # @param [Hash] opts
     # @return [Boolean]
-    def update(opts = {})
+    def update(opts = {}, raise_error=false)
       res = put "#{resource_path}/#{id}", :body => opts
       if res.success?
         res['data'] = Hash[res['data'].map {|k, v| [k.to_sym, v] }]
         @table.merge!(res['data'])
       else
-        false
+        if raise_error
+          raise StandardError.new("update to #{resource_path}/#{id} failed with #{res.code} #{res.message} #{res.body}")
+        else
+          false
+        end
       end
     end
 
+    def update!(opts = {})
+      update(opts, true)
+    end
+
     class << self
       # Sets the authentication credentials in a class variable.
       #
@@ -107,7 +115,7 @@ def create( opts = {} )
           bad_response(res,opts)
         end
       end
-      
+
       def find(id)
         res = get "#{resource_path}/#{id}"
         res.ok? ? new(res) : bad_response(res,id)
@@ -118,6 +126,11 @@ def find_by_name(name, opts={})
         res.ok? ? new_list(res) : bad_response(res,{:name => name}.merge(opts))
       end
 
+      def find_with_options(opts = { start: 0 })
+        res = get "#{resource_path}", query: opts
+        res.ok? ? new_list(res) : bad_response(res,opts)
+      end
+
       def resource_path
         # The resource path should match the camelCased class name with the
         # first letter downcased.  Pipedrive API is sensitive to capitalisation

lib/pipedrive/deal.rb

@@ -9,7 +9,7 @@ def add_product(opts = {})
     def products
       Product.all(get "#{resource_path}/#{id}/products")
     end
-    
+
     def remove_product product_attachment_id
       res = delete "#{resource_path}/#{id}/products", { :body => { :product_attachment_id => product_attachment_id } }
       res.success? ? nil : bad_response(res,product_attachment_id)
@@ -26,6 +26,5 @@ def files
     def notes(opts = {:sort_by => 'add_time', :sort_mode => 'desc'})
       Note.all( get("/notes", :query => opts.merge(:deal_id => id) ) )
     end
-
   end
 end