zammad_api 1.1.0 → 1.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7bd69b72c531ea884a30b28ecc8f1ef270f24c1f4e1be49a880fc64c376b93db
-  data.tar.gz: 88ef7a8c03356274b8c3af28d5ded91e6d58fab2ccafe8085f1b9eb4915a4987
+  metadata.gz: 42c75fd97c167f27bec8282ccd8214f3885e051691c042ad0e8e491651814d81
+  data.tar.gz: 949a756e4310f53f9ca929a9e973dcb30af4a80e494e55059da6cac345a8bc28
 SHA512:
-  metadata.gz: b09bfa14c68b10b898ebdb71d1e7a7d013c8d756d5214e07f7c913100efdc2e296bbcec1732eea99237b2873f9fb44d868e8c6acfbf7b97a4e457eaf6bbe993c
-  data.tar.gz: 016e5665568b8cfbf970575b0750d5b6468c39eaf91a27d5a967d41ab9905a634bf53e19324a5ad3facb86e2f1fe46c8f3ab355c1ef0cc220cbeb65fb71f71c5
+  metadata.gz: b6625e7abed25c87ae3fe46bf543c9095cc092f61eacc23fded2af1a9807a0fc8ebf589221d6afbb57414704e3c732e9f629f89bf460791cffecd91448398a38
+  data.tar.gz: '0379ee29e881afe691eb7d47b0c2c3dbe01ae0a6eddd73e783d24ec8c07fc0dd635dca03148c9a539c006e5b2ad6c87173b944e5511024849727ffcff58be454'

data/lib/zammad_api/client.rb

@@ -6,7 +6,6 @@ require 'zammad_api/dispatcher'
 require 'zammad_api/resources'
 
 module ZammadAPI
-
   class Client
     extend Forwardable
 
@@ -27,7 +26,7 @@ module ZammadAPI
     end
 
     def method_missing(method, *_args)
-      method     = modulize( method.to_s )
+      method     = modulize(method.to_s)
       class_name = "ZammadAPI::Resources::#{method}"
       begin
         class_object = Kernel.const_get(class_name)
@@ -41,7 +40,7 @@ module ZammadAPI
 
     def check_config
       raise 'missing url in config' if !@config[:url]
-      raise 'config url needs to start with http:// or https://' if @config[:url] !~ %r{^(http|https)://}
+      raise 'config url needs to start with http:// or https://' if !%r{^(http|https)://}.match?(@config[:url])
 
       # check for token auth
       return if @config[:http_token] && !@config[:http_token].empty?
@@ -58,9 +57,9 @@ module ZammadAPI
 
     def modulize(string)
       string.gsub(/__(.?)/) { "::#{$1.upcase}" }
-            .gsub(%r{/(.?)}) { "::#{$1.upcase}" }
-            .gsub(/(?:_+|-+)([a-z])/) { $1.upcase }
-            .gsub(/(\A|\s)([a-z])/) { $1 + $2.upcase }
+        .gsub(%r{/(.?)}) { "::#{$1.upcase}" }
+        .gsub(/(?:_+|-+)([a-z])/) { $1.upcase }
+        .gsub(/(\A|\s)([a-z])/) { $1 + $2.upcase }
     end
   end
 end

data/lib/zammad_api/dispatcher.rb

@@ -1,6 +1,5 @@
 module ZammadAPI
   class Dispatcher
-
     def initialize(transport, resource)
       @transport = transport
       @resource  = resource

data/lib/zammad_api/list_all.rb

@@ -2,7 +2,6 @@ require 'zammad_api/list_base'
 
 module ZammadAPI
   class ListAll < ListBase
-
     private
 
     def perform_request(parameter)

data/lib/zammad_api/list_base.rb

@@ -14,7 +14,6 @@ module ZammadAPI
     end
 
     def [](position)
-
       local_parameter = @parameter.merge(
         page:     position + 1,
         per_page: 1
@@ -23,7 +22,6 @@ module ZammadAPI
     end
 
     def page(page, per_page, &block)
-
       @parameter[:page]     = page
       @parameter[:per_page] = per_page
       fetch_and_yield_each(&block)
@@ -45,18 +43,14 @@ module ZammadAPI
 
     private
 
-    def fetch_and_yield_each
+    def fetch_and_yield_each(&block)
       result = perform_request(@parameter)
-      result.each { |item|
-        yield item
-      }
+      result.each(&block)
     end
 
     def request(request, url, parameter)
-
       # convert parameters into a GET query
       url += '?' + parameter.map { |key, value|
-
         if !value.is_a? String
           value = value.to_s
         end
@@ -71,11 +65,11 @@ module ZammadAPI
       end
 
       list = []
-      data.each { |local_data|
+      data.each do |local_data|
         item = @resource.new(@transport, local_data)
         item.new_instance = false
         list.push item
-      }
+      end
       list
     end
 

data/lib/zammad_api/list_search.rb

@@ -2,7 +2,6 @@ require 'zammad_api/list_base'
 
 module ZammadAPI
   class ListSearch < ListBase
-
     private
 
     def perform_request(parameter)

data/lib/zammad_api/log.rb

@@ -1,8 +1,8 @@
 module ZammadAPI
   class Log
-
     def initialize(config)
       return if !config[:logger]
+
       require 'logger'
       @logger = Logger.new($stderr)
       #@logger.level = Logger::WARN
@@ -11,6 +11,7 @@ module ZammadAPI
 
     def method_missing(method, *args)
       return if !@logger
+
       @logger.send(method, args)
     end
   end

data/lib/zammad_api/resources/base.rb

@@ -23,6 +23,7 @@ module ZammadAPI
 
       def method_missing(method, *args)
         return @attributes[method] if !method.to_s.end_with?('=')
+
         method              = method.to_s[0, method.length - 1].to_sym
         @changes[method]    = [@attributes[method], args[0]]
         @attributes[method] = args[0]
@@ -43,6 +44,7 @@ module ZammadAPI
           data = JSON.parse(response.body)
         end
         return true if response.status == 200
+
         raise "Can't destroy object (#{self.class.name}): #{data['error']}"
       end
 
@@ -78,6 +80,7 @@ module ZammadAPI
         if response.status != 200
           raise "Can't find object (#{self.class.name}): #{data['error']}"
         end
+
         item = new(transport, data)
         item.new_instance = false
         item
@@ -99,6 +102,7 @@ module ZammadAPI
 
       def saved_attributes
         return save_new if @new_instance
+
         save_existing
       end
 
@@ -106,18 +110,20 @@ module ZammadAPI
         response   = @transport.post(url: "#{@url}?expand=true", params: @attributes)
         attributes = JSON.parse(response.body)
         return attributes if response.status == 201
+
         save_error(attributes)
       end
 
       def save_existing
         attributes_to_post = {}
-        @changes.each { |name, values|
+        @changes.each do |name, values|
           attributes_to_post[name] = values[1]
-        }
+        end
         response   = @transport.put(url: "#{@url}/#{@attributes[:id]}?expand=true", params: attributes_to_post)
         attributes = JSON.parse(response.body)
 
         return attributes if response.status == 200
+
         save_error(attributes)
       end
 

data/lib/zammad_api/resources/ticket.rb

@@ -8,11 +8,11 @@ class ZammadAPI::Resources::Ticket < ZammadAPI::Resources::Base
       raise "Can't get articles (#{self.class.name}): #{data['error']}"
     end
 
-    data.collect { |raw|
+    data.collect do |raw|
       item = ZammadAPI::Resources::TicketArticle.new(@transport, raw)
       item.new_instance = false
       item
-    }
+    end
   end
 
   def article(data)
@@ -21,5 +21,4 @@ class ZammadAPI::Resources::Ticket < ZammadAPI::Resources::Base
     item.save
     item
   end
-
 end

data/lib/zammad_api/resources/ticket_article.rb

@@ -2,11 +2,10 @@ class ZammadAPI::Resources::TicketArticle < ZammadAPI::Resources::Base
   url '/api/v1/ticket_articles'
 
   def attachments
-    @attributes[:attachments].collect { |raw|
+    @attributes[:attachments].collect do |raw|
       raw[:ticket_id]  = @attributes[:ticket_id]
       raw[:article_id] = @attributes[:id]
       ZammadAPI::Resources::TicketArticleAttachment.new(@transport, raw)
-    }
+    end
   end
-
 end

data/lib/zammad_api/resources/ticket_article_attachment.rb

@@ -1,6 +1,5 @@
 class ZammadAPI::Resources::TicketArticleAttachment < ZammadAPI::Resources::Base
-
-  def initialize(transport, attributes = {})
+  def initialize(transport, attributes = {}) # rubocop:disable Lint/MissingSuper
     @transport  = transport
     @attributes = attributes
     symbolize_keys_deep!(@attributes)
@@ -13,8 +12,8 @@ class ZammadAPI::Resources::TicketArticleAttachment < ZammadAPI::Resources::Base
   def download
     response = @transport.get(url: "/api/v1/ticket_attachment/#{ticket_id}/#{article_id}/#{id}")
     return response.body if response.status == 200
+
     data = JSON.parse(response.body)
     raise "Can't get articles (#{self.class.name}): #{data['error']}"
   end
-
 end

data/lib/zammad_api/resources.rb

@@ -12,6 +12,6 @@ require 'zammad_api/resources/ticket_state'
 require 'zammad_api/resources/ticket_priority'
 
 module ZammadAPI
-  class Resource
+  class Resource # rubocop:disable Lint/EmptyClass
   end
 end

data/lib/zammad_api/transport.rb

@@ -3,7 +3,6 @@ require 'openssl'
 
 module ZammadAPI
   class Transport
-
     attr_accessor :url, :user, :password, :on_behalf_of
 
     def initialize(config, logger)
@@ -12,7 +11,7 @@ module ZammadAPI
       @conn = Faraday.new(url: config[:url]) do |faraday|
         #faraday.request  :url_encoded             # form-encode POST params
         #faraday.response :logger                  # log requests to STDOUT
-        faraday.adapter Faraday.default_adapter  # make requests with Net::HTTP
+        faraday.adapter Faraday.default_adapter # make requests with Net::HTTP
       end
       @conn.headers[:user_agent] = 'Zammad API Ruby'
       if config[:http_token] && !config[:http_token].empty?
@@ -20,22 +19,21 @@ module ZammadAPI
       elsif config[:oauth2_token] && !config[:oauth2_token].empty?
         @conn.request :authorization, 'Bearer', config[:oauth2_token]
       else
-        @conn.request :basic_auth, config[:user], config[:password]
+        @conn.request :authorization, :basic, config[:user], config[:password]
       end
     end
 
     %w[get post put delete].each do |method|
       class_eval <<-RUBY, __FILE__, __LINE__ + 1
-        def #{method}(params)
-          run_request(:#{method}, params)
-        end
+        def #{method}(params)               # def get(params)
+          run_request(:#{method}, params)   #   run_request(:get, params)
+        end                                 # end
       RUBY
     end
 
     private
 
     def run_request(verb, param)
-
       @logger.debug "#{verb.to_s.upcase}: #{@url}#{param[:url]}"
 
       with_params = !param[:params].nil?

data/lib/zammad_api/version.rb

@@ -1,3 +1,3 @@
 module ZammadAPI
-  VERSION = '1.1.0'.freeze
+  VERSION = '1.3.0'.freeze
 end

metadata

@@ -1,15 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: zammad_api
 version: !ruby/object:Gem::Version
-  version: 1.1.0
+  version: 1.3.0
 platform: ruby
 authors:
 - Martin Edenhofer
 - Martin Gruner
-autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-05-11 00:00:00.000000000 Z
+date: 1980-01-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: faraday
@@ -17,70 +16,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1'
+        version: '2'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1'
-- !ruby/object:Gem::Dependency
-  name: bundler
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: 2.2.10
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: 2.2.10
-- !ruby/object:Gem::Dependency
-  name: rake
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '13'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '13'
-- !ruby/object:Gem::Dependency
-  name: rspec
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '3.0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '3.0'
-- !ruby/object:Gem::Dependency
-  name: webmock
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
+        version: '2'
 description: Ruby wrapper for the Zammad API v1.0.
 email:
 - support@zammad.org
@@ -116,7 +59,7 @@ metadata:
   homepage_uri: https://github.com/zammad/zammad-api-client-ruby
   source_code_uri: https://github.com/zammad/zammad-api-client-ruby
   changelog_uri: https://github.com/zammad/zammad-api-client-ruby/blob/master/CHANGELOG.md
-post_install_message:
+  rubygems_mfa_required: 'true'
 rdoc_options: []
 require_paths:
 - lib
@@ -124,15 +67,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: '3.0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.3.26
-signing_key:
+rubygems_version: 3.6.9
 specification_version: 4
 summary: Zammad API v1.0 client.
 test_files: []