mini_defender 0.5.0 → 0.5.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ce828ce2c5bbb54c61b4251dbb22c7a2c47c454d8e2c40200e87def85331960b
-  data.tar.gz: 5e68bda0f0104af4d084458191a59626e29323b07cb738959784d723e44e670a
+  metadata.gz: 36f0ca2e9d174da94e991c9eed6300ae0f0315ab57b3d9372921726261318c67
+  data.tar.gz: a4f29b8f5558008c1f5b992722a8ab9dd07dbc93d51d7ec288bd3be5e9d1d91b
 SHA512:
-  metadata.gz: 3e77385c0db7fb62dd8b4cce604919fd9d0d0ae5e0fad90edf4d891f63d256cb99a9a07321114688168a2e899c7dc177d9f92e61f6b3eaa063217d2a079d73a7
-  data.tar.gz: efc638b587907dd095dc31ad90a87396e778f1eb147f48010664b6a39f11117286f1f2d342cd62900c851eddb081c39ecadbac1faada9ec0cf6541227091820d
+  metadata.gz: 885a3ff5f6a337246c9b47e6ac21b0e9dc21067a11c49d7af2d9135d66f6ab878a620b9f73cafa5b1316d5edc2d74f709f924e37309bbe74bb596500abe8348b
+  data.tar.gz: 3d6a38389f37816a4b6ebbadd7700b6c628e7011c42dfbe0a07ee907f0d354b9b7aaed18179ad97c8350ea01948966ee47182b40e5630370761a5487c58fbe97

data/lib/mini_defender/rules_expander.rb

@@ -15,14 +15,27 @@ module MiniDefender
         end
 
         k_pattern = Regexp.compile('\A' + rule_key.gsub(/\*/, '\d+') + '\Z')
+        k_pattern_result = {}
 
         flat_data.each do |value_key, _|
           next unless k_pattern.match?(value_key)
-          result[value_key] = rule_set
+          k_pattern_result[value_key] = rule_set
         end
+
+        if k_pattern_result.empty?
+          k_pattern_result[k_pattern.source.gsub(/\\[AZ]/, '').gsub('\d+', '0')] = rule_set
+        end
+
+        result.merge!(k_pattern_result)
       end
 
       result
     end
+
+    def array_patterns(rules)
+      rules
+        .filter { |key, _| key.include?('*') }
+        .map { |key, _| Regexp.compile('\A' + key.gsub(/\*/, '\d+') + '\Z') }
+    end
   end
 end

data/lib/mini_defender/validator.rb

@@ -12,10 +12,13 @@ module MiniDefender
       @coerced = {}
       @expander = RulesExpander.new
       @factory = RulesFactory.new
+      @array_patterns = @expander.array_patterns(rules)
     end
 
     def validate
-      return unless @errors.nil?
+      unless @errors.nil?
+        return
+      end
 
       @errors = {}
 
@@ -77,15 +80,15 @@ module MiniDefender
         end
       end
 
-      @validated = @validated.expand
-      @coerced = @coerced.expand
-
       @errors.reject! { |_, v| v.empty? }
     end
 
     def validate!
       validate if @errors.nil?
-      raise ValidationError.new('Data validation failed', @errors) unless @errors.empty?
+
+      unless @errors.empty?
+        raise ValidationError.new('Data validation failed', @errors)
+      end
     end
 
     def passes?
@@ -99,12 +102,12 @@ module MiniDefender
 
     def validated
       validate! if @errors.nil?
-      @validated
+      @validated_compacted ||= compact_expanded(@validated)
     end
 
     def coerced
       validate! if @errors.nil?
-      @coerced
+      @coerced_compacted = compact_expanded(@coerced)
     end
 
     # @return [Hash]
@@ -133,5 +136,57 @@ module MiniDefender
 
       Regexp.compile("\\A#{search_key.reverse.join('\.')}\\z")
     end
+
+    def compact_expanded(hash)
+      expanded = {}
+
+      hash.each do |k, v|
+        keys = k.split('.')
+        node = expanded
+
+        while keys.length > 1
+          next_key = keys.shift
+          next_node = (node[next_key] ||= {})
+
+          if next_node.is_a?(Array)
+            node[next_key] = next_node = next_node.each_with_index.to_h do |value, index|
+              [index.to_s, value]
+            end
+          end
+
+          node = next_node
+        end
+
+        node[keys.shift] = v
+      end
+
+      compact_keys(expanded)
+    end
+
+    def compact_keys(hash, current_key = '')
+      current_key_suffixed = if current_key == ''
+        ''
+      else
+        "#{current_key}."
+      end
+
+      result = hash.to_h do |k, v|
+        [k, v.is_a?(Hash) ? compact_keys(v, current_key_suffixed + k) : v]
+      end
+
+      if result.empty?
+        return result
+      end
+
+      unless @array_patterns.any? { |p| p.match?("#{current_key_suffixed}#{result.keys.first}") }
+        return result
+      end
+
+      if result.all? { |k, _v| k.match?(/\A\d+\z/) }
+        return result.values
+      end
+
+      result
+    end
   end
 end

data/lib/mini_defender/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module MiniDefender
-  VERSION = "0.5.0"
+  VERSION = "0.5.1"
 end

data/lib/mini_defender.rb

@@ -13,7 +13,6 @@ require_relative 'mini_defender/validation_helpers'
 
 # Extensions to Ruby Core
 require_relative 'mini_defender/extensions/enumerable'
-require_relative 'mini_defender/extensions/hash'
 
 module MiniDefender
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: mini_defender
 version: !ruby/object:Gem::Version
-  version: 0.5.0
+  version: 0.5.1
 platform: ruby
 authors:
 - Ali Alhoshaiyan
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2024-01-16 00:00:00.000000000 Z
+date: 2024-01-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -97,7 +97,6 @@ files:
 - Rakefile
 - lib/mini_defender.rb
 - lib/mini_defender/extensions/enumerable.rb
-- lib/mini_defender/extensions/hash.rb
 - lib/mini_defender/handles_validation_errors.rb
 - lib/mini_defender/rule.rb
 - lib/mini_defender/rules.rb

data/lib/mini_defender/extensions/hash.rb

@@ -1,39 +0,0 @@
-# frozen_string_literal: true
-
-class Hash
-  def expand
-    expanded = {}
-
-    each do |k, v|
-      keys = k.split('.')
-      node = expanded
-
-      while keys.length > 1
-        next_key = keys.shift
-        next_node = (node[next_key] ||= {})
-
-        if next_node.is_a?(Array)
-          node[next_key] = next_node = next_node.each_with_index.to_h { |value, index| [index.to_s, value] }
-        end
-
-        node = next_node
-      end
-
-      node[keys.shift] = v
-    end
-
-    expanded.compact_keys
-  end
-
-  def compact_keys
-    result = to_h do |k, v|
-      [k, v.is_a?(Hash) ? v.compact_keys : v]
-    end
-
-    if !result.empty? && result.all? { |k, _v| k.match?(/\A\d+\z/) }
-      result.values
-    else
-      result
-    end
-  end
-end