forest_admin_datasource_active_record 1.0.0.pre.beta.54 → 1.0.0.pre.beta.56

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 872ac653d0f9a281cab0421a9f01343ab3ee751ab12dd89d07aaabc870ecd91e
-  data.tar.gz: dc666fc765acabfa4ce22fccd2e3240f4c4ba308884d2b77754b6ffd33adf85d
+  metadata.gz: 0d6fb52bad1ecc5a281b51414badded38a6c4ff12eb0fa5d31b9daf6acea83bc
+  data.tar.gz: 8fd24a8be2d94a2dd617ab3e65cbe958bb2e16d345f3ad15a89f66019e78df31
 SHA512:
-  metadata.gz: 613b29a6e6f7119ecc1d33f2955b5bf3fdf69999a4d6a2d7066c1a423b4bff4af630dc8dd4aefdf2fdc88a4f2393da8dcb5edc379ab54f89ba00e00ab158c512
-  data.tar.gz: fc7ca9cc8868bdf8aca89735d1c22c550bdb45b962054958dd1cc8c01a2cd0ec8022c67dd27c1e7ea3c5c05f51442cbd7d8f22ff6e417eefd5352c667296dcb2
+  metadata.gz: 67d89437e303f22b0e860c5753b50d225a59949947d6fa4135ce5332bc2609a648bbc424b0c18d173b379d10f40338a10252872245917726129f659ef9bf7afb
+  data.tar.gz: 7994813a226847201d9aab32d68250c17216bf9e471970fba73b8ee06d42c45557e50af02ecb8f3a614feb4f735a0cb796a58b11cda46722d58246396a8c75a4

data/lib/forest_admin_datasource_active_record/utils/query.rb

@@ -50,39 +50,41 @@ module ForestAdminDatasourceActiveRecord
 
       def apply_condition_tree(condition_tree, aggregator = nil)
         if condition_tree.is_a? Nodes::ConditionTreeBranch
-          aggregator = condition_tree.aggregator.downcase
+          aggregator = condition_tree.aggregator.downcase.to_sym
           condition_tree.conditions.each do |condition|
-            query = apply_condition_tree(condition, aggregator)
-            @query = @query.send(aggregator, query)
+            @query = apply_condition_tree(condition, aggregator)
+            @query = @query.send(aggregator, @query)
           end
 
           @query
         else
-          compute_main_operator(condition_tree, aggregator || 'and')
+          @query = compute_main_operator(condition_tree, aggregator || :and)
         end
       end
 
       def compute_main_operator(condition_tree, aggregator)
         field = format_field(condition_tree.field)
         value = condition_tree.value
+        aggregator = aggregator.to_sym
 
         case condition_tree.operator
         when Operators::PRESENT
-          @query = @query.send(aggregator, @query.where.not({ field => nil }))
+          @query = query_aggregator(aggregator, @collection.model.where.not({ field => nil }))
         when Operators::EQUAL, Operators::IN
-          @query = @query.send(aggregator, @query.where({ field => value }))
+          @query = query_aggregator(aggregator, @collection.model.where({ field => value }))
         when Operators::NOT_EQUAL, Operators::NOT_IN
-          @query = @query.send(aggregator, @query.where.not({ field => value }))
+          @query = query_aggregator(aggregator, @collection.model.where.not({ field => value }))
         when Operators::GREATER_THAN
-          @query = @query.send(aggregator, @query.where(@arel_table[field.to_sym].gt(value)))
+          @query = query_aggregator(aggregator, @collection.model.where(@arel_table[field.to_sym].gt(value)))
         when Operators::LESS_THAN
-          @query = @query.send(aggregator, @query.where(@arel_table[field.to_sym].lt(value)))
+          @query = query_aggregator(aggregator, @collection.model.where(@arel_table[field.to_sym].lt(value)))
         when Operators::NOT_CONTAINS
-          @query = @query.send(aggregator, @query.where.not(@arel_table[field.to_sym].matches("%#{value}%")))
+          @query = query_aggregator(aggregator,
+                                    @collection.model.where.not(@arel_table[field.to_sym].matches("%#{value}%")))
         when Operators::LIKE
-          @query = @query.send(aggregator, @query.where(@arel_table[field.to_sym].matches(value)))
+          @query = query_aggregator(aggregator, @collection.model.where(@arel_table[field.to_sym].matches(value)))
         when Operators::INCLUDES_ALL
-          @query = @query.send(aggregator, @query.where(@arel_table[field.to_sym].matches_all(value)))
+          @query = query_aggregator(aggregator, @collection.model.where(@arel_table[field.to_sym].matches_all(value)))
         end
 
         @query
@@ -93,7 +95,7 @@ module ForestAdminDatasourceActiveRecord
           @select += @projection.columns.map { |field| "#{@collection.model.table_name}.#{field}" }
           @projection.relations.each_key do |relation|
             relation_schema = @collection.schema[:fields][relation]
-            @select << if relation_schema.type == 'OneToOne'
+            @select << if relation_schema.type == 'OneToOne' || relation_schema.type == 'PolymorphicOneToOne'
                          "#{@collection.model.table_name}.#{relation_schema.origin_key_target}"
                        else
                          "#{@collection.model.table_name}.#{relation_schema.foreign_key}"
@@ -107,33 +109,41 @@ module ForestAdminDatasourceActiveRecord
       def apply_select
         unless @projection.nil?
           @query = @query.select(@select.join(', '))
-          @query = @query.joins(@projection.relations.keys.map(&:to_sym))
+          @query = @query.includes(@projection.relations.keys.map(&:to_sym))
         end
 
         @query
       end
 
-      def add_join_relation(relation, relation_name)
-        if relation.type == 'ManyToMany'
-          # TODO: to implement
-        else
-          @query = @query.joins(relation_name.to_sym)
-        end
+      def add_join_relation(relation_name)
+        @query = @query.includes(relation_name.to_sym)
 
         @query
       end
 
       def format_field(field)
+        @select << "#{@collection.model.table_name}.#{field}"
+
         if field.include?(':')
           relation_name, field = field.split(':')
           relation = @collection.schema[:fields][relation_name]
           table_name = @collection.datasource.get_collection(relation.foreign_collection).model.table_name
-          add_join_relation(relation, relation_name)
+          add_join_relation(relation_name)
+          @select << "#{table_name}.#{field}"
+
           return "#{table_name}.#{field}"
         end
 
         field
       end
+
+      def query_aggregator(aggregator, query)
+        if !@query.respond_to?(:where_clause) || @query.where_clause.empty?
+          query
+        else
+          @query.send(aggregator, query)
+        end
+      end
     end
   end
 end

data/lib/forest_admin_datasource_active_record/version.rb

@@ -1,3 +1,3 @@
 module ForestAdminDatasourceActiveRecord
-  VERSION = "1.0.0-beta.54"
+  VERSION = "1.0.0-beta.56"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: forest_admin_datasource_active_record
 version: !ruby/object:Gem::Version
-  version: 1.0.0.pre.beta.54
+  version: 1.0.0.pre.beta.56
 platform: ruby
 authors:
 - Matthieu
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2024-06-10 00:00:00.000000000 Z
+date: 2024-06-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord