ecoportal-api-v2 0.9.7 → 1.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: cd61279f012c243d8ac7dc7168273fb76ad2555871f464c17c38a2a54147aea8
-  data.tar.gz: a7837403a1df3d5e386d9fd71a24cb29eb98c43ea12637c9a4dd19301ac13bec
+  metadata.gz: a951381010ef926bce6a42eceb729ed3b8adfe8344ee75161e060e741675f405
+  data.tar.gz: 8808bef7b78d1b333de9af88d9e2eac68511213ce0312efadc79d25ef520ff8a
 SHA512:
-  metadata.gz: bb9cdb7833b7eae455274cd89348e395c01bd423d8d9591c62b1f66a129b8e3ef9a13f6301870cf75b4707be6306e4ee83299b6bcc9ce05d3224741d44141019
-  data.tar.gz: 9009e8341f147df8610777d99026f6a2107b2fe64d5e0e0c5fa0b7623d632982899e00728a02c6a737395a6755a40f19a54e805b86ab6648cef51e46e3c3c9a4
+  metadata.gz: 75fea111bdc18536084d87d9899a4185a0f8573d3aa0567a355263bb39855880663d5af3f1443c72c6e1d2dd24b1c0420aa5a9777ddd0345209debec4334af9b
+  data.tar.gz: 42e2db2fcaca353e334b892267fd01df6702708ff79f3ed44474d79dc26d879532af761c7da78cfcb1c1b1abb313c033e80f7da1d21385f02195a88bd800d47c

data/CHANGELOG.md

@@ -1,12 +1,31 @@
 # Change Log
 All notable changes to this project will be documented in this file.
 
-## [0.9.8] - 2022-11-xx
+## [1.1.2] - 2023-03-xx
 
 ### Added
 ### Changed
 ### Fixed
 
+## [1.1.1] - 2023-03-09
+
+### Added
+### Changed
+  - upgraded core gem `ecoportal-api`
+
+### Fixed
+
+## [1.0.1] - 2023-02-24
+
+### Added
+  - Allowed `read_only` mode to support `emdeds_many` with item class without `method_key`
+    - `Ecoportal::API::Common::Content::DoubleModel`
+    - `Ecoportal::API::Common::Content::CollectionModel`
+    - `Ecoportal::API::Common::Content::ArrayModel`
+
+### Changed
+  - It is a risky change (so treated as a **braking change**)
+
 ## [0.9.7] - 2022-11-29
 
 ### Added

data/ecoportal-api-v2.gemspec

@@ -29,5 +29,5 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency "redcarpet", ">= 3.5.1",  "< 3.6"
   spec.add_development_dependency "pry"      , ">= 0.14"
 
-  spec.add_dependency 'ecoportal-api', '>= 0.8.5', '< 0.9'
+  spec.add_dependency 'ecoportal-api', '>= 0.9.1', '< 0.10'
 end

data/lib/ecoportal/api/common/content/array_model.rb

@@ -34,8 +34,8 @@ module Ecoportal
 
           inheritable_class_vars :order_matteres, :uniq
 
-          def initialize(doc = [], parent: self, key: nil)
-            super(doc, parent: parent, key: key)
+          def initialize(doc = [], parent: self, key: nil, read_only: false)
+            super(doc, parent: parent, key: key, read_only: read_only)
           end
 
           def order_matters?; self.class.order_matters; end

data/lib/ecoportal/api/common/content/collection_model.rb

@@ -52,7 +52,7 @@ module Ecoportal
             # @yieldparam doc [Hash]
             # @yieldreturn [Klass] instance object of the target `klass`
             # @return [Klass] instance object of the target `klass`
-            def new_item(doc = NOT_USED, parent: nil, key: nil, &block)
+            def new_item(doc = NOT_USED, parent: nil, key: nil, read_only: false, &block)
               if block
                 @new_item = block
               elsif used_param?(doc)
@@ -61,7 +61,7 @@ module Ecoportal
                   @new_item.call(doc, parent, key)
                 else
                   if target_class = self.klass(doc)
-                    doc.is_a?(target_class) ? doc : target_class.new(doc, parent: parent, key: key)
+                    doc.is_a?(target_class) ? doc : target_class.new(doc, parent: parent, key: key, read_only: read_only)
                   else
                     raise "Could not find a class for: #{doc}"
                   end
@@ -88,7 +88,7 @@ module Ecoportal
 
           inheritable_class_vars :klass, :order_matters, :order_key, :items_key, :new_item
 
-          def initialize(ini_doc = [], parent: self, key: nil)
+          def initialize(ini_doc = [], parent: self, key: nil, read_only: false)
             unless self.class.klass?
               raise "Undefined base 'klass' or 'new_item' callback for #{self.class}"
             end
@@ -102,7 +102,7 @@ module Ecoportal
                         []
                       end
 
-            super(ini_doc, parent: parent, key: key)
+            super(ini_doc, parent: parent, key: key, read_only: read_only)
           end
 
           # @return [Class] the class of the elements of the Collection
@@ -256,7 +256,7 @@ module Ecoportal
           private
 
           def new_item(value)
-            self.class.new_item(value, parent: self)
+            self.class.new_item(value, parent: self, read_only: self._read_only)
           end
 
           # Helper to remove tracked down instance variables

data/lib/ecoportal/api/common/content/double_model.rb

@@ -18,6 +18,9 @@ module Ecoportal
             end
           end
 
+          class NoKeyMethod < StandardError
+          end
+
           class << self
             attr_reader :key
 
@@ -171,7 +174,7 @@ module Ecoportal
 
                 define_method method do
                   return instance_variable_get(var) if instance_variable_defined?(var)
-                  new_obj = dim_class.new(parent: self, key: method)
+                  new_obj = dim_class.new(parent: self, key: method, read_only: self._read_only)
                   variable_set(var, new_obj)
                 end
               end
@@ -194,7 +197,10 @@ module Ecoportal
             # @param order_matters [Boolean] to state if the order will matter
             # @param klass [Class, String] the class of the individual elements it embeds
             # @param enum_class [Class, String] the class of the collection that will hold the individual elements
-            def embeds_many(method, key: method, order_matters: false, order_key: nil, klass: nil, enum_class: nil)
+            # @param read_only [Boolean] whether or not should try to **work around** items `klass` missing a `key`
+            #   - If set to `true` this is meant only for read purposes (won't be able to successufully insert)
+            def embeds_many(method, key: method, klass: nil, enum_class: nil,
+              order_matters: false, order_key: nil, read_only: false)
               if enum_class
                 eclass = enum_class
               elsif klass
@@ -206,7 +212,7 @@ module Ecoportal
               else
                 raise "You should either specify the 'klass' of the elements or the 'enum_class'"
               end
-              embed(method, key: key, multiple: true, klass: eclass) do |instance_with_called_method|
+              embed(method, key: key, multiple: true, klass: eclass, read_only: read_only) do |instance_with_called_method|
                 # keep reference to the original class to resolve the `klass` dependency
                 # See stackoverflow: https://stackoverflow.com/a/73709529/4352306
                 referrer_class = instance_with_called_method.class
@@ -216,7 +222,7 @@ module Ecoportal
 
             private
 
-            def embed(method, key: method, nullable: false, multiple: false, klass:, &block)
+            def embed(method, key: method, nullable: false, multiple: false, klass:, read_only: false, &block)
               method = method.to_s.freeze
               var    = instance_variable_name(method).freeze
               k      = key.to_s.freeze
@@ -230,9 +236,22 @@ module Ecoportal
                 end
                 return variable_set(var, nil) unless doc[k]
 
-                self.class.resolve_class(klass).new(
-                  doc[k], parent: self, key: k
-                ).tap {|obj| variable_set(var, obj)}
+                embedded_class = self.class.resolve_class(klass)
+
+                if multiple && read_only
+                  if doc[k].is_a?(Array) && embedded_class < Common::Content::CollectionModel
+                    if (item_class = embedded_class.klass) && !item_class.key?
+                      item_class.passkey :id
+                      doc[k].each_with_index do |item_doc, i|
+                        item_doc["id"] = "#{i}" unless item_doc.key?("id")
+                      end
+                    end
+                  end
+                end
+
+                embedded_class.new(doc[k], parent: self, key: k, read_only: self._read_only || read_only).tap do |obj|
+                  variable_set(var, obj)
+                end
               end
             end
 
@@ -244,16 +263,17 @@ module Ecoportal
 
           inheritable_class_vars :forced_model_keys, :key
 
-          attr_reader :_parent, :_key
+          attr_reader :_parent, :_key, :_read_only
 
-          def initialize(doc = {}, parent: self, key: nil)
-            @_dim_vars = []
-            @_parent   = parent || self
-            @_key      = key    || self
+          def initialize(doc = {}, parent: self, key: nil, read_only: false)
+            @_dim_vars  = []
+            @_parent    = parent || self
+            @_key       = key    || self
+            @_read_only = read_only
 
             self.class.enforce!(doc)
 
-            if _parent == self
+            if (_parent == self) || read_only
               @doc          = doc
               @original_doc = JSON.parse(@doc.to_json)
             end
@@ -271,13 +291,13 @@ module Ecoportal
 
           # @return [String] the `value` of the `key` method (i.e. `id` value)
           def key
-            raise "No key_method defined for #{self.class}" unless key_method?
+            raise NoKeyMethod.new "No key_method defined for #{self.class}" unless key_method?
             self.method(key_method).call
           end
 
           # @param [String] the `value` of the `key` method (i.e. `id` value)
           def key=(value)
-            raise "No key_method defined for #{self.class}" unless key_method?
+            raise NoKeyMethod.new "No key_method defined for #{self.class}" unless key_method?
             method = "#{key_method}="
             self.method(method).call(value)
           end
@@ -296,6 +316,7 @@ module Ecoportal
 
           # @return [nil, Hash] the underlying `Hash` model as is (carrying current changes)
           def doc
+            return @doc if doc_var?
             raise UnlinkedModel.new(from: "#{self.class}#doc", key: _key) unless linked?
             if is_root?
               @doc
@@ -378,8 +399,12 @@ module Ecoportal
 
           protected
 
+          def doc_var?
+            !!defined?(@doc)
+          end
+
           def is_root?
-            _parent == self && !!defined?(@doc)
+            _parent == self && doc_var?
           end
 
           def linked?

data/lib/ecoportal/api/v2_version.rb

@@ -1,5 +1,5 @@
 module Ecoportal
   module API
-    GEM2_VERSION = "0.9.7"
+    GEM2_VERSION = "1.1.1"
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: ecoportal-api-v2
 version: !ruby/object:Gem::Version
-  version: 0.9.7
+  version: 1.1.1
 platform: ruby
 authors:
 - Oscar Segura
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2022-11-29 00:00:00.000000000 Z
+date: 2023-03-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -130,20 +130,20 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.8.5
+        version: 0.9.1
     - - "<"
       - !ruby/object:Gem::Version
-        version: '0.9'
+        version: '0.10'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.8.5
+        version: 0.9.1
     - - "<"
       - !ruby/object:Gem::Version
-        version: '0.9'
+        version: '0.10'
 description: 
 email:
 - rien@ecoportal.co.nz