ree_lib 1.0.59 → 1.0.60

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ae96c3d6f51872601ad8c4a282cc8119e069a49a2942e624a3eed02817bbf450
-  data.tar.gz: 962693ff78a273ff60ab20f1e704759038e0402619c84ce2da5735f16cca28ad
+  metadata.gz: 77d773db4104deb076fe0ea9d9aa6b8e0fd1218d71731e6d51a70a50be09d569
+  data.tar.gz: ac943abd83280ff44db19e50991e1ca51aa92cd1f140ccaa9e82cf706fc9dc8e
 SHA512:
-  metadata.gz: 562c425026b16d0389287831ea2da44c08e79818c4f238e52f8e3ee0bb62bf36bc8e4bbfc75e3dbe658706a0b735c4c3e0e572287048e9a3095055e7183bfa62
-  data.tar.gz: 54c6f30b6c62de9a43284fc76fcf432e9b0c70691de5d0aaf074c8b4cdb9b889f816ee8219c2ba4e660684899793db9a1c3cd7f502f5567fbe32d9df105b7a67
+  metadata.gz: 6eb4b4567e977e12c59f5246f819dcba9702ccb77fb5e776584f193f6cff18c14d4e7e40d75c0377b0fc905720936f0feb888e2e953b288466f7d777ed7bd633
+  data.tar.gz: 26ee7d50cf9e346c4cb9614832282d586d0ac767dcd0b65bdb056511ed2e3058f5d1186003e6f30b89a9a661768c7b7153e4d2a3f079185b441f9b7ca5582485

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    ree_lib (1.0.59)
+    ree_lib (1.0.60)
       binding_of_caller (~> 1.0.0)
       i18n (~> 1.12.0)
       loofah (~> 2.18.0)

data/lib/ree_lib/packages/ree_dao/package/ree_dao/association.rb

@@ -160,32 +160,19 @@ module ReeDao
           root_ids = list.map(&:id).uniq
         else
           if !foreign_key
-            dto_class = assoc_dao
-              .opts[:schema_mapper]
-              .dto(:db_load)
-    
-            name = underscore(demodulize(dto_class.name))
-            
-            root_ids = list.map(&:"#{"#{name}_id".to_sym}").uniq
-            foreign_key = :id
-          else
-            root_ids = list.map(&:"#{foreign_key}")
-            foreign_key = :id
+            foreign_key = :"#{assoc_name}_id"
           end
+
+          root_ids = list.map(&:"#{foreign_key}").compact
         end
   
-        default_scope = if !scope 
-          assoc_dao&.where(foreign_key => root_ids)
-        end
-  
-        items = add_scopes(default_scope, scope, global_opts[assoc_name])
+        scope ||= assoc_dao
+        scope = scope.where((reverse ? foreign_key : :id) => root_ids)
+        
+        items = add_scopes(scope, global_opts[assoc_name])
       end
 
-      assoc = if foreign_key
-        index_by(items) { _1.send(foreign_key) }
-      else
-        items
-      end 
+      assoc = index_by(items) { _1.send(reverse ? foreign_key : :id) }
 
       populate_association(
         list,
@@ -224,11 +211,10 @@ module ReeDao
   
         root_ids = list.map(&:"#{primary_key}")
   
-        default_scope = if !scope
-          assoc_dao&.where(foreign_key => root_ids)
-        end
+        scope ||= assoc_dao
+        scope = scope.where(foreign_key => root_ids)
   
-        items = add_scopes(default_scope, scope, global_opts[assoc_name])
+        items = add_scopes(scope, global_opts[assoc_name])
       end
 
       assoc = if foreign_key
@@ -289,12 +275,12 @@ module ReeDao
       end
     end
 
-    contract(Nilor[Sequel::Dataset], Nilor[Sequel::Dataset], Nilor[Proc] => Array)
-    def add_scopes(default_scope, scope, named_scope)
-      res = scope || default_scope
-
-      if named_scope
-        res = named_scope.call(res)
+    contract(Nilor[Sequel::Dataset], Nilor[Proc] => Array)
+    def add_scopes(scope, named_scope)
+      res = if named_scope
+        named_scope.call(scope)
+      else
+        scope
       end
 
       res.all

data/lib/ree_lib/packages/ree_dao/spec/ree_dao/functions/load_agg/load_agg_spec.rb

@@ -278,6 +278,7 @@ RSpec.describe :load_agg do
         belongs_to :organization
 
         has_many :books, -> { books_opts(title) }
+        has_many :active_books, books
       end
     end
 
@@ -508,6 +509,45 @@ RSpec.describe :load_agg do
     expect(res[0].books[1].author).to eq(nil)
   }
 
+  it {
+    organizations.delete_all
+    users.delete_all
+    user_passports.delete_all
+    books.delete_all
+    chapters.delete_all
+
+    organization = ReeDaoLoadAggTest::Organization.new(name: "Test Org")
+    organizations.put(organization)
+
+    user_1 = ReeDaoLoadAggTest::User.new(name: "John", age: 33, organization_id: organization.id)
+    user_2 = ReeDaoLoadAggTest::User.new(name: "Sam", age: 21, organization_id: organization.id)
+    users.put(user_1)
+    users.put(user_2)
+
+    book_1 = ReeDaoLoadAggTest::Book.new(user_id: user_1.id, title: "1984")
+    book_2 = ReeDaoLoadAggTest::Book.new(user_id: user_1.id, title: "1408")
+
+    books.put(book_1)
+    books.put(book_2)
+
+    res = users_agg.call(
+      users.all
+    )
+
+    expect(res[0].books).to_not eq([])
+    expect(res[1].books).to eq([])
+
+    books.delete(book_1)
+    books.delete(book_2)
+
+    res = users_agg.call(
+      users.all
+    )
+
+    expect(res[0].books).to eq([])
+    expect(res[1].books).to eq([])
+  }
+
   it {
     organizations.delete_all
     users.delete_all
@@ -533,6 +573,7 @@ RSpec.describe :load_agg do
     expect(res_user.id).to eq(user_1.id)
     expect(res_user.organization).to eq(organization)
     expect(res_user.books.count).to eq(1)
+    expect(res_user.active_books.count).to eq(2)
   }
 
   it {

data/lib/ree_lib/packages/ree_dao/spec/ree_dao/functions/load_agg/ree_dao_load_agg_test.rb

@@ -105,6 +105,14 @@ class ReeDaoLoadAggTest::User
     @books
   end
 
+  def set_active_books(books)
+    @active_books = books
+  end
+
+  def active_books
+    @active_books
+  end
+
   contract(ArrayOf[ReeDaoLoadAggTest::Book] => nil)
   def set_books(books)
     @books = books; nil

data/lib/ree_lib/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module ReeLib
-  VERSION = "1.0.59"
+  VERSION = "1.0.60"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: ree_lib
 version: !ruby/object:Gem::Version
-  version: 1.0.59
+  version: 1.0.60
 platform: ruby
 authors:
 - Ruslan Gatiyatov