active_force 0.23.0 → 0.24.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 36b5f3173ab1e4adb37ab96a731723843304a96b658c41c946c65fb9d5d44b30
-  data.tar.gz: 9b2a90628f126b3df5232480282e931a3518a85d8d7a321fefd53e7eccb91e74
+  metadata.gz: 0e6bf4ca3441ea8c1d0df00f6a8925f021fa53a3a0ce2e19beda69ecab4b5881
+  data.tar.gz: 940b3627880eb3a67d8387c744246a8bb29cc816ccf828130dd375e1e33b1791
 SHA512:
-  metadata.gz: bf3392ac7ee848d2b4634002b9d6f7ac3983efff47aa30ecbaf4328c386a258220695251399d3974c1d53f3145b1e73dd31b9d29bad874730f7f67a346cea804
-  data.tar.gz: fa747a0140dd738b13e8956e58fac620af22b37fb896aa503273e8dec73a7c00bb7f1ea8cb87325286ea6c1afa52d139211e083178b5a406863901936ba31a99
+  metadata.gz: b7ab1dc49814c642b34b61391a2d8ab99fc2b0854f3113b5c4d5715a67ec681d3061fa2e3e53eea3ec9780525aeba3804c7043901b7652baa67113d2659bd8dc
+  data.tar.gz: d69e76e37be28552a80a9a3e2872a48ff15e04df4178f5ce2b670426be23570a7ca11256e41ed757ba04fc20270bff875730d8aea2927eee7e0129124c5f994e

data/CHANGELOG.md

@@ -2,6 +2,9 @@
 
 ## Not released
 
+## 0.24.0
+- Add support for nested select statements that are used in conjuction with nested includes (https://github.com/Beyond-Finance/active_force/pull/102)
+
 ## 0.23.0
 - Partially addresses #90. `#select` accepts a block and returns an array of filtered SObjects. (https://github.com/Beyond-Finance/active_force/pull/99)
 

data/README.md

@@ -218,6 +218,17 @@ Comment.includes(post: :owner)
 Comment.includes({post: {owner: :account}})
 ```
 
+You can also use #select with a multi level #includes.
+
+Examples:
+
+```ruby
+Comment.select(:body, post: [:title, :is_active]).includes(post: :owner)
+Comment.select(:body, account: :owner_id).includes({post: {owner: :account}})
+```
+
+The Sobject name in the #select must match the Sobject in the #includes for the fields to be filtered.
+
 ### Aggregates
 
 Summing the values of a column:

data/lib/active_force/active_query.rb

@@ -1,5 +1,6 @@
 require 'active_support/all'
 require 'active_force/query'
+require 'active_force/select_builder'
 require 'forwardable'
 
 module ActiveForce
@@ -25,7 +26,7 @@ module ActiveForce
   class ActiveQuery < Query
     extend Forwardable
 
-    attr_reader :sobject, :association_mapping, :belongs_to_association_mapping
+    attr_reader :sobject, :association_mapping, :belongs_to_association_mapping, :nested_query_fields
 
     def_delegators :sobject, :sfdc_client, :build, :table_name, :mappings
     def_delegators :to_a, :blank?, :present?, :any?, :each, :map, :inspect, :pluck, :each_with_object
@@ -36,6 +37,7 @@ module ActiveForce
       @belongs_to_association_mapping = {}
       super custom_table_name || table_name
       fields sobject.fields
+      @nested_query_fields = []
     end
 
     def to_a
@@ -86,8 +88,11 @@ module ActiveForce
         end
         result
       else
-        selected_fields.map! { |field| mappings[field] }
-        super *selected_fields
+        fields_collection = ActiveForce::SelectBuilder.new(selected_fields, self).parse
+        nested_query_fields.concat(fields_collection[:nested_query_fields]) if fields_collection[:nested_query_fields]
+        return self if fields_collection[:non_nested_query_fields].blank?
+  
+        super *fields_collection[:non_nested_query_fields]
       end
     end
 
@@ -114,7 +119,7 @@ module ActiveForce
     end
 
     def includes(*relations)
-      includes_query = Association::EagerLoadBuilderForNestedIncludes.build(relations, sobject)
+      includes_query = Association::EagerLoadBuilderForNestedIncludes.build(relations, sobject, nil, nested_query_fields) 
       fields includes_query[:fields]
       association_mapping.merge!(includes_query[:association_mapping])
       self

data/lib/active_force/association/eager_load_builder_for_nested_includes.rb

@@ -6,18 +6,19 @@ module ActiveForce
     class EagerLoadBuilderForNestedIncludes
 
       class << self
-        def build(relations, current_sobject, parent_association_field = nil)
-          new(relations, current_sobject, parent_association_field).projections
+        def build(relations, current_sobject, parent_association_field = nil, query_fields = nil)
+          new(relations, current_sobject, parent_association_field, query_fields).projections
         end
       end
 
-      attr_reader :relations, :current_sobject, :association_mapping, :parent_association_field, :fields
+      attr_reader :relations, :current_sobject, :association_mapping, :parent_association_field, :fields, :query_fields
 
-      def initialize(relations, current_sobject, parent_association_field = nil)
+      def initialize(relations, current_sobject, parent_association_field = nil, query_fields = nil)
         @relations = [relations].flatten
         @current_sobject = current_sobject
         @association_mapping = {}
         @parent_association_field = parent_association_field
+        @query_fields = query_fields
         @fields = []
       end
 
@@ -37,10 +38,17 @@ module ActiveForce
       end
 
       def build_includes(association)
-        fields.concat(EagerLoadProjectionBuilder.build(association, parent_association_field))
+        fields.concat(EagerLoadProjectionBuilder.build(association, parent_association_field, query_fields_for(association)))
         association_mapping[association.sfdc_association_field.downcase] = association.relation_name
       end
 
+      def query_fields_for(association)
+        return nil if query_fields.blank?
+        query_fields_with_association = query_fields.find { |nested_field| nested_field[association.relation_name].present? }
+        return nil if query_fields_with_association.blank?
+        query_fields_with_association[association.relation_name].map { |field| association.relation_model.mappings[field] }
+      end
+
       def build_hash_includes(relation, model = current_sobject, parent_association_field = nil)
         relation.each do |key, value|
           association = model.associations[key]
@@ -63,10 +71,10 @@ module ActiveForce
 
       def build_relation(association, nested_includes)
         builder_class = ActiveForce::Association::EagerLoadProjectionBuilder.projection_builder_class(association)
-        projection_builder = builder_class.new(association)
+        projection_builder = builder_class.new(association, nil, query_fields_for(association))
         sub_query = projection_builder.query_with_association_fields
         association_mapping[association.sfdc_association_field.downcase] = association.relation_name
-        nested_includes_query = self.class.build(nested_includes, association.relation_model)
+        nested_includes_query = self.class.build(nested_includes, association.relation_model, nil, query_fields)
         sub_query.fields nested_includes_query[:fields]
         { fields: ["(#{sub_query})"], association_mapping: nested_includes_query[:association_mapping] }
       end
@@ -78,7 +86,7 @@ module ActiveForce
         else
           current_parent_association_field = association.sfdc_association_field
         end
-        self.class.build(nested_includes, association.relation_model, current_parent_association_field)
+        self.class.build(nested_includes, association.relation_model, current_parent_association_field, query_fields)
       end
     end
   end

data/lib/active_force/association/eager_load_projection_builder.rb

@@ -3,8 +3,8 @@ module ActiveForce
     class InvalidEagerLoadAssociation < StandardError; end
     class EagerLoadProjectionBuilder
       class << self
-        def build(association, parent_association_field = nil)
-          new(association, parent_association_field).projections
+        def build(association, parent_association_field = nil, query_fields = nil)
+          new(association, parent_association_field, query_fields).projections
         end
 
         def projection_builder_class(association)
@@ -15,26 +15,27 @@ module ActiveForce
         end
       end
 
-      attr_reader :association, :parent_association_field
+      attr_reader :association, :parent_association_field, :query_fields
 
-      def initialize(association, parent_association_field = nil)
+      def initialize(association, parent_association_field = nil, query_fields = nil)
         @association = association
         @parent_association_field = parent_association_field
+        @query_fields = query_fields
       end
 
       def projections
         builder_class = self.class.projection_builder_class(association)
-        builder_class.new(association, parent_association_field).projections
+        builder_class.new(association, parent_association_field, query_fields).projections
       end
-
     end
 
     class AbstractProjectionBuilder
-      attr_reader :association, :parent_association_field
+      attr_reader :association, :parent_association_field, :query_fields
 
-      def initialize(association, parent_association_field = nil)
+      def initialize(association, parent_association_field = nil, query_fields = nil)
         @association = association
         @parent_association_field = parent_association_field
+        @query_fields = query_fields
       end
 
       def projections
@@ -54,8 +55,8 @@ module ActiveForce
       # to be pluralized
       def query_with_association_fields
         relationship_name = association.sfdc_association_field
-        query = ActiveQuery.new(association.relation_model, relationship_name)
-        query.fields association.relation_model.fields
+        selected_fields = query_fields || association.relation_model.fields
+        query = ActiveQuery.new(association.relation_model, relationship_name).select(*selected_fields)
         apply_association_scope(query)
       end
     end
@@ -79,7 +80,8 @@ module ActiveForce
                             else
                               association.sfdc_association_field
                             end
-        association.relation_model.fields.map do |field|
+        selected_fields = query_fields || association.relation_model.fields
+        selected_fields.map do |field|
           "#{ association_field }.#{ field }"
         end
       end

data/lib/active_force/query.rb

@@ -1,6 +1,6 @@
 module ActiveForce
   class Query
-    attr_reader :table
+    attr_reader :table, :query_fields
 
     def initialize table
       @table = table

data/lib/active_force/select_builder.rb

@@ -0,0 +1,41 @@
+module ActiveForce
+  class SelectBuilder
+
+    attr_reader :selected_fields, :nested_query_fields, :non_nested_query_fields, :query
+
+    def initialize(selected_fields, query)
+      @query = query
+      @selected_fields = selected_fields
+      @non_nested_query_fields = []
+      @nested_query_fields = []
+    end
+
+    def parse
+      selected_fields.each do |field|
+        case field
+        when Symbol
+          non_nested_query_fields << query.mappings[field]
+        when Hash
+          populate_nested_query_fields(field)
+        when String
+          non_nested_query_fields << field
+        end
+      end
+      {non_nested_query_fields: non_nested_query_fields, nested_query_fields: nested_query_fields}
+    end
+
+    private
+
+    def populate_nested_query_fields(field)
+      field.each do |key, value|
+        case value
+        when Symbol
+          field[key] = [value]
+        when Hash
+          raise ArgumentError, 'Nested Hash is not supported in select statement, you may wish to use an Array'
+        end
+      end
+      nested_query_fields << field
+    end
+  end
+end

data/lib/active_force/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module ActiveForce
-  VERSION = '0.23.0'
+  VERSION = '0.24.0'
 end

data/spec/active_force/sobject/includes_spec.rb

@@ -43,6 +43,25 @@ module ActiveForce
           expect(territory.quota.id).to eq "321"
         end
 
+        context 'when nested select statement' do
+          it 'formulates the correct SOQL query' do
+            soql = Salesforce::Territory.select(:id, :quota_id, quota: :id).includes(:quota).where(id: '123').to_s
+            expect(soql).to eq "SELECT Id, QuotaId, QuotaId.Id FROM Territory WHERE (Id = '123')"
+          end
+
+          it 'errors when correct format is not followed' do
+            expect{Salesforce::Territory.select(:id, :quota_id, quota: {id: :quote}).includes(:quota).where(id: '123').to_s}.to raise_error ArgumentError
+          end
+
+          context 'when nested includes statement' do
+            it 'formulates the correct SOQL query' do
+              soql = Comment.select(:post_id, :body, post: [:title, :is_active], blog: :name).includes(post: :blog).to_s
+
+              expect(soql).to eq "SELECT PostId, Body__c, PostId.Title__c, PostId.IsActive, PostId.BlogId.Name FROM Comment__c"
+            end
+          end
+        end
+
         context 'with namespaced SObjects' do
           it 'queries the API for the associated record' do
             soql = Salesforce::Territory.includes(:quota).where(id: '123').to_s
@@ -156,6 +175,20 @@ module ActiveForce
       end
 
       context 'has_many' do
+        context 'when nested select statement' do
+          it 'formulates the correct SOQL query' do
+            soql = Account.select(opportunities: :id).includes(:opportunities).where(id: '123').to_s
+            expect(soql).to eq "SELECT Id, OwnerId, (SELECT Id FROM Opportunities) FROM Account WHERE (Id = '123')"
+          end
+        end
+
+        context 'when normal select with nested includes' do
+          it 'formulates the correct SOQL query' do
+            soql = Blog.select(:id, :link).includes(posts: :comments).to_s
+            expect(soql).to eq "SELECT Id, Link__c, (SELECT Id, Title__c, BlogId, IsActive, (SELECT Id, PostId, PosterId__c, FancyPostId, Body__c FROM Comments__r) FROM Posts__r) FROM Blog__c"
+          end
+        end
+
         context 'with standard objects' do
           it 'formulates the correct SOQL query' do
             soql = Account.includes(:opportunities).where(id: '123').to_s
@@ -294,6 +327,13 @@ module ActiveForce
       end
 
       context 'has_one' do
+        context 'when nested select statement is present' do
+          it 'formulates the correct SOQL query' do
+            soql = ClubMember.select(:name, :email, membership: :type).includes(:membership).to_s
+            expect(soql).to eq "SELECT Name, Email, (SELECT Type FROM Membership__r) FROM ClubMember__c"
+          end
+        end
+
         context 'when assocation has a scope' do
           it 'formulates the correct SOQL query with the scope applied' do
             soql = Post.includes(:last_comment).where(id: '1234').to_s

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: active_force
 version: !ruby/object:Gem::Version
-  version: 0.23.0
+  version: 0.24.0
 platform: ruby
 authors:
 - Eloy Espinaco
@@ -11,7 +11,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-09-27 00:00:00.000000000 Z
+date: 2024-10-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activemodel
@@ -163,6 +163,7 @@ files:
 - lib/active_force/field.rb
 - lib/active_force/mapping.rb
 - lib/active_force/query.rb
+- lib/active_force/select_builder.rb
 - lib/active_force/sobject.rb
 - lib/active_force/standard_types.rb
 - lib/active_force/table.rb
@@ -219,7 +220,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.5.6
+rubygems_version: 3.4.10
 signing_key:
 specification_version: 4
 summary: Help you implement models persisting on Sales Force within Rails using RESTForce