rubyzoho 0.1.3 → 0.1.4

data/README.rdoc

@@ -2,6 +2,7 @@
 
 {<img src="https://gemnasium.com/amalc/rubyzoho.png" alt="Dependency Status" />}[https://gemnasium.com/amalc/rubyzoho]
 
+
 = rubyzoho
 
 Abstracting Zoho's API into a set of Ruby classes, with reflection of Zoho's fields using a more familiar
@@ -194,8 +195,20 @@ either the related \module's record id, which is stored with the Task. But no, c
 == Bugs and Enhancements
 Please open an issue on GitHub. Or better yet, send in a pull request with the fix or enhancement!
 
-=== Known Bugs
+=== Known Bugs or Issues
 1. Caching is causing a problem with Quotes.
+2. If you're having trouble with updating custom fields, be sure to check the permission of the user that created the custom field.
+
+=== Roadmap (Ranked)
+1. Get related records using AR style syntax, e.g.
+        pp a.contacts
+   to get contacts associated with an account.
+2. Support for multiple find fields.
+3. AR style master/detail updates e.g. where +a+ is an account.
+    a << RubyZoho::Crm::Contact.new(
+        :last_name => 'Last Name',
+        :first_name => 'First Name'
+    )
 
 == Contributing to rubyzoho
 

data/VERSION

@@ -1 +1 @@
-0.1.3
+0.1.4

data/lib/ruby_zoho.rb

@@ -103,16 +103,22 @@ module RubyZoho
       end
     end
 
-    def create(object_attribute_hash)
-      initialize(object_attribute_hash)
-      save
+    def self.method_missing(meth, *args, &block)
+      if meth.to_s =~ /^find_by_(.+)$/
+        run_find_by_method($1, *args, &block)
+      else
+        super
+      end
     end
 
-    def save
-      h = {}
-      @fields.each { |f| h.merge!({ f => eval("self.#{f.to_s}") }) }
-      h.delete_if { |k, v| v.nil? }
-      RubyZoho.configuration.api.add_record(Crm.module_name, h)
+    def self.run_find_by_method(attrs, *args, &block)
+      attrs = attrs.split('_and_')
+      conditions = Array.new(args.size, '=')
+      h = RubyZoho.configuration.api.find_records(
+          Crm.module_name, ApiUtils.string_to_symbol(attrs[0]), conditions[0], args[0]
+      )
+      return h.collect { |r| new(r) } unless h.nil?
+      nil
     end
 
     def self.all         #TODO Refactor into low level API
@@ -126,26 +132,21 @@ module RubyZoho
       result.collect { |r| new(r) }
     end
 
-    def self.delete(id)
-      RubyZoho.configuration.api.delete_record(Crm.module_name, id)
+    def create(object_attribute_hash)
+      initialize(object_attribute_hash)
+      save
     end
 
-    def self.method_missing(meth, *args, &block)
-      if meth.to_s =~ /^find_by_(.+)$/
-        run_find_by_method($1, *args, &block)
-      else
-        super
-      end
+    def self.delete(id)
+      RubyZoho.configuration.api.delete_record(Crm.module_name, id)
     end
 
-    def self.run_find_by_method(attrs, *args, &block)
-      attrs = attrs.split('_and_')
-      conditions = Array.new(args.size, '=')
-      h = RubyZoho.configuration.api.find_records(
-          Crm.module_name, ApiUtils.string_to_symbol(attrs[0]), conditions[0], args[0]
-      )
-      return h.collect { |r| new(r) } unless h.nil?
-      nil
+    def save
+      h = {}
+      @fields.each { |f| h.merge!({ f => eval("self.#{f.to_s}") }) }
+      h.delete_if { |k, v| v.nil? }
+      r = RubyZoho.configuration.api.add_record(Crm.module_name, h)
+      up_date(r)
     end
 
     def self.update(object_attribute_hash)
@@ -155,6 +156,25 @@ module RubyZoho
       RubyZoho.configuration.api.update_record(Crm.module_name, id, object_attribute_hash)
     end
 
+    def up_date(object_attribute_hash)
+      retry_counter = object_attribute_hash.length
+      begin
+        object_attribute_hash.map { |(k, v)| public_send("#{k}=", v) }
+      rescue NoMethodError => e
+        m = e.message.slice(/`(.*?)=/)
+        unless m.nil?
+          m.gsub!('`', '')
+          m.gsub!('(', '')
+          m.gsub!(')', '')
+          RubyZoho::Crm.create_accessor(self.class, [m.chop])
+        end
+        retry_counter -= 1
+        retry if retry_counter > 0
+      end
+      self
+    end
+
+
     def self.setup_classes
       RubyZoho.configuration.crm_modules.each do |module_name|
         klass_name = module_name.chop

data/lib/zoho_api.rb

@@ -2,6 +2,7 @@ $:.unshift File.join('..', File.dirname(__FILE__), 'lib')
 
 require 'httmultiparty'
 require 'rexml/document'
+require 'net/https'
 require 'net/http/post/multipart'
 require 'mime/types'
 require 'ruby_zoho'
@@ -35,30 +36,35 @@ module ZohoApi
       element = x.add_element module_name
       row = element.add_element 'row', { 'no' => '1'}
       fields_values_hash.each_pair { |k, v| add_field(row, ApiUtils.symbol_to_string(k), v) }
-      #pp x.to_s
       r = self.class.post(create_url(module_name, 'insertRecords'),
           :query => { :newFormat => 1, :authtoken => @auth_token,
                       :scope => 'crmapi', :xmlData => x },
           :headers => { 'Content-length' => '0' })
       check_for_errors(r)
+      x_r = REXML::Document.new(r.body).elements.to_a('//recorddetail')
+      to_hash_with_id(x_r, module_name)[0]
     end
 
     def add_field(row, field, value)
       r = (REXML::Element.new 'FL')
       r.attributes['val'] = field
-      r.add_text(value)
+      r.add_text(value.to_s)
       row.elements << r
       row
     end
 
     def attach_file(module_name, record_id, file_path)
-      r = self.class.post(create_url(module_name, 'uploadFile'),
-          :query => { :newFormat => 1, :authtoken => @auth_token,
-            :scope => 'crmapi',
-            :id => record_id, :content => File.open(file_path) },
-          :headers => { 'Content-length' => '0' })
-      raise(RuntimeError, 'Attaching file failed.', r.body.to_s) unless r.response.code == '200'
-      r.code
+      mime_type = (MIME::Types.type_for(file_path)[0] || MIME::Types["application/octet-stream"][0])
+      url = URI.parse(create_url(module_name, 'uploadFile') + "?authtoken=#{@auth_token}&scope=crmapi&id=#{record_id}")
+        File.open(file_path) do |file|
+          req = Net::HTTP::Post::Multipart.new url.path,
+              "file" => UploadIO.new(file, mime_type, File.basename(file_path))
+          http = Net::HTTP.new(url.host, url.port)
+          http.use_ssl = url.port == 443 if url.scheme == "https"
+          res = Net::HTTP.start(url.host, url.port) { |http| http.request(req) }
+        end
+      pp res
+      pp res.code
     end
 
     def add_file(module_name, record_id, file_path)
@@ -242,6 +248,16 @@ module ZohoApi
       r
     end
 
+    def to_hash_with_id(xml_results, module_name)
+      h = to_hash(xml_results)
+      primary_key = module_name.chop.downcase + 'id'
+      h.each do |e|
+        e.merge!({ primary_key.to_sym => e[:id] }) if e[primary_key.to_sym].nil? && !e[:id].nil?
+        e.merge!({ e[:id] => primary_key.to_sym }) if e[:id].nil? && !e[primary_key.to_sym].nil?
+      end
+      h
+    end
+
     def update_module_fields(mod_name, module_name, r)
       @@module_fields[mod_name] = []
       x = REXML::Document.new(r.body)

data/rubyzoho.gemspec

@@ -5,7 +5,7 @@
 
 Gem::Specification.new do |s|
   s.name = "rubyzoho"
-  s.version = "0.1.3"
+  s.version = "0.1.4"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["amalc"]

data/spec/ruby_zoho_spec.rb

@@ -14,7 +14,7 @@ describe RubyZoho::Crm do
       #config.api_key = 'e194b2951fb238e26bc096de9d0cf5f8'
       config.api_key = '62cedfe9427caef8afb9ea3b5bf68154'
       config.crm_modules = %w(Quotes)
-      config.cache_fields = false
+      config.cache_fields = true
     end
     #r = RubyZoho::Crm::Contact.find_by_last_name('Smithereens')
     #r.each { |m| RubyZoho::Crm::Contact.delete(m.contactid) } unless r.nil?
@@ -173,6 +173,19 @@ describe RubyZoho::Crm do
     r.each { |c|  RubyZoho::Crm::Lead.delete(c.leadid) }
   end
 
+  it 'should save and retrieve an account record with a custom field' do
+    accounts = RubyZoho::Crm::Account.all
+    a = accounts.first
+    if defined?(a.par_ltd)
+      RubyZoho::Crm::Lead.update(
+          :id => a.accountid,
+          :test_custom => '$1,000,000'
+      )
+      a2 = RubyZoho::Crm::Account.find_by_accountid(a.accountid)
+      a2.first.test_custom.should eq('$1,000,000')
+    end
+  end
+
   it 'should save and retrieve a potential record' do
     accounts = RubyZoho::Crm::Account.all
     h = {

data/spec/zoho_api_spec.rb

@@ -85,7 +85,7 @@ describe ZohoApi do
     pending
     @zoho.add_record('Contacts', @h_smith)
     contacts = @zoho.find_records('Contacts', :email, '=', @h_smith[:email])
-    @zoho.add_file('Contacts', contacts[0][:contactid], @sample_pdf)
+    @zoho.attach_file('Contacts', contacts[0][:contactid], @sample_pdf)
     #@zoho.delete_record('Contacts', contacts[0][:contactid])
   end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rubyzoho
 version: !ruby/object:Gem::Version
-  version: 0.1.3
+  version: 0.1.4
   prerelease: 
 platform: ruby
 authors: