active_force 0.21.0 → 0.21.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: acbf91e00afed35fd89a0dc1505ebb40d275883bca6aba9237dad7d554e10b7b
-  data.tar.gz: 488fe99179b48383967688ecbfc2c2f93f3f565ec177e972fb6e9ad5e68b5d8e
+  metadata.gz: 0a2d59fa533f638635afa0e8654fae4f77540310fba228ae697e05eaaf965525
+  data.tar.gz: 0570306ac71f575994b07aace62ad9579d0103f812eceb59aaa83b1a6ddc1911
 SHA512:
-  metadata.gz: 1b29a58711a0d6b63c82604c1bc2f3bfdc9b06e435468016fa7055bdd8ace5c779d09d4f11fdc32e7292ffd8c41a5784698610c1a6184116044bfa77b1c93cb5
-  data.tar.gz: 1bb0165385e5f06c7767d9463842e9daf742e11184e7518b008142162f8fb46bb2d35d5a1ba8f64be702cf169f5d9a6a9bfb3b8a1e83331fc11114c2ae30846a
+  metadata.gz: ceccbbfb84f6f80190d11ae8722ba8fbbe423cc4164b0567df2fd912f7e45150ddfb4092da3af528d431f03527676dd897e5565f6d5b18fdf70492d489667896
+  data.tar.gz: 91962473ab612ca13f755f7b94b891d8c6f849677999ef0333a98924b9f6a001566cd602fc3ee3c9c8e1b6cdcef039fbfeb13e4f1a8f2e284877e5d771a84e3c

data/CHANGELOG.md

@@ -2,6 +2,9 @@
 
 ## Not released
 
+## 0.21.1
+- Fixes #91. Applies scopes to eager-loaded associations when they are nested. (https://github.com/Beyond-Finance/active_force/pull/92)
+
 ## 0.21.0
 
 - Uninitialized attributes will ERROR instead of returning as `nil` (https://github.com/Beyond-Finance/active_force/pull/78)

data/lib/active_force/association/eager_load_builder_for_nested_includes.rb

@@ -62,8 +62,9 @@ module ActiveForce
       private
 
       def build_relation(association, nested_includes)
-        sub_query = Query.new(association.sfdc_association_field)
-        sub_query.fields association.relation_model.fields
+        builder_class = ActiveForce::Association::EagerLoadProjectionBuilder.projection_builder_class(association)
+        projection_builder = builder_class.new(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)
         sub_query.fields nested_includes_query[:fields]

data/lib/active_force/association/eager_load_projection_builder.rb

@@ -6,6 +6,13 @@ module ActiveForce
         def build(association, parent_association_field = nil)
           new(association, parent_association_field).projections
         end
+
+        def projection_builder_class(association)
+          klass = association.class.name.demodulize
+          ActiveForce::Association.const_get "#{klass}ProjectionBuilder"
+        rescue NameError
+          raise "No projection builder exists for #{klass}"
+        end
       end
 
       attr_reader :association, :parent_association_field
@@ -16,12 +23,10 @@ module ActiveForce
       end
 
       def projections
-        klass = association.class.name.split('::').last
-        builder_class = ActiveForce::Association.const_get "#{klass}ProjectionBuilder"
+        builder_class = self.class.projection_builder_class(association)
         builder_class.new(association, parent_association_field).projections
-      rescue NameError
-        raise "Don't know how to build projections for #{klass}"
       end
+
     end
 
     class AbstractProjectionBuilder
@@ -42,26 +47,28 @@ module ActiveForce
 
         query.instance_exec(&association.scoped_as)
       end
-    end
 
-    class HasManyAssociationProjectionBuilder < AbstractProjectionBuilder
       ###
       # Use ActiveForce::Query to build a subquery for the SFDC
       # relationship name. Per SFDC convention, the name needs
       # to be pluralized
-      def projections
+      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
-        ["(#{apply_association_scope(query).to_s})"]
+        apply_association_scope(query)
+      end
+    end
+
+    class HasManyAssociationProjectionBuilder < AbstractProjectionBuilder
+      def projections
+        ["(#{query_with_association_fields.to_s})"]
       end
     end
 
     class HasOneAssociationProjectionBuilder < AbstractProjectionBuilder
       def projections
-        query = ActiveQuery.new(association.relation_model, association.sfdc_association_field)
-        query.fields association.relation_model.fields
-        ["(#{apply_association_scope(query).to_s})"]
+        ["(#{query_with_association_fields.to_s})"]
       end
     end
 

data/lib/active_force/version.rb

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

data/spec/active_force/association_spec.rb

@@ -386,7 +386,7 @@ describe ActiveForce::SObject do
     it 'allows passing a foreign key' do
       Comment.belongs_to :post, foreign_key: :fancy_post_id
       allow(comment).to receive(:fancy_post_id).and_return "2"
-      expect(client).to receive(:query).with("SELECT Id, Title__c, BlogId FROM Post__c WHERE (Id = '2') LIMIT 1")
+      expect(client).to receive(:query).with("SELECT Id, Title__c, BlogId, IsActive FROM Post__c WHERE (Id = '2') LIMIT 1")
       comment.post
       Comment.belongs_to :post # reset association to original value
     end

data/spec/active_force/sobject/includes_spec.rb

@@ -228,7 +228,7 @@ module ActiveForce
         context 'when assocation has a scope' do
           it 'formulates the correct SOQL query with the scope applied' do
             soql = Post.includes(:impossible_comments).where(id: '1234').to_s
-            expect(soql).to eq "SELECT Id, Title__c, BlogId, (SELECT Id, PostId, PosterId__c, FancyPostId, Body__c FROM Comments__r WHERE (1 = 0)) FROM Post__c WHERE (Id = '1234')"
+            expect(soql).to eq "SELECT Id, Title__c, BlogId, IsActive, (SELECT Id, PostId, PosterId__c, FancyPostId, Body__c FROM Comments__r WHERE (1 = 0)) FROM Post__c WHERE (Id = '1234')"
           end
         end
 
@@ -297,7 +297,7 @@ module ActiveForce
         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
-            expect(soql).to eq "SELECT Id, Title__c, BlogId, (SELECT Id, PostId, PosterId__c, FancyPostId, Body__c FROM Comment__r WHERE (NOT ((Body__c = NULL))) ORDER BY CreatedDate DESC) FROM Post__c WHERE (Id = '1234')"
+            expect(soql).to eq "SELECT Id, Title__c, BlogId, IsActive, (SELECT Id, PostId, PosterId__c, FancyPostId, Body__c FROM Comment__r WHERE (NOT ((Body__c = NULL))) ORDER BY CreatedDate DESC) FROM Post__c WHERE (Id = '1234')"
           end
         end
 
@@ -397,9 +397,41 @@ module ActiveForce
         end
       end
 
+      context 'when the associations have scopes' do
+        it 'generates the correct SOQL query' do
+          soql = Blog.includes(active_posts: :impossible_comments).where(id: '123').to_s
+          expect(soql).to eq <<-SOQL.squish
+            SELECT Id, Name, Link__c,
+              (SELECT Id, Title__c, BlogId, IsActive,
+                (SELECT Id, PostId, PosterId__c, FancyPostId, Body__c
+                  FROM Comments__r WHERE (1 = 0))
+                FROM Posts__r
+                WHERE (IsActive = true))
+              FROM Blog__c
+              WHERE (Id = '123')
+          SOQL
+        end
+
+        it 'builds the associated objects and caches them' do
+          response = [build_restforce_sobject({
+            'Id' => '123',
+            'Posts__r' => build_restforce_collection([
+                {'Id' => '213', 'IsActive' => true, 'Comments__r' => [{'Id' => '987'}]},
+                {'Id' => '214', 'IsActive' => true, 'Comments__r' => [{'Id' => '456'}]}
+              ])
+          })]
+          allow(client).to receive(:query).once.and_return response
+          blog = Blog.includes(active_posts: :impossible_comments).find '123'
+          expect(blog.active_posts).to be_an Array
+          expect(blog.active_posts.all? { |o| o.is_a? Post }).to eq true
+          expect(blog.active_posts.first.impossible_comments.first).to be_a Comment
+          expect(blog.active_posts.first.impossible_comments.first.id).to eq '987'
+        end
+      end
+
       context 'with namespaced sobjects' do
         it 'formulates the correct SOQL query' do
-          soql = Salesforce::Account.includes({partner_opportunities: :owner}).where(id: '123').to_s
+          soql = Salesforce::Account.includes({opportunities: :owner}).where(id: '123').to_s
           expect(soql).to eq <<-SOQL.squish
             SELECT Id, Business_Partner__c,
               (SELECT Id, OwnerId, AccountId, Business_Partner__c, Owner.Id
@@ -417,11 +449,11 @@ module ActiveForce
               {'Id' => '214', 'AccountId' => '123', 'OwnerId' => '321', 'Business_Partner__c' => '123', 'Owner' => {'Id' => '321'}}            ])
           })]
           allow(client).to receive(:query).once.and_return response
-          account = Salesforce::Account.includes({partner_opportunities: :owner}).find '123'
-          expect(account.partner_opportunities).to be_an Array
-          expect(account.partner_opportunities.all? { |o| o.is_a? Salesforce::Opportunity }).to eq true
-          expect(account.partner_opportunities.first.owner).to be_a Salesforce::User
-          expect(account.partner_opportunities.first.owner.id).to eq '321'
+          account = Salesforce::Account.includes({opportunities: :owner}).find '123'
+          expect(account.opportunities).to be_an Array
+          expect(account.opportunities.all? { |o| o.is_a? Salesforce::Opportunity }).to eq true
+          expect(account.opportunities.first.owner).to be_a Salesforce::User
+          expect(account.opportunities.first.owner.id).to eq '321'
         end
       end
 
@@ -631,7 +663,7 @@ module ActiveForce
           soql = Comment.includes(post: :blog).where(id: '123').to_s
           expect(soql).to eq <<-SOQL.squish
             SELECT Id, PostId, PosterId__c, FancyPostId, Body__c,
-              PostId.Id, PostId.Title__c, PostId.BlogId,
+              PostId.Id, PostId.Title__c, PostId.BlogId, PostId.IsActive,
                 PostId.BlogId.Id, PostId.BlogId.Name, PostId.BlogId.Link__c
             FROM Comment__c
             WHERE (Id = '123')

data/spec/support/sobjects.rb

@@ -11,6 +11,7 @@ class Post < ActiveForce::SObject
   self.table_name = "Post__c"
   field :title
   field :blog_id, from: "BlogId"
+  field :is_active, from: "IsActive", as: :boolean
   has_many :comments
   has_many :impossible_comments, model: Comment, scoped_as: ->{ where('1 = 0') }
   has_many :reply_comments, model: Comment, scoped_as: ->(post){ where(body: "RE: #{post.title}").order('CreationDate DESC') }
@@ -25,6 +26,7 @@ class Blog < ActiveForce::SObject
   field :name, from: 'Name'
   field :link, from: 'Link__c'
   has_many :posts
+  has_many :active_posts, model: 'Post', scoped_as: -> { where(is_active: true) }
 end
 class Territory < ActiveForce::SObject
   field :quota_id, from: "Quota__c"
@@ -149,6 +151,7 @@ module Salesforce
   end
   class Account < ActiveForce::SObject
     field :business_partner
+    has_many :opportunities, model: Opportunity
     has_many :partner_opportunities, model: Opportunity, scoped_as: ->(account){ where(business_partner: account.business_partner).includes(:owner) }
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: active_force
 version: !ruby/object:Gem::Version
-  version: 0.21.0
+  version: 0.21.1
 platform: ruby
 authors:
 - Eloy Espinaco
@@ -11,7 +11,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-02-15 00:00:00.000000000 Z
+date: 2024-05-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activemodel