Style fixes.

lib/pipedrive/activity-type.rb

@@ -1,4 +1,4 @@
 module Pipedrive
   class ActivityType < Base
   end
-end
+end

lib/pipedrive/activity.rb

@@ -1,4 +1,4 @@
 module Pipedrive
   class Activity < Base
   end
-end
+end

lib/pipedrive/authorization.rb

@@ -1,4 +1,4 @@
 module Pipedrive
   class Authorization < Base
   end
-end
+end

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
 
@@ -72,50 +72,50 @@ def authenticate(token)
       # Examines a bad response and raises an appropriate exception
       #
       # @param [HTTParty::Response] response
-      def bad_response(response, params={})
+      def bad_response(response, params = {})
         puts params.inspect
         if response.class == HTTParty::Response
           raise HTTParty::ResponseError, response
         end
         raise StandardError, 'Unknown error'
       end
 
-      def new_list( attrs )
+      def new_list(attrs)
         attrs['data'].is_a?(Array) ? attrs['data'].map {|data| self.new( 'data' => data ) } : []
       end
 
-      def all(response = nil, options={},get_absolutely_all=false)
+      def all(response = nil, options = {}, get_absolutely_all = false)
         res = response || get(resource_path, options)
         if res.ok?
           data = res['data'].nil? ? [] : res['data'].map{|obj| new(obj)}
           if get_absolutely_all && res['additional_data']['pagination'] && res['additional_data']['pagination'] && res['additional_data']['pagination']['more_items_in_collection']
             options[:query] = options[:query].merge({:start => res['additional_data']['pagination']['next_start']})
-            data += self.all(nil,options,true)
+            data += self.all(nil, options, true)
           end
           data
         else
-          bad_response(res,attrs)
+          bad_response(res, options)
         end
       end
 
-      def create( opts = {} )
+      def create(opts = {})
         res = post resource_path, :body => opts
         if res.success?
           res['data'] = opts.merge res['data']
           new(res)
         else
-          bad_response(res,opts)
+          bad_response(res, opts)
         end
       end
-      
+
       def find(id)
         res = get "#{resource_path}/#{id}"
-        res.ok? ? new(res) : bad_response(res,id)
+        res.ok? ? new(res) : bad_response(res, id)
       end
 
-      def find_by_name(name, opts={})
+      def find_by_name(name, opts = {})
         res = get "#{resource_path}/find", :query => { :term => name }.merge(opts)
-        res.ok? ? new_list(res) : bad_response(res,{:name => name}.merge(opts))
+        res.ok? ? new_list(res) : bad_response(res, {:name => name}.merge(opts))
       end
 
       def resource_path

lib/pipedrive/currency.rb

@@ -1,4 +1,4 @@
 module Pipedrive
   class Currency < Base
   end
-end
+end

lib/pipedrive/deal.rb

@@ -3,16 +3,16 @@ class Deal < Base
 
     def add_product(opts = {})
       res = post "#{resource_path}/#{id}/products", :body => opts
-      res.success? ? res['data']['product_attachment_id'] : bad_response(res,opts)
+      res.success? ? res['data']['product_attachment_id'] : bad_response(res, opts)
     end
 
     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)
+      res.success? ? nil : bad_response(res, product_attachment_id)
     end
 
     def activities
@@ -24,8 +24,8 @@ def files
     end
 
     def notes(opts = {:sort_by => 'add_time', :sort_mode => 'desc'})
-      Note.all( get("/notes", :query => opts.merge(:deal_id => id) ) )
+      Note.all(get("/notes", :query => opts.merge(:deal_id => id)))
     end
-    
+
   end
 end

lib/pipedrive/file.rb

@@ -1,4 +1,4 @@
 module Pipedrive
   class File < Base
   end
-end
+end

lib/pipedrive/filter.rb

@@ -1,4 +1,4 @@
 module Pipedrive
   class Filter < Base
   end
-end
+end

lib/pipedrive/goal.rb

@@ -1,4 +1,4 @@
 module Pipedrive
   class Goal < Base
   end
-end
+end

lib/pipedrive/note.rb

@@ -2,7 +2,7 @@ module Pipedrive
   class Note < Base
 
     class << self
-
     end
+
   end
-end
+end

lib/pipedrive/organization.rb

@@ -16,5 +16,6 @@ def find_or_create_by_name(name, opts={})
       end
 
     end
+
   end
 end

lib/pipedrive/permission-set.rb

@@ -1,4 +1,4 @@
 module Pipedrive
   class PermissionSet < Base
   end
-end
+end

lib/pipedrive/person.rb

@@ -12,5 +12,6 @@ def find_or_create_by_name(name, opts={})
     def deals()
       Deal.all(get "#{resource_path}/#{id}/deals", :everyone => 1)
     end
+
   end
-end
+end

lib/pipedrive/pipeline.rb

@@ -1,17 +1,19 @@
 module Pipedrive
   class Pipeline < Base
+
     def stages
       Stage.all(get "/stages", { :pipeline_id => self.id })
     end
 
     def statistics(id, start_date, end_date)
       res = get("#{resource_path}/#{id}/movement_statistics",
                 :query => {:start_date => start_date, :end_date => end_date})
-      res.ok? ? new(res) : bad_response(res,{:id=>id,:start_date=>start_date,:end_date=>end_date})
+      res.ok? ? new(res) : bad_response(res, {:id => id, :start_date => start_date, :end_date => end_date})
     end
 
     def deals(id, stage_id)
       Pipedrive::Deal.all(get "#{resource_path}/#{id}/deals", :stage_id => stage_id )
     end
+
   end
-end
+end

lib/pipedrive/product-field.rb

@@ -1,4 +1,4 @@
 module Pipedrive
   class ProductField < Base
   end
-end
+end

lib/pipedrive/push-notification.rb

@@ -1,4 +1,4 @@
 module Pipedrive
   class PushNotification < Base
   end
-end
+end

lib/pipedrive/role.rb

@@ -1,4 +1,4 @@
 module Pipedrive
   class Role < Base
   end
-end
+end

lib/pipedrive/search-result.rb

@@ -3,26 +3,26 @@ class SearchResult < Base
 
     # Class Methods
     class << self
-      
+
       def search(term, start=0, limit=nil)
         res = get(resource_path, :query => { :term => term, :start => start, :limit => limit})
         if res.ok?
           res['data'].nil? ? [] : res['data'].map{|obj| new(obj)}
         else
-          bad_response(res,{:term=>term,:start=>start,:limit=>limit})
+          bad_response(res, {:term => term, :start => start, :limit => limit})
         end
       end
-  
+
       def field(term, field_type, field_key, opts={})
         res = get("#{resource_path}/field", :query => opts.merge(:term => term, :field_type => field_type, :field_key => field_key) )
         if res.ok?
           res['data'].nil? ? [] : res['data'].map{|obj| new(obj)}
         else
-          bad_response(res,{:term=>term,:field_type=>field_type,:field_key=>field_key}.merge(opts))
+          bad_response(res, {:term => term, :field_type => field_type, :field_key => field_key}.merge(opts))
         end
       end
-    
+
     end
-    
+
   end
-end
+end

lib/pipedrive/user-connection.rb

@@ -1,4 +1,4 @@
 module Pipedrive
   class UserConnection < Base
   end
-end
+end

lib/pipedrive/user-setting.rb

@@ -1,4 +1,4 @@
 module Pipedrive
   class UserSetting < Base
   end
-end
+end

lib/pipedrive/user.rb

@@ -1,4 +1,4 @@
 module Pipedrive
   class User < Base
   end
-end
+end

test/test_pipedrive_authentication.rb

@@ -9,11 +9,11 @@ class TestPipedriveAuthentication < Test::Unit::TestCase
   should "send authentication token with each request" do
     Pipedrive.authenticate("some-token")
 
-    stub_request(:get, "http://api.pipedrive.com/v1/?api_token=some-token").
+    stub_request(:get, "https://api.pipedrive.com/v1/?api_token=some-token").
       with(:headers => {
-        'Accept'=>'application/json',
-        'Content-Type'=>'application/x-www-form-urlencoded',
-        'User-Agent'=>'Ruby.Pipedrive.Api'
+        'Accept' => 'application/json',
+        'Content-Type' => 'application/x-www-form-urlencoded',
+        'User-Agent' => 'Ruby.Pipedrive.Api'
       }).
       to_return(:status => 200, :body => "", :headers => {})
     Pipedrive::Base.get("/")

test/test_pipedrive_deal.rb

@@ -13,12 +13,12 @@ def setup
       "value" => "37k"
     }
 
-    stub_request(:post, "http://api.pipedrive.com/v1/deals?api_token=some-token").
+    stub_request(:post, "https://api.pipedrive.com/v1/deals?api_token=some-token").
       with(:body => body,
         :headers => {
-          'Accept'=>'application/json',
-          'Content-Type'=>'application/x-www-form-urlencoded',
-          'User-Agent'=>'Ruby.Pipedrive.Api'
+          'Accept' => 'application/json',
+          'Content-Type' => 'application/x-www-form-urlencoded',
+          'User-Agent' => 'Ruby.Pipedrive.Api'
         }).
       to_return(
         :status => 200,
@@ -45,4 +45,4 @@ def setup
     #TODO
     # flunk "to be tested"
   end
-end
+end

test/test_pipedrive_note.rb

@@ -7,17 +7,17 @@ def setup
 
   should "execute a valid person request" do
     body = {
-      "content"=>"whatever html body",
-      "person_id"=>"1"
+      "content" => "whatever html body",
+      "person_id" => "1"
       # org_id
       # deal_id
     }
 
-    stub_request(:post, "http://api.pipedrive.com/v1/notes?api_token=some-token").
+    stub_request(:post, "https://api.pipedrive.com/v1/notes?api_token=some-token").
       with(:body => body, :headers => {
-          'Accept'=>'application/json',
-          'Content-Type'=>'application/x-www-form-urlencoded',
-          'User-Agent'=>'Ruby.Pipedrive.Api'
+          'Accept' => 'application/json',
+          'Content-Type' => 'application/x-www-form-urlencoded',
+          'User-Agent' => 'Ruby.Pipedrive.Api'
         }).
       to_return(
         :status => 200,

test/test_pipedrive_organization.rb

@@ -6,14 +6,14 @@ def setup
   end
 
   should "execute a valid person request" do
-    stub_request(:post, "http://api.pipedrive.com/v1/organizations?api_token=some-token").
+    stub_request(:post, "https://api.pipedrive.com/v1/organizations?api_token=some-token").
       with(:body => {
           "name" => "Dope.org"
         },
         :headers => {
-          'Accept'=>'application/json',
-          'Content-Type'=>'application/x-www-form-urlencoded',
-          'User-Agent'=>'Ruby.Pipedrive.Api'
+          'Accept' => 'application/json',
+          'Content-Type' => 'application/x-www-form-urlencoded',
+          'User-Agent' => 'Ruby.Pipedrive.Api'
         }).
       to_return(
         :status => 200,

test/test_pipedrive_person.rb

@@ -7,17 +7,17 @@ def setup
 
   should "execute a valid person request" do
     body = {
-      "email"=>["john@dope.org"],
-      "name"=>"John Dope",
-      "org_id"=>"404",
-      "phone"=>["0123456789"]
+      "email" => ["john@dope.org"],
+      "name" => "John Dope",
+      "org_id" => "404",
+      "phone" => ["0123456789"]
     }
 
-    stub_request(:post, "http://api.pipedrive.com/v1/persons?api_token=some-token").
+    stub_request(:post, "https://api.pipedrive.com/v1/persons?api_token=some-token").
       with(:body => body, :headers => {
-          'Accept'=>'application/json',
-          'Content-Type'=>'application/x-www-form-urlencoded',
-          'User-Agent'=>'Ruby.Pipedrive.Api'
+          'Accept' => 'application/json',
+          'Content-Type' => 'application/x-www-form-urlencoded',
+          'User-Agent' => 'Ruby.Pipedrive.Api'
         }).
       to_return(
         :status => 200,