economic-rest 0.5.7 → 0.5.8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 20f67e5c9705573a15158f9d1b584f84cf5b16024fbc4b199ae20193c2885df2
-  data.tar.gz: 1975cf48bfc94353e99c50be229707e997da7d069fd5e1a7a178796044535a23
+  metadata.gz: 76c90f66b43ea548df326ecb709ff8f88a6c2f3851c3614a140ae9c57f4e9aaa
+  data.tar.gz: b5c43c292e4a5ebc3fe6ec2b3d2ce0f0ee3bd8c9095837afbd60254bcb183412
 SHA512:
-  metadata.gz: d6b9d5f0136b6b89c8e4e322c6af72daf4a660d2db18021535f26ad5e3a897ca96f99ef409047a608b132f76f1eb2f7fe4b94bc89a707a9371a838db82a545b2
-  data.tar.gz: 074e3535e62b3220ad0a96d8cc070767565c67b31e852b8cce897514ae5d44899a8687a4bf2e673dd08a088d32a8fcff523f1041347e49b548e2bb6c0023d8d7
+  metadata.gz: afe07c891403a05e3efbd671e671489dd193c3a95ad2fd64a6d89df224519f4eca961bed2cea103ad3ae68313d385d6d7e707b728b66776dc89b484d228013de
+  data.tar.gz: fc7ffafe19d596a6fc8cca97b83d71e14b671fa2f4172c94e3ad3ed9eafbfbac043dc1c82f090a86d95cda0b543555427bd0fdf1c415877b8995829bcf77cb32

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    economic-rest (0.5.6)
+    economic-rest (0.5.8)
       activesupport
       rest-client
       savon

data/lib/economic/base.rb

@@ -29,12 +29,8 @@ module Economic
     def self.relation(name, fields:, multiple: false)
       economic_cased_attibute_name = name.to_s
       add_relation economic_cased_attibute_name, fields, multiple
-      if multiple
-        attr_accessor economic_cased_attibute_name
-        alias_method "#{snake_case(economic_cased_attibute_name)}=", "#{economic_cased_attibute_name}="
-      else
-        attr_reader economic_cased_attibute_name
-      end
+      attr_accessor economic_cased_attibute_name
+      alias_method "#{snake_case(economic_cased_attibute_name)}=", "#{economic_cased_attibute_name}="
       alias_method snake_case(economic_cased_attibute_name), economic_cased_attibute_name
     end
 
@@ -94,13 +90,6 @@ module Economic
       return_hash
     end
 
-    def dirty?
-      self.class.attributes.each do |attribute|
-        return true unless send(attribute) == @internal_hash[attribute]
-      end
-      false
-    end
-
     def self.snake_case(camel_cased)
       camel_cased.to_s.gsub(/::/, "/")
         .gsub(/([A-Z]+)([A-Z][a-z])/, '\1_\2')

data/lib/economic/base_repo.rb

@@ -10,19 +10,21 @@ module Economic
       attr_accessor :endpoint
 
       def save(model, url: endpoint_url)
-        post_or_put = model.id_key.nil? ? :post : :put
-        url += "/" + id_to_url_formatted_id(model.id_key)
+        if model.id_key.nil?
+          post_or_put = :post
+        else
+          post_or_put = :put
+          url += "/" + id_to_url_formatted_id(model.id_key)
+        end
 
         response = send_request(method: post_or_put, url: url, payload: model.to_h.to_json)
 
         modelize_response(response)
       end
 
-      # TODO: This method does not seem to do anything that the save method cannot do - is there any reason to keep it? Posting to a not-existing id is apparenly fine
       def send(model, url: endpoint_url)
-        response = send_request(method: :post, url: url, payload: model.to_h.to_json)
-
-        modelize_response(response)
+        warn "use #{self}.save().  #{self}.send() is deprecated"
+        save(model, url: endpoint_url)
       end
 
       def all(filter_text: "", url: endpoint_url)

data/lib/economic/invoices/repo.rb

@@ -1,15 +1,23 @@
 module Economic
   module Invoices
     class Repo < Economic::BaseRepo
-      def self.all
-        invoices = super
-        invoices.each do |invoice|
-          invoice.remove_instance_variable("@lines")
-          class << invoice
-            define_method(:lines) { raise NoMethodError }
+      class << self
+        def all
+          invoices = super
+          invoices.each do |invoice|
+            invoice.remove_instance_variable("@lines")
+            class << invoice
+              define_method(:lines) { raise NoMethodError }
+            end
           end
+          invoices
+        end
+
+        def send(model, url: endpoint_url)
+          response = send_request(method: :post, url: url, payload: model.to_h.to_json)
+
+          modelize_response(response)
         end
-        invoices
       end
     end
   end

data/lib/economic/nested_base_repo.rb

@@ -18,7 +18,8 @@ module Economic
       end
 
       def send(model, on:)
-        super(model, url: nested_endpoint_url(on))
+        warn "use #{self}.save().  #{self}.send() is deprecated"
+        save(model, on: on)
       end
 
       def save(model, on:)

data/lib/economic/orders/repo.rb

@@ -1,15 +1,23 @@
 module Economic
   module Orders
     class Repo < Economic::BaseRepo
-      def self.all
-        orders = super
-        orders.each do |order|
-          order.remove_instance_variable("@lines")
-          class << order
-            define_method(:lines) { raise NoMethodError }
+      class << self
+        def all
+          orders = super
+          orders.each do |order|
+            order.remove_instance_variable("@lines")
+            class << order
+              define_method(:lines) { raise NoMethodError }
+            end
           end
+          orders
+        end
+
+        def send(model, url: endpoint_url)
+          response = send_request(method: :post, url: url, payload: model.to_h.to_json)
+
+          modelize_response(response)
         end
-        orders
       end
     end
   end

data/lib/economic/rest/version.rb

@@ -1,5 +1,5 @@
 module Economic
   module Rest
-    VERSION = "0.5.7".freeze
+    VERSION = "0.5.8".freeze
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: economic-rest
 version: !ruby/object:Gem::Version
-  version: 0.5.7
+  version: 0.5.8
 platform: ruby
 authors:
 - Peter Klogborg
@@ -317,8 +317,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.7.6
+rubygems_version: 3.0.3
 signing_key: 
 specification_version: 4
 summary: Ruby wrapper for the e-conomic REST API, that aims at making working with