activegraph-extensions 0.0.2-java → 0.0.3-java

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6c32c0f026083ce5175029f15de509da256e64f55456d29ee5c26358a43694ac
-  data.tar.gz: 78ce8b9d21c30b326a4314487ac75d6d0b5e449dcf20fe3c9ef6da29f332ce9f
+  metadata.gz: fbb2be9a5d8f1afc55b3888e1afc33940a2eb709de5afae7144f25992d2d0873
+  data.tar.gz: 23a259b8d2dffd7e6f73e552275fe0d9b9849bfcb80bc437be21c6e32944d5fe
 SHA512:
-  metadata.gz: 00bdc4fb09667e26efcc0f376c6e54d566dd2f3ab1fc804f6e24b95da84981060ca920163c9b7c550d2da1bc44a50137beb6be7dad0063bdc1d71581df57ad1c
-  data.tar.gz: 61f86edaea0e316af6a97190d16dd25d582e1cd96a0778547101db87c520fdff8973fc5e71a2901169eee0ce63730c810aa95a90281a9b9fb6cf680e36c34f8a
+  metadata.gz: 55a7e0e10065e825acef207348a9277162af5d6ec81b158b957b266e37ba0fa98c6aa620cca84a31b253a58574880742a13d7f7f7cda576fd1f7dab3e60eda15
+  data.tar.gz: 679aa1b9d1ad94633506b1938cb3d50ea3bd2af2bd9392f352370f5d4410476cabf1a327bae1d6cafdbd18f5b62650b94584c826ea65cb4bb61be1a035eaa57a

data/CHANGELOG.md

@@ -1 +1,7 @@
-# Change Log
+# Change Log
+
+## [0.0.3] 2023-02-23
+
+## Fixed
+
+- Fixed issue where there was 500 error when we are eagerloding relations with multiple optional matches and some privious match was haveing authorization and next one was skipping authorization for those same nodes. This would result in a collection of related nodes where some nodes were null and corresponding next optional match nodes on path had values.

data/activegraph-extensions.gemspec

@@ -33,7 +33,6 @@ DESCRIPTION
   s.add_development_dependency('guard-rspec')
   s.add_development_dependency('guard-rubocop')
   s.add_development_dependency('neo4j-rake_tasks', '>= 0.3.0')
-  # s.add_development_dependency("neo4j-#{ENV['driver'] == 'java' ? 'java' : 'ruby'}-driver", '~> 4.1.0.beta.1')
   s.add_development_dependency('os')
   s.add_development_dependency('pry')
   s.add_development_dependency('railties', '>= 4.0')

data/lib/active_graph_extensions/node/query/query_proxy_eager_loading/association_limiting.rb

@@ -45,8 +45,9 @@ module ActiveGraphExtensions
           end
 
           def limit_node_in_where_clause(query, path)
-            root_path = path[0..0]
-            query.where("`#{path_name(root_path)}` in [i IN #{node_from_collection(root_path)} | i[1]]")
+            (path.length - 1).times.inject(query) do |query_with_where, index|
+              query_with_where.where("`#{path_name(path[0..index])}` in [i IN #{node_from_collection(path[0..index])} | i[1]]")
+            end
           end
 
           def node_from_collection(path_step)

data/lib/active_graph_extensions/node/query/query_proxy_eager_loading/enhanced_tree.rb

@@ -35,20 +35,10 @@ module ActiveGraphExtensions
           end
 
           def process_string(str)
-            # head, rest = str.split('.', 2)
-            # head, association_limit = extract_assoc_limit(head)
-            # k, length = head.split('*', -2)
-            # length = { max: length } if length
-            #add_nested(k.to_sym, rest, length, association_limit)
             map = StringParsers::RelationParser.new.parse(str)
             add_nested(map[:rel_name].to_sym, map[:rest_str].to_s.presence, map[:length_part], map[:limit_digit])
           end
 
-          # def extract_assoc_limit(str)
-          #   transformer = StringParsers::RelationParamTransformer.new(str)
-          #   [transformer.rel_name_n_length, transformer.rel_limit_number]
-          # end
-
           def process_hash(spec)
             spec = spec.dup
             @options = spec.delete(:_options)

data/lib/active_graph_extensions/node/query/query_proxy_eager_loading/scope_eager_loading.rb

@@ -10,10 +10,8 @@ module ActiveGraphExtensions
             rel_model = relationship_model(path)
             return {} if @opts.blank? || !(auth_scope = authorized_scope(rel_model, path))
             conf = { rels: [], chain: {} }
-            proxy = auth_scope.call(var, "#{var}_rel", user: @opts[:user],
-                                                       properties: properties_for(rel_model),
-                                                       privileges: @opts[:privileges],
-                                                       rel_length: path.last.rel_length)
+            proxy = auth_scope.call(var, "#{var}_rel", @opts.merge(properties: properties_for(rel_model),
+                                                                   rel_length: path.last.rel_length))
             proxy_rel_parts(proxy.instance_variable_get(:@break_proxy) || proxy, conf)
             conf
           end

data/lib/active_graph_extensions/node/query/query_proxy_eager_loading.rb

@@ -57,7 +57,7 @@ module ActiveGraphExtensions
 
         def optional_match_with_where(query, path, vars)
           computed_query = super
-          computed_query = limit_node_in_where_clause(computed_query, path) if multipath_with_sideload_limit?(path)
+          computed_query = limit_node_in_where_clause(computed_query, path) if multipath?(path)
           skip_order? && !path.last.rel_length ? computed_query : optional_order(computed_query, path, vars)
         end
 

data/lib/active_graph_extensions/version.rb

@@ -1,3 +1,3 @@
 module ActiveGraphExtensions
-  VERSION = '0.0.2'
+  VERSION = '0.0.3'
 end

metadata

@@ -1,22 +1,22 @@
 --- !ruby/object:Gem::Specification
 name: activegraph-extensions
 version: !ruby/object:Gem::Version
-  version: 0.0.2
+  version: 0.0.3
 platform: java
 authors:
 - Amit Suryavanshi
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-09-23 00:00:00.000000000 Z
+date: 2023-02-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
-  name: parslet
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+  name: parslet
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -25,12 +25,12 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: activegraph
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+  name: activegraph
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -39,12 +39,12 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: guard
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+  name: guard
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -53,12 +53,12 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: guard-rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+  name: guard-rspec
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -67,12 +67,12 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: guard-rubocop
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+  name: guard-rubocop
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -81,12 +81,12 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: neo4j-rake_tasks
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
         version: 0.3.0
+  name: neo4j-rake_tasks
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -95,12 +95,12 @@ dependencies:
       - !ruby/object:Gem::Version
         version: 0.3.0
 - !ruby/object:Gem::Dependency
-  name: os
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+  name: os
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -109,12 +109,12 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: pry
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+  name: pry
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -123,12 +123,12 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: railties
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
         version: '4.0'
+  name: railties
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -137,12 +137,12 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '4.0'
 - !ruby/object:Gem::Dependency
-  name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+  name: rake
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -151,12 +151,12 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: rubocop
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
         version: 0.56.0
+  name: rubocop
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -165,12 +165,12 @@ dependencies:
       - !ruby/object:Gem::Version
         version: 0.56.0
 - !ruby/object:Gem::Dependency
-  name: yard
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+  name: yard
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -179,12 +179,12 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: dryspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+  name: dryspec
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -193,12 +193,12 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "<"
       - !ruby/object:Gem::Version
         version: '3.10'
+  name: rspec
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -206,8 +206,8 @@ dependencies:
     - - "<"
       - !ruby/object:Gem::Version
         version: '3.10'
-description: "  Additional features to activegraph, like sideload limiting, authorizing
-  sideloads etc.\n"
+description: "  Additional features to activegraph, like sideload limiting, authorizing\
+  \ sideloads etc.\n"
 email: amitbsuryavabshi@mail.com
 executables: []
 extensions: []
@@ -255,7 +255,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.3.7
+rubygems_version: 3.0.6
 signing_key:
 specification_version: 4
 summary: Additional features to activegraph