ecoportal-api-v2 2.0.10 → 2.0.12

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3df004ef45ea643952802e19774bfaa2f6941ea21904f5d28a00fa0eb4a252a1
-  data.tar.gz: e491e2111d01d551225289e91a32129ee1dd91638951315429b6598a0ca872ad
+  metadata.gz: 933cb180c2a1d901d3ec065c20527d8d9ee30db24634c20ae3fa2867042311da
+  data.tar.gz: 5c7bf8198ff1ac64b776538c83f80abde8fba0cf6aa6a9f729444548f6b72936
 SHA512:
-  metadata.gz: 3dabbbe1c8c62ba1aef3f632e8d2cf38b468699c5a6e6eb6dd5d052a2a206ff81d8d60a4cee0a01263a90c7a8ba11c6531a484b647069d9008410c8e9966819f
-  data.tar.gz: cf999bb89aa0d15b4f210fa11e6ae7cf44efe780434a7cc35acaaa7df49e3a7a107f08ca43e832661a8236b87ecc8654433f1374014e70725675ee1af513bb47
+  metadata.gz: c47162752062ca85c5a215ce249c3dee0b702d5b11cf095d0f9d9f68bd178af1e9edb4a3bc5c42792133761c6b036cbe2b8a509b4306327a0c72ce0d77a647bc
+  data.tar.gz: 0b2b8d9ac691e3ba75d599adb23012d01600be0d2888bb6a4f1548c3686d8ccac0a95d1aff5e586b241a185b917cd0f2bff05f82b907670148cf18e372d0e81a

data/CHANGELOG.md

@@ -2,6 +2,33 @@
 
 All notable changes to this project will be documented in this file.
 
+## [2.0.14] - 2024-11-xx
+
+### Added
+
+### Changed
+
+- upgrade `ecoportal-api` core gem
+
+### Fixed
+
+## [2.0.12] - 2024-11-21
+
+### Changed
+
+- upgrade `ecoportal-api` core gem
+
+## [2.0.11] - 2024-10-18
+
+### Changed
+
+- upgrade `ecoportal-api` core gem
+
+### Fixed
+
+- `Ecoportal::API::V2::Page::Component::ReferenceField#delete`
+  - It wasn't deleting references
+
 ## [2.0.10] - 2024-10-01
 
 ### Added
@@ -12,8 +39,6 @@ All notable changes to this project will be documented in this file.
 
 - upgraded core `ecoportal-api` **gem** dependency
 
-### Fixed
-
 ## [2.0.9] - 2024-08-22
 
 ### Added

data/ecoportal-api-v2.gemspec

@@ -25,15 +25,15 @@ Gem::Specification.new do |spec|
   spec.executables   = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
   spec.require_paths = ['lib']
 
-  spec.add_development_dependency 'pry'      , '~> 0.14'
+  spec.add_development_dependency 'pry',       '~> 0.14'
   spec.add_development_dependency 'rake',      '>= 13.0.3', '< 14'
   spec.add_development_dependency 'redcarpet', '>= 3.6.0',  '< 4'
   spec.add_development_dependency 'rspec',     '>= 3.12.0', '< 4'
   spec.add_development_dependency 'rubocop',   '~> 1'
   spec.add_development_dependency 'rubocop-rake', '~> 0'
-  spec.add_development_dependency 'yard',      '~> 0.9'
+  spec.add_development_dependency 'yard',         '~> 0.9'
 
-  spec.add_dependency 'ecoportal-api', '~> 0.10', '>= 0.10.2'
+  spec.add_dependency 'ecoportal-api', '~> 0.10', '>= 0.10.7'
   spec.add_dependency 'mime-types',    '~> 3.5',  '>= 3.5.2'
 end
 

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

@@ -1,3 +1,4 @@
+# rubocop:disable Naming/MethodParameterName
 module Ecoportal
   module API
     module Common
@@ -5,9 +6,9 @@ module Ecoportal
         module HashDiffPatch
           extend DocHelpers
 
-          ID_KEYS    = %w[id]
-          META_KEYS  = %w[id patch_ver]
-          NO_CHANGES = "%not-changed!%"
+          ID_KEYS    = %w[id].freeze
+          META_KEYS  = %w[id patch_ver].freeze
+          NO_CHANGES = "%not-changed!%".freeze
 
           class << self
             # The `patch data` is built as follows:
@@ -38,14 +39,13 @@ module Ecoportal
             # @param b [Hash] previous hash model
             # @return [Hash] a `patch data`
             def patch_diff(a, b)
-              case
-              when b.is_a?(Hash) && !empty?(b) && empty?(a)
+              if b.is_a?(Hash) && !empty?(b) && empty?(a)
                 patch_delete(b)
-              when a.is_a?(Hash) && !empty?(a) && empty?(b)
+              elsif a.is_a?(Hash) && !empty?(a) && empty?(b)
                 patch_new(a)
-              when a.is_a?(Hash) && b.is_a?(Hash)
+              elsif a.is_a?(Hash) && b.is_a?(Hash)
                 patch_update(a, b)
-              when any_array?(a, b)
+              elsif any_array?(a, b)
                 patch_data_array(a, b)
               else
                 a
@@ -54,9 +54,12 @@ module Ecoportal
 
             private
 
-            def equal_values(a, b)
+            def equal_values?(a, b)
               if a.is_a?(String) || b.is_a?(String)
-                return true if a.to_s.strip.empty? && b.to_s.strip.empty?
+                a_empty = a.to_s.strip.empty?
+                b_empty = b.to_s.strip.empty?
+
+                return true if a_empty && b_empty
               end
               a == b
             end
@@ -66,124 +69,137 @@ module Ecoportal
             def patch_data(a, b = nil, delete: false)
               {}.tap do |data_hash|
                 if delete
-                  patch_ver = (a && a["patch_ver"]) || 1
-                  data_hash["patch_ver"] = patch_ver
+                  patch_ver = (a && a['patch_ver']) || 1
+                  data_hash['patch_ver'] = patch_ver
                   next
                 end
+
                 a.each do |key, a_value|
-                  b_value = b[key] if b_has_key = b && b.key?(key)
-                  is_meta_key = META_KEYS.include?(key)
-                  skip_equals = b_has_key && equal_values(a_value, b_value)
-                  next if is_meta_key || skip_equals
+                  next if META_KEYS.include?(key)
+
+                  b_value = nil
+
+                  if b&.key?(key)
+                    b_value = b[key]
+                    next if equal_values?(a_value, b_value)
+                  end
+
                   data_hash[key] = patch_diff(a_value, b_value)
+
                   data_hash.delete(key) if data_hash[key] == NO_CHANGES
                 end
+
                 #if (data_hash.keys - ID_KEYS).empty?
-                if (data_hash.keys - META_KEYS).empty?
-                  return NO_CHANGES
-                else
-                  #patch_ver = (b && b["patch_ver"]) || 1
-                  #data_hash["patch_ver"] = patch_ver
-                  if b && b.key?("patch_ver")
-                    data_hash["patch_ver"] = b["patch_ver"]
-                  elsif a && a.key?("patch_ver")
-                    data_hash["patch_ver"] = a["patch_ver"]
-                  end
-                  data_hash.delete("force_patch")
+                return NO_CHANGES if (data_hash.keys - META_KEYS).empty?
+
+                if b&.key?('patch_ver')
+                  data_hash['patch_ver'] = b['patch_ver']
+                elsif a&.key?('patch_ver')
+                  data_hash['patch_ver'] = a['patch_ver']
                 end
+
+                data_hash.delete("force_patch")
               end
             end
 
             def patch_delete(b)
               return NO_CHANGES unless b.is_a?(Hash)
-              if id = get_id(b, exception: false)
-                {
-                  "id"        => id,
-                  "operation" => "deleted",
-                  "data"      => patch_data(b, delete: true)
-                }
-              else
-                nil
-              end
+              return nil        unless (id = get_id(b, exception: false))
+
+              {
+                'id'        => id,
+                'operation' => 'deleted',
+                'data'      => patch_data(b, delete: true)
+              }
             end
 
             def patch_new(a)
               return NO_CHANGES unless a.is_a?(Hash)
-              if id = get_id(a, exception: false)
-                {
-                  "id"        => id,
-                  "operation" => "new",
-                  "data"      => patch_data(a)
-                }
-              else
-                a
-              end
+              return a          unless (id = get_id(a, exception: false))
+
+              {
+                'id'        => id,
+                'operation' => 'new',
+                'data'      => patch_data(a)
+              }
             end
 
             def patch_update(a, b)
               return NO_CHANGES unless a.is_a?(Hash)
-              if id = get_id(a, exception: false)
-                {
-                  "id"        => id,
-                  "operation" => "changed",
-                  "data"      => patch_data(a, b)
-                }.tap do |update_hash|
-                  return nil unless update_hash["data"] != NO_CHANGES
-                end
-              else
-                a
+              return a          unless (id = get_id(a, exception: false))
+
+              {
+                'id'        => id,
+                'operation' => 'changed',
+                'data'      => patch_data(a, b)
+              }.tap do |update_hash|
+                return nil unless update_hash['data'] != NO_CHANGES
               end
             end
 
             def patch_data_array(a, b)
+              return patch_data_nested_array(a, b) if nested_array?(a, b)
+
+              patch_data_flat_array(a, b)
+            end
+
+            def patch_data_flat_array(a, b)
               original_b = b
-              a ||= []; b ||= []
-              if !nested_array?(a, b)
-                if a.length == b.length && (a & b).length == b.length
-                  if original_b
-                    NO_CHANGES
-                  else
-                    a
-                  end
-                else
-                  a
-                end
-              else # array with nested elements
-                a_ids = array_ids(a)
-                b_ids = array_ids(b)
-
-                del_ids = b_ids - a_ids
-                oth_ids = b_ids & a_ids
-                new_ids = a_ids - b_ids
-
-                arr_delete = del_ids.map do |id|
-                  patch_delete(array_id_item(b, id))
-                end.compact
-
-                arr_update = oth_ids.map do |id|
-                  patch_update(array_id_item(a, id), array_id_item(b, id))
-                end.compact
-
-                arr_new = new_ids.map do |id|
-                  patch_new(array_id_item(a, id))
-                end.compact
-
-                (arr_delete.concat(arr_update).concat(arr_new)).tap do |patch_array|
-                  # remove data with no `id`
-                  patch_array.reject! {|item| !item.is_a?(Hash)}
-                  return NO_CHANGES if patch_array.empty?
-                end
+              a        ||= []
+              b        ||= []
+
+              same_elements =
+                a.length == b.length &&
+                (a & b).length == b.length
+
+              return a          unless same_elements
+              return NO_CHANGES if original_b
+
+              a
+            end
+
+            def patch_data_nested_array(a, b)
+              a ||= []
+              b ||= []
+
+              # array with nested elements
+              a_ids = array_ids(a)
+              b_ids = array_ids(b)
+
+              del_ids = b_ids - a_ids
+              oth_ids = b_ids & a_ids
+              new_ids = a_ids - b_ids
+
+              arr_delete = del_ids.map do |id|
+                patch_delete(array_id_item(b, id))
+              end.compact
+
+              arr_update = oth_ids.map do |id|
+                patch_update(array_id_item(a, id), array_id_item(b, id))
+              end.compact
+
+              arr_new = new_ids.map do |id|
+                patch_new(array_id_item(a, id))
+              end.compact
+
+              arr_delete.concat(arr_update).concat(arr_new).tap do |patch_array|
+                # remove data with no `id`
+                patch_array.select! {|item| item.is_a?(Hash)}
+                return NO_CHANGES if patch_array.empty?
               end
             end
 
             def nested_array?(*arr)
-              case
-              when arr.length > 1
+              if arr.length > 1
                 arr.any? {|a| nested_array?(a)}
-              when arr.length == 1
-                arr = arr.first
+              elsif arr.length == 1
+                arr = arr.first || []
                 arr.any? do |item|
-                  item.is_a?(Hash) && item.has_key?("patch_ver")
+                  next false unless item.is_a?(Hash)
+                  next true  if item.key?('patch_ver')
+                  # next true  if item.key?('id')
+
+                  false
                 end
               else
                 false
@@ -197,11 +213,13 @@ module Ecoportal
             def empty?(a)
               bool   = !a
               bool ||= a.respond_to?(:empty?) && a.empty?
+              bool
             end
-
           end
         end
       end
     end
   end
 end
+
+# rubocop:enable Naming/MethodParameterName

data/lib/ecoportal/api/v2/page/component/reference_field.rb

@@ -16,18 +16,18 @@ module Ecoportal
 
             def reference_ids
               references.map do |ref|
-                ref["id"]
+                ref['id']
               end
             end
 
             def add(*ref_ids)
-              doc["references"].tap do |refs|
+              doc['references'].tap do |refs|
                 ref_ids.each do |ref_id|
                   next if reference_ids.include?(ref_id)
                   refs.push({
-                    "id"        => ref_id,
-                    "weight"    => 0,
-                    "patch_ver" => 0
+                    'id'        => ref_id,
+                    'weight'    => 0,
+                    'patch_ver' => 0
                   })
                 end
               end
@@ -39,13 +39,20 @@ module Ecoportal
 
             def delete(*ref_ids)
               ref_ids.each do |ref_id|
-                doc_ref = doc["references"].find do |df|
-                  df["id"] == ref_id
+                doc_ref = doc['references'].find do |df|
+                  df['id'] == ref_id
                 end
 
                 next unless doc_ref
 
-                doc["references"].delete(doc_ref)
+                ori_ref = original_doc['references'].find do |od|
+                  od['id'] == ref_id
+                end
+
+                # force patch version
+                ori_ref['patch_ver'] ||= 0
+
+                doc['references'].delete(doc_ref)
               end
             end
 

data/lib/ecoportal/api/v2_version.rb

@@ -1,5 +1,5 @@
 module Ecoportal
   module API
-    GEM2_VERSION = '2.0.10'.freeze
+    GEM2_VERSION = '2.0.12'.freeze
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: ecoportal-api-v2
 version: !ruby/object:Gem::Version
-  version: 2.0.10
+  version: 2.0.12
 platform: ruby
 authors:
 - Oscar Segura
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2024-09-30 00:00:00.000000000 Z
+date: 2024-11-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: pry
@@ -135,7 +135,7 @@ dependencies:
         version: '0.10'
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.10.2
+        version: 0.10.7
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -145,7 +145,7 @@ dependencies:
         version: '0.10'
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.10.2
+        version: 0.10.7
 - !ruby/object:Gem::Dependency
   name: mime-types
   requirement: !ruby/object:Gem::Requirement
@@ -302,7 +302,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.5.18
+rubygems_version: 3.5.23
 signing_key:
 specification_version: 4
 summary: A collection of helpers for interacting with the ecoPortal MS's V2 API