forest_liana 7.0.0 → 7.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 22a57de11c5ddee0b53cba1df6f74bebe5cb049534b7eabe58e8c77b5f7e8454
-  data.tar.gz: 3c26aeac71c9b1aab9e5fcd76434136e8a0779e88433f2522c1e39a18b44442c
+  metadata.gz: 66ccf63de4bb8113b724cd22c63957431d50f53b5b5db53d1f48c8d9aa2e6b06
+  data.tar.gz: 134e434c2ad58ba2fa80253625e770b1b0c541f33784c9f184d666c14a75c77a
 SHA512:
-  metadata.gz: ab71e9560ee8823ad87a3db7cb47ea142f020385dbe33b1679223b09f2190b98e30166864ac3765b58bbca1b8181036d5bd59dcfb616ef5b764a66d89f5d1138
-  data.tar.gz: 3d9c7dd145ec8616b85df2d7f11c27e65dc5311b5e4e3b03f427143f4a0a906f14cd1b797d840c4830147ace5866d2c5840afc377c36ebfd58412286ecc7e8dd
+  metadata.gz: e45967d23524fa78600a52fbe2407d7c076cb3fea24d89c4705ca22ba9836cad534eeafc0a506e061279b57fb3529069f2ffe64c1d3c1a63ca44296e41f63db1
+  data.tar.gz: afc9d99afc6d83b3fed291fd9ccf508104bb30c0631506f174a367e1a762868fc2568fb42369b141b28c47b72c9b774a2068aad38ed5d3523497f1babb244501

data/app/services/forest_liana/filters_parser.rb

@@ -109,16 +109,7 @@ module ForestLiana
       parsed_value = parse_value(operator, value)
       field_and_operator = "#{parsed_field} #{parsed_operator}"
 
-      # parenthesis around the parsed_value are required to make the `IN` operator work
-      # and have no side effects on other requests
-      if Rails::VERSION::MAJOR < 5
-        "#{field_and_operator} (#{ActiveRecord::Base.sanitize(parsed_value)})"
-      # NOTICE: sanitize method as been removed in Rails 5.1 and sanitize_sql introduced in Rails 5.2.
-      elsif Rails::VERSION::MAJOR == 5 && Rails::VERSION::MINOR == 1
-        "#{field_and_operator} (#{ActiveRecord::Base.connection.quote(parsed_value)})"
-      else
-        ActiveRecord::Base.sanitize_sql(["#{field_and_operator} (?)", parsed_value])
-      end
+      sanitize_condition(field_and_operator, operator, parsed_value)
     end
 
     def parse_aggregation_operator(aggregator_operator)
@@ -296,5 +287,26 @@ module ForestLiana
         raise ForestLiana::Errors::HTTP422Error.new('Invalid condition format')
       end
     end
+
+    private
+
+    def prepare_value_for_operator(operator, value)
+      # parenthesis around the parsed_value are required to make the `IN` operator work
+      operator == 'in' ? "(#{value})" : value
+    end
+
+    def sanitize_condition(field_and_operator, operator, parsed_value)
+      if Rails::VERSION::MAJOR < 5
+        condition_value = prepare_value_for_operator(operator, ActiveRecord::Base.sanitize(parsed_value))
+        "#{field_and_operator} #{condition_value}"
+        # NOTICE: sanitize method as been removed in Rails 5.1 and sanitize_sql introduced in Rails 5.2.
+      elsif Rails::VERSION::MAJOR == 5 && Rails::VERSION::MINOR == 1
+        condition_value = prepare_value_for_operator(operator, ActiveRecord::Base.connection.quote(parsed_value))
+        "#{field_and_operator} #{condition_value}"
+      else
+        condition_value = prepare_value_for_operator(operator, '?')
+        ActiveRecord::Base.sanitize_sql(["#{field_and_operator} #{condition_value}", parsed_value])
+      end
+    end
   end
 end

data/lib/forest_liana/version.rb

@@ -1,3 +1,3 @@
 module ForestLiana
-  VERSION = "7.0.0"
+  VERSION = "7.0.1"
 end

data/spec/services/forest_liana/filters_parser_spec.rb

@@ -11,6 +11,7 @@ module ForestLiana
     let(:date_condition_1) { { 'field' => 'created_at', 'operator' => 'before', 'value' => 2.hours.ago } }
     let(:date_condition_2) { { 'field' => 'created_at', 'operator' => 'today' } }
     let(:date_condition_3) { { 'field' => 'created_at', 'operator' => 'previous_x_days', 'value' => 2 } }
+    let(:presence_condition) { { 'field' => 'name', 'operator' => 'present' } }
 
     before {
       island = Island.create!(name: "L'île de la muerta")
@@ -274,6 +275,11 @@ module ForestLiana
         let(:filters) { { 'aggregator' => 'or', 'conditions' => [simple_condition_2, simple_condition_3] } }
         it { expect(resource.where(query).count).to eq 2 }
       end
+
+      context "'name ends_with \"3\"' 'or' 'name is not null'" do
+        let(:filters) { { 'aggregator' => 'or', 'conditions' => [simple_condition_2, presence_condition] } }
+        it { expect(resource.where(query).count).to eq 3 }
+      end
     end
 
     describe 'parse_condition' do
@@ -281,7 +287,27 @@ module ForestLiana
       let(:result) { filter_parser.parse_condition(condition) }
 
       context 'on valid condition' do
-        it { expect(result).to eq "\"trees\".\"name\" LIKE ('%3')" }
+        context 'when the condition uses the contains operator' do
+          it { expect(result).to eq "\"trees\".\"name\" LIKE '%3'" }
+        end
+
+        context 'when the condition uses the blank operator' do
+          let(:condition) { { 'field' => 'name', 'operator' => 'blank' } }
+
+          it { expect(result).to eq "\"trees\".\"name\" IS NULL" }
+        end
+
+        context 'when the condition uses the presence operator' do
+          let(:condition) { presence_condition }
+
+          it { expect(result).to eq "\"trees\".\"name\" IS NOT NULL" }
+        end
+
+        context 'when the condition uses the in operator' do
+          let(:condition) { { 'field' => 'name', 'operator' => 'in', 'value' => ['Tree n1', 'Tree n3'] } }
+
+          it { expect(result).to eq "\"trees\".\"name\" IN ('Tree n1','Tree n3')" }
+        end
       end
 
       context 'on belongs_to condition' do

data/spec/services/forest_liana/resource_updater_spec.rb

@@ -84,7 +84,7 @@ module ForestLiana
           subject.perform
 
           expect(subject.record).to be nil
-          expect(subject.errors[0][:detail]).to eq 'Couldn\'t find User with \'id\'=1 [WHERE (("users"."id" > (2)))]'
+          expect(subject.errors[0][:detail]).to eq 'Couldn\'t find User with \'id\'=1 [WHERE (("users"."id" > 2))]'
         end
       end
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: forest_liana
 version: !ruby/object:Gem::Version
-  version: 7.0.0
+  version: 7.0.1
 platform: ruby
 authors:
 - Sandro Munda
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-07-20 00:00:00.000000000 Z
+date: 2021-07-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails