Revert "fix(filters): properly parse filters from request params" (#746)

Revert "fix(filters): properly parse filters from request params (#743)"

This reverts commit 801f5d1.

Author: matthv

app/services/forest_liana/resources_getter.rb

@@ -198,29 +198,15 @@ def field_names_requested
 
     def extract_associations_from_filter
       associations = []
-
-      filters = @params[:filters]
-      filters = JSON.parse(filters) if filters.is_a?(String)
-
-      conditions = []
-
-      if filters.is_a?(Hash) && filters.key?('conditions')
-        conditions = filters['conditions']
-      elsif filters.is_a?(Hash) && filters.key?('field')
-        conditions = [filters]
-      end
-
-      conditions.each do |condition|
-        field = condition['field']
-        if field&.include?(':')
+      @params[:filter]&.each do |field, _|
+        if field.include?(':')
           associations << field.split(':').first.to_sym
           @count_needs_includes = true
         end
       end
-
       @count_needs_includes = true if @params[:search]
 
-      associations.uniq
+      associations
     end
 
     def prepare_query

spec/services/forest_liana/resources_getter_spec.rb

@@ -309,7 +309,7 @@ def association_connection.current_database
     describe 'when filtering on an ambiguous field' do
       let(:resource) { Tree }
       let(:pageSize) { 5 }
-      let(:fields) { { 'Tree' => 'id,name', 'cutter' => 'id' } }
+      let(:fields) { { 'Tree' => 'id' } }
       let(:filters) { {
         aggregator: 'and',
         conditions: [{
@@ -332,6 +332,7 @@ def association_connection.current_database
         expect(count).to eq 1
         expect(records.first.id).to eq 3
         expect(records.first.name).to eq 'Apple Tree'
+        expect(records.first.cutter.name).to eq 'Michel'
       end
     end
 
@@ -378,7 +379,7 @@ def association_connection.current_database
     describe 'when sorting on an ambiguous field name with a filter' do
       let(:resource) { Tree }
       let(:sort) { '-name' }
-      let(:fields) { { 'Tree' => 'id,name' } }
+      let(:fields) { { 'Tree' => 'id' } }
       let(:filters) { {
         field: 'cutter:name',
         operator: 'equal',
@@ -416,7 +417,7 @@ def association_connection.current_database
 
     describe 'when filtering on an updated_at field of an associated collection' do
       let(:resource) { Tree }
-      let(:fields) { { 'Tree' => 'id,name' } }
+      let(:fields) { { 'Tree' => 'id' } }
       let(:filters) { {
         field: 'island:updated_at',
         operator: 'previous_year'
@@ -591,64 +592,6 @@ def association_connection.current_database
         end
       end
 
-      describe '#extract_associations_from_filter' do
-        let(:resource) { Tree }
-
-        before { init_scopes }
-
-        context 'with a single filter as JSON string' do
-          let(:filters) {
-            {
-              field: 'island:updated_at',
-              operator: 'equal',
-              value: '2024-01-01'
-            }.to_json
-          }
-
-          it 'extracts the correct association' do
-            expect(getter.send(:extract_associations_from_filter)).to eq [:island]
-          end
-        end
-
-        context 'with grouped conditions as JSON string' do
-          let(:filters) {
-            {
-              aggregator: 'and',
-              conditions: [
-                { field: 'island:updated_at', operator: 'equal', value: '2024-01-01' },
-                { field: 'owner:name', operator: 'equal', value: 'Michel' },
-                { field: 'id', operator: 'present', value: nil }
-              ]
-            }.to_json
-          }
-
-          it 'extracts all unique associations' do
-            expect(getter.send(:extract_associations_from_filter)).to match_array [:island, :owner]
-          end
-        end
-
-        context 'when filters has no association field' do
-          let(:filters) {
-            {
-              field: 'id',
-              operator: 'equal',
-              value: 1
-            }.to_json
-          }
-
-          it 'returns an empty array' do
-            expect(getter.send(:extract_associations_from_filter)).to eq []
-          end
-        end
-
-        context 'when filters is nil' do
-          let(:filters) { nil }
-
-          it 'returns an empty array' do
-            expect(getter.send(:extract_associations_from_filter)).to eq []
-          end
-        end
-      end
     end
   end
 end