ecoportal-api-v2 0.8.26 → 0.8.27

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f4693a8a82dab15ddfd0722b1b987efb80ddc7ef9d748e76b7ccff87c0317133
-  data.tar.gz: 999fbaa368b677cdab96889fb34c38fe0c71c4235c000d82e6e38fedd0cf7d10
+  metadata.gz: 82d67186b579f8307a36bb9e17d649eaef352bab449e1628aea603198a6ffb3f
+  data.tar.gz: 390106d2c6aafe1eb2e6154b7a81aba3622400234738140db187704a30bcba86
 SHA512:
-  metadata.gz: 76b33ecae1e147b8e1c1f1dffb118d7533eee71b4e9acc7a5e6609947c7880c4d6f2e3ffc32aac8c5df4af3133573103148861335bfe03a6e85ecf87af5b9e60
-  data.tar.gz: bcd91f124438cfd8aad7b181ea1f62744c8b37d92053e394a5b681a18338f19254b0d7ca5b59e1d50fc16357b138985c0aa2a67a4a57930952f775b9fbcc4248
+  metadata.gz: 9aa958e8c59e53b7c9a71059f8333b8949ee35ce02bc2df4e65810b2f5c8291f67c1271dd2c44a27c37559ccd6c7ad614a31e2ede665437b9e7ed2d84bfa6624
+  data.tar.gz: 2884d7264fd3d1f72d970c144a6eecf3ab6298a81a3427fb87ad8ad05d4024b2280e66d9f22ac60998d39e957924e4496cd73c589cdcd54808d8c84dfeb72a12

data/CHANGELOG.md

@@ -1,14 +1,75 @@
 # Change Log
 All notable changes to this project will be documented in this file.
 
-## [0.8.26] - 2022-02-xx
+## [0.8.27] - 2022-03-xx
 
 ### Added
-  `Ecoportal::API::V2::Page::Stage#add_permit`
-  `Ecoportal::API::V2::Page::Permit.new_doc`
-  
+  - `Ecoportal::API::Common::Content::CollectionModel#clear`
+    - It calls `#delete!` on each item from an `Array` copy
+  - `Ecoportal::API::V2::Page::Components#add`
+    - Now it allows to use a `Hash` _doc_ model to create the `Component`
+  - `Ecoportal::API::V2::Page::Component::SelectionField` **added** some methods:
+    - `#numeric?`, `#text?` to check the `data_type` of the _selection options_
+    - `#switch_type!` to switch the `data_type` between `txt` and `num`
+    - `#deselect` to mark as `false` the property `selected` of some _selection option_
+    - `#name` counterpart of `#value`
+    - `#values` and `#names` they return always an `Array` (no matter if the field is **not** `multiple`)
+    - `#options_by_value` which returns a `Hash` with all the options by their `value`
+    - `#options_by_name` which returns a `Hash` with all the options by their `name`
+  - `Ecoportal::API::V2::Page::Component` **added** some methods:
+    - `#forces`, `#bindings` and `#bindings?` to check if a _Component_ has bindings in what forces and what those bindings are.
+  - `Ecoportal::API::V2::Page::Force#bind` to add a _binding_
+  - `Ecoportal::API::V2::Page::Force::Binding` **added** a couple of methods:
+    - `#force` to refer to the _Force_ object of a binding
+    - `#reference` to retrieve the _component_ or _section_ the binding refers to
+  - `Ecoportal::API::V2::Page::Force::Bindings` **added** some methods:
+    - `#force` to refer to the _Force_ object of the bindings
+    - `#by_name` and `#by_reference` to obtain a `Hash` with all the _bindings_
+    - `#reference?` to check if a _section_ or _component_ have a binding to the _force_
+    - `#get_by_reference` to retrieve an `Array` with all the _bindings_ to one _section_ or _component_
+  - `Ecoportal::API::V2::Page::Forces#bindings_by_reference`
+    - For (script) **optimization** purposes. It gets all the _bindings_ as a `Hash` of _reference_ to `Array<Binding>`
+  - `Ecoportal::API::V2::Page::Section` **added** some methods:
+    - `#shared?` to know if the _section_ is shared by multiple stages
+    - `#bindings` to obtain the _bindings_ of the _section_ in any _force_
+    - `#forces` all the _forces_ that this _section_ has a _binding_ to
+    - `#bindings?` whether or not the _section_ has any force _binding_
+  - `Ecoportal::API::V2::Page::Sections` **added** some methods:
+    - `#any_shared?` whether or not there are shared sections
+    - `#move` to **move** the _section_ to some specific position
+
 ### Changed
+  - `ecoportal-api` dependency upgrade
+  - `Ecoportal::API::V2::Page::PermissionFlags#configure`
+    - Added option to pass a `Hash` with pairs `Symbol`, `Boolean`
+    - `flags.configure {subscribed: true}`
+  - `Ecoportal::API::Common::Content::ModelHelpers#same_string?`
+    - Added parameter `mild:` to compare the strings with only alphabetic characters
+  - `Ecoportal::API::V2::Page::Component::SelectionField` **added** some methods:
+    - `#selected` **added** parameters to modify the _returned_ value
+      - `by_name:` and `by_value:` to return a `Hash`
+      - `value:` and `name:` which combines respectively with the above, to cast them in the `Hash`
+  - `Ecoportal::API::V2::Page::Sections#get_by_heading`
+    - added `mild:` option to allow more flexibility on searches
+
 ### Fixed
+  - `Ecoportall::API::Common::Content::DoubleModel#dirty?`
+    - It was checking that `#as_update` equaled `{}`. Now it also checks on `null` too.
+  - `Ecoportal::API::V2::Page::Component::SelectionOption#numeric!`
+    - It was using `to_i` which would return `0` when the source `value` was not numeric.
+    - Now it uses `Float(value)`, provided that it can throw an exception
+  - `Ecoportal::API::V2::Pages::Stages#get`
+    - The _page_ `id` and `stage_id` were used exchanged in the message
+  - `Ecoportal::API::V2::Page::Forces` fixed the `weight` (position) scoping
+    - It was not positioning correctly
+  - `Ecoportal::API::V2::Page::Sections` fixed the `weight` (position) scoping
+    - It was not positioning correctly
+
+## [0.8.26] - 2022-03-25
+
+### Added
+  `Ecoportal::API::V2::Page::Stage#add_permit`
+  `Ecoportal::API::V2::Page::Permit.new_doc`
 
 ## [0.8.25] - 2022-02-04
 

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.4', '< 0.9'
+  spec.add_dependency 'ecoportal-api', '>= 0.8.5', '< 0.9'
 end

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

@@ -191,6 +191,11 @@ module Ecoportal
             end
           end
 
+          # Deletes all the elements of this `CollectionModel` instance
+          def clear
+            self.to_a.each {|item| delete!(item)}
+          end
+
           # Deletes `value` from this `CollectionModel` instance
           # @param value [String, Hash, Ecoportal::API::Common::Content::DoubleModel]
           #   - When used as `String`, the `key` value (i.e. `id` value) is expected

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

@@ -329,7 +329,8 @@ module Ecoportal
 
           # @return [Boolean] stating if there are changes
           def dirty?
-            as_update != {}
+            au = as_update
+            !((au == {}) || (au == nil))
           end
 
           # It makes `original_doc` to be like `doc`

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

@@ -7,13 +7,21 @@ module Ecoportal
           private
 
           # Offers multiple ways to compare two strings
-          def same_string?(value1, value2, exact: false)
+          # Offers multiple ways to compare two strings
+          def same_string?(value1, value2, exact: false, mild: false)
             case
             when value1.is_a?(String) && value2.is_a?(String)
               if exact
                 value1 == value2
               else
-                value1.to_s.strip.downcase == value2.to_s.strip.downcase
+                v1 = value1.to_s.strip.downcase
+                v2 = value2.to_s.strip.downcase
+
+                if mild
+                  v1 = v1.gsub(/[^a-z ]+/, ' ').gsub(/\s+/, ' ').strip
+                  v2 = v2.gsub(/[^a-z ]+/, ' ').gsub(/\s+/, ' ').strip
+                end
+                v1 == v2
               end
             when value1.is_a?(Regexp) && value2.is_a?(String)
               value2 =~ value1

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

@@ -10,6 +10,14 @@ module Ecoportal
 
             embeds_many :options, klass: "Ecoportal::API::V2::Page::Component::SelectionOption", order_key: :weight
 
+            def numeric?
+              self.data_type == "num"
+            end
+
+            def text?
+              self.data_type == "txt"
+            end
+
             def numeric!(&block)
               ordered_options.each {|opt| opt.numeric!(&block)}
               self.data_type = "num"
@@ -20,19 +28,42 @@ module Ecoportal
               self.data_type = "str"
             end
 
-            def select(value)
-              opt = options.find {|opt| opt.value == value}
+            def switch_type!(&block)
+              numeric? ? text!(&block) : numeric!(&block)
+            end
+
+            def select(value_name, by_name: false)
+              opt = find_option(value_name, by_name: by_name)
               sel = selected
               return true if !multiple && opt == sel
               sel.selected = false if !multiple && sel
               opt.selected = true unless !opt
             end
 
-            def selected
-              if multiple
-                options.select {|opt| opt.selected}
+            def deselect(value_name, by_name: false)
+              if opt = find_option(value_name, by_name: by_name)
+                opt.selected = false
+              end
+            end
+
+            def selected(by_name: false, by_value: false, value: false, name: false)
+              case
+              when by_value
+                elems = [selected].flatten.compact
+                options_hash(elems) do |option|
+                  name ? option.name : option
+                end
+              when by_name
+                elems = [selected].flatten.compact
+                options_hash(elems, by_name: true) do |option|
+                  value ? option.value : option
+                end
               else
-                options.find {|opt| opt.selected}
+                if multiple
+                  ordered_options.select {|opt| opt.selected}
+                else
+                  options.find {|opt| opt.selected}
+                end
               end
             end
 
@@ -44,6 +75,22 @@ module Ecoportal
               end
             end
 
+            def values
+              [value].flatten.compact
+            end
+
+            def name
+              if multiple
+                selected.map {|opt| opt.name}
+              else
+                selected&.name
+              end
+            end
+
+            def names
+              [name].flatten.compact
+            end
+
             def add_option(value:, name: nil, pos: NOT_USED, before: NOT_USED, after: NOT_USED)
               opt_doc = options.items_class.new_doc
               options.upsert!(opt_doc, pos: pos, before: before, after: after) do |option|
@@ -63,6 +110,26 @@ module Ecoportal
               end
             end
 
+            # @param name [Boolean] whether or not the values of the `Hash` should be the `names`
+            # @return [Hash] of **key** _value_ and **value**:
+            #   1. _option_ if `name` is `false`
+            #   2. _name_ if `name` is `true`
+            def options_by_value(name: false)
+              options_hash do |option|
+                name ? option.name : option
+              end
+            end
+
+            # @param value [Boolean] whether or not the values of the `Hash` should be the `values`
+            # @return [Hash] of **key** _name_ and **value**:
+            #   1. _option_ if `value` is `false`
+            #   2. _value_ if `value` is `true`
+            def options_by_name(value: false)
+              options_hash(by_name: true) do |option|
+                value ? option.value : option
+              end
+            end
+
             def to_s(name: true, delimiter: "\n")
               [selected].flatten.compact.map do |opt|
                 name ? opt.name : opt.value
@@ -75,7 +142,7 @@ module Ecoportal
             #   - `:multiple` to allow multiple selection
             #   - `:single` to set to singular selection
             #   - `:other` to enable `other` button
-            #   - `:options` to add options (`Hash<value, name>`)
+            #   - `:options` to add options (`Hash<value, name>`) or update option names
             #   - `:type` to define the type
             #     - `:num`
             #     - `:str`
@@ -122,8 +189,13 @@ module Ecoportal
             private
 
             def configure_options(opts)
+              hopts = self.options_by_value
               opts.each do |val, nm|
-                add_option(value: val, name: nm)
+                if option = hopts[val]
+                  option.name = nm
+                else
+                  add_option(value: val, name: nm)
+                end
               end
             end
 
@@ -140,7 +212,20 @@ module Ecoportal
               opts[pos]
             end
 
+            def find_option(value_name, by_name: false)
+              if by_name
+                opt = options.find {|opt| same_string?(opt.name, value_name)}
+              else
+                opt = options.find {|opt| opt.value == value_name}
+              end
+            end
 
+            def options_hash(elems = ordered_options, by_name: false)
+              elems.each_with_object({}) do |option, hash|
+                key = by_name ? option.name : option.value
+                hash[key] = block_given?? yield(option) : option
+              end
+            end
           end
         end
       end

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

@@ -21,13 +21,27 @@ module Ecoportal
             passboolean :selected
 
             def numeric!
-              self.value = block_given?? yield(value) : value.to_i
+              self.value = block_given?? yield(value) : to_i(value)
             end
 
             def text!
               self.value = block_given?? yield(value) : value.to_s
             end
 
+            private
+
+            def to_i
+              Float(value).to_i
+            end
+
+            # https://stackoverflow.com/a/5661695/4352306
+            def is_number?(value)
+              begin
+                true if Float(value)
+              rescue ArgumentError => e
+                false
+              end
+            end
           end
         end
       end

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

@@ -124,6 +124,26 @@ module Ecoportal
             secs.length > 1
           end
 
+          # @return [Array<Ecoportal::API::V2::Page::Force::Binding] the force bindings if any.
+          def bindings
+            ooze.forces.each_with_object([]) do |force, out|
+              out.push(*force.bindings.get_by_reference(self))
+            end
+            out
+          end
+
+          # @return [Array<Ecoportal::API::V2::Page::Force] the forces this component is bound to if any.
+          def forces
+            ooze.forces.select do |force|
+              force.bindings.reference?(self)
+            end
+          end
+
+          # @return [Boolean] `true` if the component is bound to any force, `false` otherwise
+          def bindings?
+            forces.count > 0
+          end
+
           def indexable_label
             self.class.indexable_label(label)
           end

data/lib/ecoportal/api/v2/page/components.rb

@@ -42,15 +42,18 @@ module Ecoportal
           end
 
           # It creates a **new** component
+          # @note
+          #   - You can use either `type` and `label` **or** `doc`
           # @param label [String, nil]
           # @param type [String] the type of the field
+          # @param doc [Hash] to copy another field model
           # @yield [field] do some stuff with field
           # @yieldparam [Ecoportal::API::V2::Page::Component] the created field
           # @return [Ecoportal::API::V2::Page::Component] the created field.
-          def add(label:, type:)
-            fld_doc = component_class.new_doc(type: type)
+          def add(doc: nil, label: doc && doc["label"], type: doc && doc["type"])
+            fld_doc = doc ? JSON.parse(doc.to_json) : component_class.new_doc(type: type)
             upsert!(fld_doc) do |fld|
-              fld.label  = label
+              fld.label  = label if !doc
               yield(fld) if block_given?
             end
           end

data/lib/ecoportal/api/v2/page/force/binding.rb

@@ -22,6 +22,10 @@ module Ecoportal
               self._parent.ooze
             end
 
+            def force
+              self._parent.force
+            end
+
             # @return [Boolean] whether or not this the binding refers to a Component
             def component?
               type == COMPONENT_TYPE
@@ -43,6 +47,11 @@ module Ecoportal
               raise "This is not a section binding, but a component binding" unless section?
               ooze.sections.get_by_id(reference_id)
             end
+
+            # @return [Ecoportal::API::V2::Page::Component, Ecoportal::API::V2::Page::Section]
+            def reference
+              self.component? ? component : section
+            end
           end
         end
       end

data/lib/ecoportal/api/v2/page/force/bindings.rb

@@ -13,6 +13,63 @@ module Ecoportal
               self._parent.ooze
             end
 
+            def force
+              self._parent
+            end
+
+            # @note first local binding name will shadow later ones
+            # @param only_winner [Boolean] specifies if retrieving multiple bindings with same name or just the winner
+            # @return [Hash] where **key** is `name` and **value** is
+            #   1. a single _binding_, if `only_winner` is `true`
+            #   2. an `Array` of _bindings_ with same _name_, otherwise
+            def by_name(only_winner: false)
+              self.each_with_object({}) do |binding, hash|
+                if winner_only
+                  hash[binding.name] ||= binding
+                else
+                  hash[binding.name] ||= []
+                  hash[binding.name].push(binding)
+                end
+              end
+            end
+
+
+            # @note first local binding name will shadow later ones.
+            # @param only_winner [Boolean] specifies if shadowed bindings (inactive) should be excluded (`true`)
+            # @return [Hash] where **key** is a _section_ or a _component_ and **value** is eitheran `Array` of _bindings_.
+            def by_reference(only_winner: false)
+              if only_winner
+                by_name(only_winner: true).each_with_object({}) do |(name, binds), hash|
+                  if binds.is_a?(Array)
+                    binds.each {|binding| (hash[binding.reference] ||= []).push(binding)}
+                  else
+                    (hash[binds.reference] ||= []).push(binds)
+                  end
+                end
+              else
+                self.each_with_object({}) do |binding, hash|
+                  (hash[binding.reference] ||= []).push(binding)
+                end
+              end
+            end
+
+            # @param obj [Ecoportal::API::V2::Page::Section, Ecoportal::API::V2::Page::Component)] the referred component or section.
+            # @return [Boolean] `true` if `obj` is referred in the bindings, `false` otherwise
+            def reference?(obj)
+              get_by_reference(obj).count > 0
+            end
+
+            # @param obj [Ecoportal::API::V2::Page::Section, Ecoportal::API::V2::Page::Component)] the referred component or section.
+            # @return [Array<Ecoportal::API::V2::Page::Force::Binding>] binding to the component or section.
+            def get_by_reference(obj)
+              unless obj.is_a?(Ecoportal::API::V2::Page::Section) || obj.is_a?(Ecoportal::API::V2::Page::Component)
+                raise ArgumentError.new("Expected either a Ecoportal::API::V2::Page::Section or a Ecoportal::API::V2::Page::Component. Given: #{obj.class}")
+              end
+              self.select do |bind|
+                bind.reference == obj
+              end
+            end
+
             # @param id [String] the `id` of the binding to find.
             # @return [Ecoportal::API::V2::Page::Force::Binding] binding with `id`
             def get_by_id(id)

data/lib/ecoportal/api/v2/page/force.rb

@@ -24,6 +24,11 @@ module Ecoportal
           def ooze
             self._parent.ooze
           end
+
+          # @see Ecoportal::API::V2::Page::Force::Bindings#add
+          def bind(reference, **kargs, &block)
+            bindings.add(reference, **kargs, &block)
+          end
         end
       end
     end

data/lib/ecoportal/api/v2/page/forces.rb

@@ -12,6 +12,18 @@ module Ecoportal
             self._parent.ooze
           end
 
+          # Helper to optimize binding searches
+          # @return [Hash] where
+          #   - **key** is a _section_ or a _component_
+          #   - **value** is `Array<Ecoportal::API::V2::Page::Force::Binding>`.
+          def bindings_by_reference(only_winner: false)
+            self.each_with_object({}) do |force, out|
+              force.bindings.by_reference(only_winner: only_winner).each do |ref, bindings|
+                (out[ref] ||= []).push(*bindings)
+              end
+            end
+          end
+
           # @return [Ecoportal::API::V2::Page::Force]
           def get_by_id(id)
             self.find do |force|
@@ -79,7 +91,7 @@ module Ecoportal
                 after.weight
               end
             end.yield_self do |weight|
-              weight = ordered.reject do |frc|
+              weight ||= ordered.reject do |frc|
                 frc.id == force.id
               end.last&.weight
               weight ||= force_class.const_get(:INITIAL_WEIGHT)

data/lib/ecoportal/api/v2/page/permission_flags.rb

@@ -36,26 +36,40 @@ module Ecoportal
           #   - `:subscribed_to_tasks`
           def configure(*conf)
             conf.each_with_object([]) do |cnf, unused|
-              case cnf
-              when :restructure
-                self.can_restructure = true
-              when :configure
-                self.can_configure  = true
-              when :can_permission
-                self.can_permission = true
-              when :create_actions
-                self.can_create_actions = true
-              when :admin_actions
-                self.can_administrate_actions = true
-              when :subscribed
-                self.subscribed = true
-              when :subscribed_to_tasks
-                self.subscribed_to_tasks = true
+              if cnf.is_a?(Symbol)
+                set_flag(cnf, true, residual: unused)
+              elsif cnf.is_a?(Hash)
+                cnf.each do |key, val|
+                  set_flag(key, val, residual: unused)
+                end
               else
-                unused.push(cnf)
+                raise ArgumentError.new("Expecting 'Symbol' or 'Hash'. Given: #{cnf.class}")
               end
             end.yield_self do |unused|
-              raise "Unknown configuaration options #{unused}" unless unused.empty?
+              raise ArgumentError.new("Unknown configuaration options #{unused}") unless unused.empty?
+            end
+          end
+
+          private
+
+          def set_flag(key, val = true, residual: [])
+            case key
+            when :restructure
+              self.can_restructure = val
+            when :configure
+              self.can_configure  = val
+            when :can_permission
+              self.can_permission = val
+            when :create_actions
+              self.can_create_actions = val
+            when :admin_actions
+              self.can_administrate_actions = val
+            when :subscribed
+              self.subscribed = val
+            when :subscribed_to_tasks
+              self.subscribed_to_tasks = val
+            else
+              residual.push(cnf)
             end
           end
         end

data/lib/ecoportal/api/v2/page/section.rb

@@ -43,6 +43,30 @@ module Ecoportal
             ooze.stages.select {|stg| stg.section?(self)}
           end
 
+          # @return [Boolean] `true` if the section belongs to more than 1 stage, `false` otherwise
+          def shared?
+            stages.count > 1
+          end
+
+          # @return [Array<Ecoportal::API::V2::Page::Force::Binding] the force bindings if any.
+          def bindings
+            ooze.forces.each_with_object([]) do |force, out|
+              out.push(*force.bindings.get_by_reference(self))
+            end
+          end
+
+          # @return [Array<Ecoportal::API::V2::Page::Force] the forces this section is bound to if any.
+          def forces
+            ooze.forces.select do |force|
+              force.bindings.reference?(self)
+            end
+          end
+
+          # @return [Boolean] `true` if the section is bound to any force, `false` otherwise
+          def bindings?
+            force.count > 0
+          end
+
           # @return [Boolean] whether or not the section appears in an ooze instance
           def attached?
             !ooze.stages? || stages.any? {|stg| stg.section?(self)}

data/lib/ecoportal/api/v2/page/sections.rb

@@ -11,23 +11,33 @@ module Ecoportal
             self._parent.ooze
           end
 
+          # @return [Boolean] `true` if there is one or more sections shared, `false` otherwise
+          def any_shared?
+            self.any? {|sec| sec.shared?}
+          end
+
           # Creates a new `section`
           # @note
-          #   - It won't fix weights unless all the sections of the ooze are present
+          #   - It won't fix weights unless all the sections of the ooze are present or there aren't shared sections
           #   - This means that it doesn't fix section weights on stages,
           #     as shared sections could change order in other stages
           def add(name: nil, split: false, pos: NOT_USED, before: NOT_USED, after: NOT_USED)
             sec_doc = section_class.new_doc(split: split)
             upsert!(sec_doc) do |section| #, pos: pos, before: before, after: after) do |section|
               section.heading  = name
-              if weight = scope_weight(section, pos: pos, before: before, after: after)
-                section.weight = weight
-              end
-              fix_weights!
+              move(section, pos: pos, before: before, after: after)
               yield(section) if block_given?
             end
           end
 
+          # Moves an existing `section`
+          def move(section, pos: NOT_USED, before: NOT_USED, after: NOT_USED)
+            if weight = scope_weight(section, pos: pos, before: before, after: after)
+              section.weight = weight
+            end
+            fix_weights!
+          end
+
           # @return [Ecoportal::API::V2::Page::Section]
           def get_by_id(id)
             self.find do |sec|
@@ -42,11 +52,12 @@ module Ecoportal
             end
           end
 
+          # @param mild [Boolean] modifier to only compare alphabetic characters
           # @return [Array<Ecoportal::API::V2::Page::Section>]
-          def get_by_heading(heading)
+          def get_by_heading(heading, mild: false)
             ordered.select do |sec|
               value = heading == :unnamed ? nil : heading
-              same_string?(sec.heading, value)
+              same_string?(sec.heading, value, mild: mild)
             end
           end
 
@@ -96,7 +107,7 @@ module Ecoportal
                 after.weight
               end
             end.yield_self do |weight|
-              weight = ordered.reject do |sec|
+              weight ||= ordered.reject do |sec|
                 sec.id == section.id
               end.last&.weight
               weight ||= section_class.const_get(:INITIAL_WEIGHT)
@@ -115,7 +126,7 @@ module Ecoportal
           end
 
           def fix_weights!
-            unless self._parent.is_a?(Ecoportal::API::V2::Pages::PageStage)
+            unless self.any_shared? && self._parent.is_a?(Ecoportal::API::V2::Pages::PageStage)
               ordered.each_with_index do |section, index|
                 section.weight = index
               end

data/lib/ecoportal/api/v2/pages/stages.rb

@@ -31,7 +31,7 @@ module Ecoportal
             wrapped  = Common::Content::WrappedResponse.new(response, page_stage_class)
 
             return wrapped.result if wrapped.success?
-            raise "Could not get stage {#{id}} of page #{stage_id} - Error #{response.status}: #{response.body}"
+            raise "Could not get stage {#{stage_id}} of page #{id} - Error #{response.status}: #{response.body}"
           end
 
           # Requests to update an existing stage via api.

data/lib/ecoportal/api/v2_version.rb

@@ -1,5 +1,5 @@
 module Ecoportal
   module API
-    GEM2_VERSION = "0.8.26"
+    GEM2_VERSION = "0.8.27"
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: ecoportal-api-v2
 version: !ruby/object:Gem::Version
-  version: 0.8.26
+  version: 0.8.27
 platform: ruby
 authors:
 - Oscar Segura
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2022-03-24 00:00:00.000000000 Z
+date: 2022-04-25 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -130,7 +130,7 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.8.4
+        version: 0.8.5
     - - "<"
       - !ruby/object:Gem::Version
         version: '0.9'
@@ -140,7 +140,7 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.8.4
+        version: 0.8.5
     - - "<"
       - !ruby/object:Gem::Version
         version: '0.9'