mautic 0.1.3 → 0.1.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 11db7c9607e84c68c33c020711014b1f6c858073
-  data.tar.gz: 3bdca3d4ab03eb07970fbdb7d14ac06133694093
+  metadata.gz: c77ac23dffbdf501bf3193b5da058a6664aa887a
+  data.tar.gz: 5e70349a9999dae0f43725e6b2b0028eec68f858
 SHA512:
-  metadata.gz: 47d1a229897761802e43a7fc317ee01bf6b2bdce1ad6bda948e5432fd52a6f8c122e9df5c3b04d606012bb276cac2b8f8d3c99415c91786da0e09c328fd1af62
-  data.tar.gz: b4e767ec3ede605578d385af2f5091ef8803ac53d6196476a9c9f33c72a1c61024146055138089505fc03e6a523dd3049f5942bbf0f08767a702cda077996d69
+  metadata.gz: 403f7b5fefaffc8efd92088b916342051750d234a011d7f57e6bba1fa5a37d500f04cb2467146170e06ed34c6535e2bfad6cd93b7f6ef00641d17be7c0d4c2dc
+  data.tar.gz: 91744d8458f445803d0b43021183e1cd9339c739bce7b7382e13edfd906270f96b3dc4dd01499486db02a1b9e571d623900de5e1483d9c01364be2b3d1b227a5

data/README.md

@@ -6,6 +6,13 @@ RoR helper / wrapper for Mautic API and forms
 ### Gem provides API connection to your Mautic(s)
   1. Create mautic connection
   2. Authorize it
+      
+      In mautic you need add API oauth2 login.
+      For URI callback allow:
+      ```
+      http://localhost:3000/mautic/connections/:ID/oauth2
+      ```
+      ID = is your Mautic::Connection ID
   
    Find connection which you want to use:
   ```ruby

data/app/models/mautic/connection.rb

@@ -39,9 +39,40 @@ module Mautic
     end
 
     def request(type, path, params = {})
-      raise NotImplementedError
+      @last_request = [type, path, params]
+      response = raise NotImplementedError
+      parse_response(response)
     end
 
+    private
+
+    def parse_response(response)
+      case response.status
+      when 400
+        raise Mautic::ValidationError.new(response)
+      when 404
+        raise Mautic::RecordNotFound.new(response)
+      when 200, 201
+        json = JSON.parse(response.body) rescue {}
+        Array(json['errors']).each do |error|
+          case error['code'].to_i
+          when 401
+            raise Mautic::TokenExpiredError.new(response) if @try_to_refresh
+            @try_to_refresh = true
+            refresh!
+            json = request(*@last_request)
+          when 404
+            raise Mautic::RecordNotFound.new(response)
+          else
+            raise Mautic::RequestError.new(response)
+          end
+        end
+      else
+        raise Mautic::RequestError.new(response)
+      end
+
+      json
+    end
 
   end
 end

data/app/models/mautic/connections/oauth2.rb

@@ -30,21 +30,9 @@ module Mautic
       end
 
       def request(type, path, params = {})
-        json = JSON.parse connection.request(type, path, params).body
-        Array(json['errors']).each do |error|
-          case error['code']
-          when 400
-            # Validation error
-          when 401
-            raise Mautic::TokenExpiredError.new(error['message']) if @try_to_refresh
-            @try_to_refresh = true
-            refresh!
-            json = request(type, path, params)
-          else
-            raise Mautic::AuthorizeError.new("#{error['code']} - #{error['message']}")
-          end
-        end
-        json
+        @last_request = [type, path, params]
+        response = connection.request(type, path, params)
+        parse_response(response)
       end
 
       private

data/app/models/mautic/contact.rb

@@ -1,6 +1,8 @@
 module Mautic
   class Contact < Model
 
+    alias_attribute :first_name, :firstname
+    alias_attribute :last_name, :lastname
     def self.in(connection)
       Proxy.new(connection, endpoint, default_params: { search: '!is:anonymous' })
     end

data/lib/mautic.rb

@@ -9,14 +9,46 @@ module Mautic
   autoload :Proxy, 'mautic/proxy'
   autoload :Model, 'mautic/model'
 
-  class TokenExpiredError < StandardError
+  class RequestError < StandardError
+
+    attr_reader :response, :errors
+
+    def initialize(response, message = nil)
+      @errors ||= []
+      @response = response
+      json_body = JSON.parse(response.body) rescue {}
+      message ||= Array(json_body['errors']).collect do |error|
+        msg = error['code'].to_s
+        msg << " (#{error['type']}):" if error['type']
+        msg << " #{error['message']}"
+        @errors << error['message']
+        msg
+      end.join(', ')
+
+      super(message)
+    end
+
   end
 
-  class ValidationError < StandardError
+  class TokenExpiredError < RequestError
+  end
+
+  class ValidationError < RequestError
+
+    def initialize(response, message = nil)
+      @response = response
+      json_body = JSON.parse(response.body) rescue {}
+      @errors = Array(json_body['errors']).inject({}) { |mem, var| mem.merge!(var['details']); mem }
+      message ||= @errors.collect { |field, msg| "#{field}: #{msg.join(', ')}" }.join('; ')
+      super(response, message)
+    end
+
+  end
 
+  class AuthorizeError < RequestError
   end
 
-  class AuthorizeError < StandardError
+  class RecordNotFound < RequestError
   end
 
   configure do |config|

data/lib/mautic/model.rb

@@ -32,6 +32,7 @@ module Mautic
       @table = MauticHash.new
       self.attributes = { created_at: hash['dateAdded']&.to_time, updated_at: hash['dateModified']&.to_time } if hash
       assign_attributes(hash)
+      clear_changes
     end
 
     def save(force = false)
@@ -40,29 +41,37 @@ module Mautic
 
     def update(force = false)
       return false if changes.blank?
-      json = @connection.request((force && :put || :patch), "api/#{endpoint}/#{id}/edit", { body: to_h })
-      if json['errors']
-        self.errors = json['errors']
-      else
+      begin
+        json = @connection.request((force && :put || :patch), "api/#{endpoint}/#{id}/edit", { body: to_h })
         self.attributes = json[endpoint.singularize]
+        clear_changes
+      rescue ValidationError => e
+        self.errors = e.errors
       end
-      json['errors'].blank?
+
+      self.errors.blank?
     end
 
     def create
-      json = @connection.request(:post, "api/#{endpoint}/#{id}/new", { body: to_h })
-      if json['errors']
-        self.errors = json['errors']
-      else
+      begin
+        json = @connection.request(:post, "api/#{endpoint}/#{id}/new", { body: to_h })
         self.attributes = json[endpoint.singularize]
+        clear_changes
+      rescue ValidationError => e
+        self.errors = e.errors
       end
-      json['errors'].blank?
+
+      self.errors.blank?
     end
 
     def destroy
-      json = @connection.request(:delete, "api/#{endpoint}/#{id}/delete")
-      self.errors = json['errors'] if json['errors']
-      json['errors'].blank?
+      begin
+        @connection.request(:delete, "api/#{endpoint}/#{id}/delete")
+        true
+      rescue RequestError => e
+        self.errors = e.errors
+        false
+      end
     end
 
     def changes
@@ -73,20 +82,23 @@ module Mautic
       @table.to_h
     end
 
-    private
-
-    def endpoint
-      self.class.endpoint
-    end
-
     def attributes=(hash)
       hash.each_pair do |k, v|
         k = k.to_sym
         @table[k] = v
       end
+    end
+
+    private
+
+    def clear_changes
       @table.instance_variable_set(:@changes, nil)
     end
 
+    def endpoint
+      self.class.endpoint
+    end
+
     def assign_attributes(source = nil)
       source ||= {}
       data = {}

data/lib/mautic/version.rb

@@ -1,3 +1,3 @@
 module Mautic
-  VERSION = '0.1.3'
+  VERSION = '0.1.4'
 end

data/spec/rails_helper.rb

@@ -1,4 +1,7 @@
 ENV['RAILS_ENV'] ||= 'test'
+require 'simplecov'
+SimpleCov.start
+
 require File.expand_path("../dummy/config/environment.rb", __FILE__)
 # Prevent database truncation if the environment is production
 abort('The Rails environment is running in production mode!') if Rails.env.production?

data/spec/spec_helper.rb

@@ -3,6 +3,7 @@ require 'factory_bot_rails'
 require 'database_cleaner'
 require 'faker'
 require 'webmock/rspec'
+require 'pry-rails'
 # require 'capybara/rspec'
 
 ActiveRecord::Migration.maintain_test_schema!
@@ -34,10 +35,6 @@ RSpec.configure do |config|
   end
 
   config.include FactoryBot::Syntax::Methods
-  # DatabaseCleaner.clean_with(
-  #   :truncation,
-  #   except: %w(ar_internal_metadata)
-  # )
 
   config.before(:suite) do
     DatabaseCleaner.clean_with(:truncation, except: %w(ar_internal_metadata))
@@ -47,10 +44,6 @@ RSpec.configure do |config|
     DatabaseCleaner.strategy = :transaction
   end
 
-  config.before(:each, js: true) do
-    DatabaseCleaner.strategy = :truncation
-  end
-
   config.before(:each) do
     DatabaseCleaner.start
   end
@@ -65,21 +58,4 @@ RSpec.configure do |config|
 
   config.filter_rails_from_backtrace!
 
-  # config.before(:each) do |ex|
-  #   meta = ex.metadata
-  #   unless meta[:null]
-  #     case meta[:logged]
-  #     when :admin
-  #       logged(Symphonia::User, FactoryBot.create(:admin_user))
-  #     when true
-  #       logged(Symphonia::User, FactoryBot.create(:user))
-  #     end
-  #   end
-  # end
-
-end
-
-
-def logged(model, user)
-  allow(model).to receive(:current).and_return(user)
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: mautic
 version: !ruby/object:Gem::Version
-  version: 0.1.3
+  version: 0.1.4
 platform: ruby
 authors:
 - Lukáš Pokorný
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-11-22 00:00:00.000000000 Z
+date: 2017-12-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails