pipedrive-ruby 0.2.4 → 0.2.5

data/Rakefile

@@ -20,7 +20,7 @@ Jeweler::Tasks.new do |gem|
   gem.summary = %Q{Ruby wrapper for the Pipedrive API}
   gem.description = %Q{Ruby wrapper for the Pipedrive API}
   gem.email = "jan@general-scripting.com"
-  gem.authors = ["Waldemar Kusnezow", "Jan Schwenzien", "Patrick Kozma"]
+  gem.authors = ["Jan Schwenzien", "Waldemar Kusnezow"]
   # dependencies defined in Gemfile
 end
 Jeweler::RubygemsDotOrgTasks.new

data/VERSION

@@ -1 +1 @@
-0.2.4
+0.2.5

data/lib/pipedrive-ruby.rb

@@ -2,6 +2,7 @@ require 'pipedrive/base'
 require 'pipedrive/deal'
 require 'pipedrive/organization'
 require 'pipedrive/person'
+require 'pipedrive/product'
 
 module Pipedrive
 

data/lib/pipedrive/base.rb

@@ -1,45 +1,26 @@
 require 'httparty'
 require 'ostruct'
+require 'forwardable'
 
 module Pipedrive
-  
+
   # Globally set request headers
   HEADERS = {
     "User-Agent"    => "Ruby.Pipedrive.Api",
     "Accept"        => "application/json",
     "Content-Type"  => "application/x-www-form-urlencoded"
   }
-  
+
   # Base class for setting HTTParty configurations globally
   class Base < OpenStruct
-    
+
     include HTTParty
     base_uri 'api.pipedrive.com/v1'
     headers HEADERS
     format :json
-    
-    # Sets the authentication credentials in a class variable.
-    #
-    # @param [String] email cl.ly email
-    # @param [String] password cl.ly password
-    # @return [Hash] authentication credentials
-    def self.authenticate(token)
-      self.default_params :api_token => token
-    end
-    
-    # Examines a bad response and raises an approriate exception
-    #
-    # @param [HTTParty::Response] response
-    def self.bad_response(response)
-      if response.class == HTTParty::Response
-        raise ResponseError, response
-      end
-      raise StandardError, "Unkown error"
-    end
 
-    def self.new_list( attrs )
-      attrs['data'].is_a?(Array) ? attrs['data'].map {|data| self.new( 'data' => data ) } : []
-    end
+    extend Forwardable
+    def_delegators 'self.class', :get, :post, :put, :resource_path, :bad_response
 
     attr_reader :data
 
@@ -50,9 +31,86 @@ module Pipedrive
     # @param [Hash] attributes
     # @return [CloudApp::Base]
     def initialize(attrs = {})
-      super( attrs['data'] )
+      if attrs['data']
+        super( attrs['data'] )
+      else
+        super(attrs)
+      end
+    end
+
+    # Updates the object.
+    #
+    # @param [Hash] opts
+    # @return [Boolean]
+    def update(opts = {})
+      res = put "#{resource_path}/#{id}", :body => opts
+      !!(res.success? && @table.merge!(res['data'].symbolize_keys))
+    end
+
+    class << self
+      # Sets the authentication credentials in a class variable.
+      #
+      # @param [String] email cl.ly email
+      # @param [String] password cl.ly password
+      # @return [Hash] authentication credentials
+      def authenticate(token)
+        default_params :api_token => token
+      end
+
+      # Examines a bad response and raises an appropriate exception
+      #
+      # @param [HTTParty::Response] response
+      def bad_response(response)
+        if response.class == HTTParty::Response
+          raise HTTParty::ResponseError, response
+        end
+        raise StandardError, 'Unknown error'
+      end
+
+      def new_list( attrs )
+        attrs['data'].is_a?(Array) ? attrs['data'].map {|data| self.new( 'data' => data ) } : []
+      end
+
+      def all(response = nil)
+        res = response || get(resource_path)
+        if res.ok?
+          res['data'].nil? ? [] : res['data'].map{|obj| new(obj)}
+        else
+          bad_response(res)
+        end
+      end
+
+      def create( opts = {} )
+        res = post resource_path, :body => opts
+        if res.success?
+          res['data'] = opts.merge res['data']
+          new(res)
+        else
+          bad_response(res)
+        end
+      end
+
+      def find(id)
+        res = get "#{resource_path}/#{id}"
+        res.ok? ? new(res) : bad_response(res)
+      end
+
+      def find_by_name(name, opts={})
+        res = get "#{resource_path}/find", :query => { :term => name }.merge(opts)
+        res.ok? ? new_list(res) : bad_response(res)
+      end
+
+      def resource_path
+        "/#{name.split('::').last.downcase}s"
+      end
     end
-    
-  end  
-  
+
+    # TODO Rewrite this.
+    module Deals
+      def deals
+        Deal.all(get "#{resource_path}/#{id}/deals")
+      end
+    end
+  end
+
 end

data/lib/pipedrive/deal.rb

@@ -1,26 +1,10 @@
 module Pipedrive
-
   class Deal < Base
 
-    def self.create( opts = {} )
-      res = post "/deals", :body => opts
-      if res.success?
-        res['data'] = opts.merge res['data']
-        Deal.new(res)
-      else
-        bad_response(res)
-      end
-    end
-
-    def self.find(id)
-      res = get "/deals/#{id}"
-      if res.ok?
-        Deal.new(res)
-      else
-        bad_response(res)
-      end
+    def add_product(opts = {})
+      res = post "#{resource_path}/#{id}/products", :body => opts
+      res.success? ? res['data']['product_attachment_id'] : bad_response(res)
     end
 
   end
-
 end

data/lib/pipedrive/organization.rb

@@ -1,39 +1,18 @@
 module Pipedrive
-
   class Organization < Base
+    include Deals
 
-    def self.create( opts = {} )
-      res = post "/organizations", :body => opts
-      if res.success?
-        res['data'] = opts.merge res['data']
-        Organization.new(res)
-      else
-        bad_response(res)
-      end
+    # TODO Rewrite this.
+    def persons
+      Person.all(get "#{resource_path}/#{id}/persons")
     end
 
-    def self.find_or_create_by_name( name, opts={} )
-      find_by_name( name ).first || create( opts.merge( :title => name ) )
-    end
+    class << self
 
-    def self.find(id)
-      res = get "/organizations/#{id}"
-      if res.ok?
-        Organization.new(res)
-      else
-        bad_response(res)
+      def find_or_create_by_name(name, opts={})
+        find_by_name(name).first || create(opts.merge(:title => name))
       end
-    end
 
-    def self.find_by_name(name, opts={})
-      res = get "/organizations/find", :query => { :term => name }.merge(opts)
-      if res.ok?
-        Organization.new_list(res)
-      else
-        bad_response(res)
-      end
     end
-
   end
-
 end

data/lib/pipedrive/person.rb

@@ -1,39 +1,12 @@
 module Pipedrive
-
   class Person < Base
 
-    def self.create( opts = {} )
-      res = post "/persons", :body => opts
-      if res.success?
-        res['data'] = opts.merge res['data']
-        Person.new(res)
-      else
-        bad_response(res)
-      end
-    end
-
-    def self.find_or_create_by_name( name, opts={} )
-      find_by_name( name, :org_id => opts[:org_id] ).first || create( opts.merge( :name => name ) )
-    end
+    class << self
 
-    def self.find(id)
-      res = get "/persons/#{id}"
-      if res.ok?
-        Person.new(res)
-      else
-        bad_response(res)
+      def find_or_create_by_name(name, opts={})
+        find_by_name(name, :org_id => opts[:org_id]).first || create(opts.merge(:name => name))
       end
-    end
 
-    def self.find_by_name(name, opts={})
-      res = get "/persons/find", :query => { :term => name }.merge(opts)
-      if res.ok?
-        Person.new_list(res)
-      else
-        bad_response(res)
-      end
     end
-
   end
-
 end

data/lib/pipedrive/product.rb

@@ -0,0 +1,6 @@
+module Pipedrive
+  class Product < Base
+    include Deals
+
+  end
+end

data/pipedrive-ruby.gemspec

@@ -5,11 +5,11 @@
 
 Gem::Specification.new do |s|
   s.name = "pipedrive-ruby"
-  s.version = "0.2.4"
+  s.version = "0.2.5"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
-  s.authors = ["Waldemar Kusnezow", "Jan Schwenzien", "Patrick Kozma"]
-  s.date = "2012-11-06"
+  s.authors = ["Jan Schwenzien", "Waldemar Kusnezow"]
+  s.date = "2013-02-09"
   s.description = "Ruby wrapper for the Pipedrive API"
   s.email = "jan@general-scripting.com"
   s.extra_rdoc_files = [
@@ -27,6 +27,7 @@ Gem::Specification.new do |s|
     "lib/pipedrive/deal.rb",
     "lib/pipedrive/organization.rb",
     "lib/pipedrive/person.rb",
+    "lib/pipedrive/product.rb",
     "pipedrive-ruby.gemspec",
     "test/helper.rb",
     "test/test_pipedrive-rails.rb"

metadata

@@ -1,17 +1,16 @@
 --- !ruby/object:Gem::Specification
 name: pipedrive-ruby
 version: !ruby/object:Gem::Version
-  version: 0.2.4
+  version: 0.2.5
   prerelease: 
 platform: ruby
 authors:
-- Waldemar Kusnezow
 - Jan Schwenzien
-- Patrick Kozma
+- Waldemar Kusnezow
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-11-06 00:00:00.000000000 Z
+date: 2013-02-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: httparty
@@ -127,6 +126,7 @@ files:
 - lib/pipedrive/deal.rb
 - lib/pipedrive/organization.rb
 - lib/pipedrive/person.rb
+- lib/pipedrive/product.rb
 - pipedrive-ruby.gemspec
 - test/helper.rb
 - test/test_pipedrive-rails.rb
@@ -145,7 +145,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 3219907950570205056
+      hash: -526297699486921758
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements: