jsonpath 0.7.2 → 0.8.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 86c55c63011d6bd71967469b464bfeb537f7c54f
-  data.tar.gz: 57d966160fa4106318d122cc2e088a9032925d47
+SHA256:
+  metadata.gz: 250b6d90a78bf853330c06f41cacee3207c39e1cb5a13a0ea747f6511660fefb
+  data.tar.gz: 22a728dbfe6c1ca44884db61e4e2dc2e89c355b7fa4c308fb0bcfc9a2bb3e815
 SHA512:
-  metadata.gz: 4e96817450882137b8907cd90bec5564a3ecd21da7803ca11034ae4a8c4b2a65bdc661efe0b210858af3b4a71e029d8689cbf30f5cefe2beb8047a9b2e07ffc4
-  data.tar.gz: 050745e85fe208a99a963c54eac2426cdfaf701a9f6658e6cd2f5683bfa0730af1cf690a92e0d3fc8bb006afa8f8a2221c99d92178944689bb7f1b0efc35e095
+  metadata.gz: 44abf571b8d09b3dd4c5ec51a801ca064cfc08569a4121d58d456bd8403c3396b33f8f369992b258016374c8d823825f1619430c7434ebba2eec1bc19dd10b8c
+  data.tar.gz: 11ec6a115406f90775f3eb73b959befc1249e47d998b6679f5fcca671b04a694872282aa6ac6b79faad8d6fd1767c84b65abea536e0773518d76329ebb866ceb

data/README.md

@@ -103,8 +103,6 @@ enum.any?{ |c| c == 'red' }
 # => true
 ```
 
-You can optionally prevent eval from being called on sub-expressions by passing in :allow_eval => false to the constructor.
-
 ### More examples
 
 For more usage examples and variations on paths, please visit the tests. There are some more complex ones as well.

data/lib/jsonpath.rb

@@ -3,6 +3,7 @@ require 'multi_json'
 require 'jsonpath/proxy'
 require 'jsonpath/enumerable'
 require 'jsonpath/version'
+require 'jsonpath/parser'
 
 # JsonPath: initializes the class with a given JsonPath and parses that path
 # into a token array.

data/lib/jsonpath/enumerable.rb

@@ -1,19 +1,12 @@
 class JsonPath
   class Enumerable
     include ::Enumerable
-    attr_reader :allow_eval
-    alias_method :allow_eval?, :allow_eval
 
     def initialize(path, object, mode, options = nil)
       @path = path.path
       @object = object
       @mode = mode
       @options = options
-      @allow_eval = if @options && @options.key?(:allow_eval)
-                      @options[:allow_eval]
-                    else
-                      true
-                    end
     end
 
     def each(context = @object, key = nil, pos = 0, &blk)
@@ -27,10 +20,6 @@ class JsonPath
         each(context, key, pos + 1, &blk) if node == @object
       when /^\[(.*)\]$/
         handle_wildecard(node, expr, context, key, pos, &blk)
-      else
-        if pos == (@path.size - 1) && node && allow_eval?
-          yield_value(blk, context, key) if instance_eval("node #{@path[pos]}")
-        end
       end
 
       if pos > 0 && @path[pos - 1] == '..' || (@path[pos - 1] == '*' && @path[pos] != '..')
@@ -75,11 +64,12 @@ class JsonPath
     end
 
     def handle_question_mark(sub_path, node, pos, &blk)
-      raise 'Cannot use ?(...) unless eval is enabled' unless allow_eval?
       case node
       when Array
         node.size.times do |index|
           @_current_node = node[index]
+          # exps = sub_path[1, sub_path.size - 1]
+          # if @_current_node.send("[#{exps.gsub(/@/, '@_current_node')}]")
           if process_function_or_literal(sub_path[1, sub_path.size - 1])
             each(@_current_node, nil, pos + 1, &blk)
           end
@@ -113,40 +103,22 @@ class JsonPath
     def process_function_or_literal(exp, default = nil)
       return default if exp.nil? || exp.empty?
       return Integer(exp) if exp[0] != '('
-      return nil unless allow_eval? && @_current_node
+      return nil unless @_current_node
 
       identifiers = /@?((?<!\d)\.(?!\d)(\w+))+/.match(exp)
-      # puts JsonPath.on(@_current_node, "#{identifiers}") unless identifiers.nil? ||
-      #                                                           @_current_node
-      #                                                           .methods
-      #                                                           .include?(identifiers[2].to_sym)
-
       unless identifiers.nil? ||
              @_current_node.methods.include?(identifiers[2].to_sym)
         exp_to_eval = exp.dup
         exp_to_eval[identifiers[0]] = identifiers[0].split('.').map do |el|
-          el == '@' ? '@_current_node' : "['#{el}']"
+          el == '@' ? '@' : "['#{el}']"
         end.join
-
         begin
-          return instance_eval(exp_to_eval)
-          # if eval failed because of bad arguments or missing methods
+          return JsonPath::Parser.new(@_current_node).parse(exp_to_eval)
         rescue StandardError
           return default
         end
       end
-
-      # otherwise eval as is
-      # TODO: this eval is wrong, because hash accessor could be nil and nil
-      # cannot be compared with anything, for instance,
-      # @a_current_node['price'] - we can't be sure that 'price' are in every
-      # node, but it's only in several nodes I wrapped this eval into rescue
-      # returning false when error, but this eval should be refactored.
-      begin
-        instance_eval(exp.gsub(/@/, '@_current_node'))
-      rescue
-        false
-      end
+      JsonPath::Parser.new(@_current_node).parse(exp)
     end
   end
 end

data/lib/jsonpath/parser.rb

@@ -0,0 +1,62 @@
+require 'strscan'
+
+class JsonPath
+  # Parser parses and evaluates an expression passed to @_current_node.
+  class Parser
+    def initialize(node)
+      @_current_node = node
+    end
+
+    def parse(exp)
+      exps = exp.split(/(&&)|(\|\|)/)
+      ret = parse_exp(exps.shift)
+      exps.each_with_index do |item, index|
+        case item
+        when '&&'
+          ret &&= parse_exp(exps[index + 1])
+        when '||'
+          ret ||= parse_exp(exps[index + 1])
+        end
+      end
+      ret
+    end
+
+    def parse_exp(exp)
+      exp = exp.gsub(/@/, '').gsub(/[\(\)]/, '')
+      scanner = StringScanner.new(exp)
+      elements = []
+      until scanner.eos?
+        if scanner.scan(/\./)
+          sym = scanner.scan(/\w+/)
+          op = scanner.scan(/./)
+          num = scanner.scan(/\d+/)
+          return @_current_node.send(sym.to_sym).send(op.to_sym, num.to_i)
+        end
+        if t = scanner.scan(/\['\w+'\]+/)
+          elements << t.gsub(/\[|\]|'|\s+/, '')
+        elsif t = scanner.scan(/\s+[<>=][<>=]?\s+?/)
+          operator = t
+        elsif t = scanner.scan(/(\s+)?'?(\w+)?[.,]?(\w+)?'?(\s+)?/) # @TODO: At this point I should trim somewhere...
+          operand = t.delete("'").strip
+        elsif t = scanner.scan(/.*/)
+          raise "Could not process symbol: #{t}"
+        end
+      end
+      el = dig(elements, @_current_node)
+      return false unless el
+      return true if operator.nil? && el
+      operand = operand.to_f if operand.to_i.to_s == operand || operand.to_f.to_s == operand
+      el.send(operator.strip, operand)
+    end
+
+    private
+
+    # @TODO: Remove this once JsonPath no longer supports ruby versions below 2.3
+    def dig(keys, hash)
+      return nil unless hash.key?(keys.first)
+      return hash.fetch(keys.first) if keys.size == 1
+      prev = keys.shift
+      dig(keys, hash.fetch(prev))
+    end
+  end
+end

data/lib/jsonpath/version.rb

@@ -1,3 +1,3 @@
 class JsonPath
-  VERSION = '0.7.2'.freeze
+  VERSION = '0.8.2'.freeze
 end

data/test/test_jsonpath.rb

@@ -80,8 +80,8 @@ class TestJsonpath < MiniTest::Unit::TestCase
     assert_equal [@object['store']['book'][1]], JsonPath.new("$..book[?(@['price'] < 23.0 && @['price'] > 9.0)]").on(@object)
   end
 
-  def test_no_eval
-    assert_equal [], JsonPath.new('$..book[(@.length-2)]', allow_eval: false).on(@object)
+  def test_eval_with_floating_point
+    assert_equal [@object['store']['book'][1]], JsonPath.new("$..book[?(@['price'] == 13.0)]").on(@object)
   end
 
   def test_paths_with_underscores
@@ -109,7 +109,7 @@ class TestJsonpath < MiniTest::Unit::TestCase
   end
 
   def test_counting
-    assert_equal 54, JsonPath.new('$..*').on(@object).to_a.size
+    assert_equal 57, JsonPath.new('$..*').on(@object).to_a.size
   end
 
   def test_space_in_path
@@ -255,6 +255,10 @@ class TestJsonpath < MiniTest::Unit::TestCase
       JsonPath.new("$.store._links").on(@object)
   end
 
+  # def test_filter_support_include
+  #   #assert_equal true, JsonPath.new("$.store.book[(@.tags == 'asdf3')]").on(@object)
+  #   assert_equal true, JsonPath.new("$.store.book..tags[?(@ == 'asdf')]").on(@object)
+  # end
 
   def example_object
     { 'store' => {
@@ -262,7 +266,8 @@ class TestJsonpath < MiniTest::Unit::TestCase
         { 'category' => 'reference',
           'author' => 'Nigel Rees',
           'title' => 'Sayings of the Century',
-          'price' => 9 },
+          'price' => 9,
+          'tags' => ['asdf', 'asdf2']},
         { 'category' => 'fiction',
           'author' => 'Evelyn Waugh',
           'title' => 'Sword of Honour',

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: jsonpath
 version: !ruby/object:Gem::Version
-  version: 0.7.2
+  version: 0.8.2
 platform: ruby
 authors:
 - Joshua Hull
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-05-10 00:00:00.000000000 Z
+date: 2017-05-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: multi_json
@@ -117,6 +117,7 @@ files:
 - jsonpath.gemspec
 - lib/jsonpath.rb
 - lib/jsonpath/enumerable.rb
+- lib/jsonpath/parser.rb
 - lib/jsonpath/proxy.rb
 - lib/jsonpath/version.rb
 - test/test_jsonpath.rb