forest_liana 9.20.4 → 9.20.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e11078e22f2bf038e25056074bc9389f3335eb43dd2369e114c2df52dbe3d6b0
-  data.tar.gz: 44d80d3b1c9074ba4c2992ae1c45d3cb7b67e764ab940e0f428f1d14bebcd2e2
+  metadata.gz: 31b55dea3884f93640870ded7c0f4079cea1792d7842b05c0b3d1c610a463d0b
+  data.tar.gz: bc7faa9c4f0302159f57d5e1e376d697047fd0a0c2ad1c94f3fb651078a87395
 SHA512:
-  metadata.gz: 11f3ba2356911a83238bf2e2186cede6746c6079a4ab59add90288dcb3c92d2e0f6a795e776433b96b6548a6857b4a7e0b210f3711a0304dcb70461a00455a4a
-  data.tar.gz: 7e6b38924b464bcc9ba5ea9a617f860f0ef29e7b5c4e14920752e9059b31c2839a3ca1e374f5d2bf089c0e791cb5c17639b7bff7986f17e0444279512416fb01
+  metadata.gz: 2e707a5df3a857e9bbb61c0cefa2d593feae5b724f05b3d18f45df2a0abdcc724d6db0367b514e89d9074c5646b8994d2c2d0330735b9da8e4a3810f5460317d
+  data.tar.gz: f757f804039e32d4ef70b0b544ca1297933d633ef063ff692fa79c5fc8d7ee5daeee817a859956c7ed4fcfaaab966ef2fc6dd70d47061cbcd1218e059136bcad

data/app/controllers/forest_liana/associations_controller.rb

@@ -27,6 +27,11 @@ module ForestLiana
     def count
       find_resource
       find_association
+      # NOTICE: find_association renders a 404 (without halting) when the
+      #         association name does not resolve; stop here instead of falling
+      #         through and dereferencing a nil @association, which surfaced as
+      #         a double-render / 500 rather than the intended 404.
+      return if performed?
       begin
         getter = HasManyGetter.new(@resource, @association, params, forest_user)
         getter.count

data/app/services/forest_liana/pie_stat_getter.rb

@@ -33,7 +33,9 @@ module ForestLiana
               key = @resource.defined_enums[@params[:groupByFieldName]].invert[key]
             elsif @resource.columns_hash[@params[:groupByFieldName]] &&
               @resource.columns_hash[@params[:groupByFieldName]].type == :datetime
-              key = (key + timezone_offset.hours).strftime('%d/%m/%Y %T')
+              # NOTICE: A nil datetime group key would raise NoMethodError on
+              #         `nil + timezone_offset.hours`; label it explicitly.
+              key = key.present? ? (key + timezone_offset.hours).strftime('%d/%m/%Y %T') : 'No value'
             end
 
             { key: key, value: value }

data/lib/forest_liana/version.rb

@@ -1,3 +1,3 @@
 module ForestLiana
-  VERSION = "9.20.4"
+  VERSION = "9.20.6"
 end

data/spec/requests/count_spec.rb

@@ -74,6 +74,20 @@ describe 'Requesting Owner', :type => :request  do
     end
   end
 
+  describe 'count on an unknown relationship' do
+    params = {
+      fields: { 'Tree' => 'id,name,owner' },
+      page: { 'number' => '1', 'size' => '10' },
+      searchExtended: '0',
+      timezone: 'Europe/Paris'
+    }
+
+    it 'should respond 404 instead of 500' do
+      get '/forest/Owner/5/relationships/unknown_things/count', params: params, headers: headers
+      expect(response.status).to eq(404)
+    end
+  end
+
   describe 'deactivate_count_response' do
     params = {
       fields: { 'Owner' => 'id,name' },

data/spec/services/forest_liana/pie_stat_getter_spec.rb

@@ -44,6 +44,33 @@ module ForestLiana
       end
     end
 
+    describe 'grouping by a nullable datetime field with a null value' do
+      let(:scopes) { {'scopes' => {}, 'team' => {'id' => '1', 'name' => 'Operations'}} }
+      let(:model) { Tree }
+      let(:collection) { 'trees' }
+      let(:params) {
+        {
+          type: 'Pie',
+          sourceCollectionName: collection,
+          timezone: 'Europe/Paris',
+          aggregator: 'Count',
+          groupByFieldName: 'created_at'
+        }
+      }
+
+      subject { PieStatGetter.new(model, params, user) }
+
+      before do
+        tree = Tree.create!(name: 'No date tree', age: 1)
+        tree.update_columns(created_at: nil)
+      end
+
+      it 'labels the null bucket "No value" instead of raising on nil + offset' do
+        expect { subject.perform }.not_to raise_error
+        expect(subject.record.value.map { |entry| entry[:key] }).to include('No value')
+      end
+    end
+
     describe 'with valid aggregate function' do
       let(:model) { Tree }
       let(:collection) { 'trees' }

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: forest_liana
 version: !ruby/object:Gem::Version
-  version: 9.20.4
+  version: 9.20.6
 platform: ruby
 authors:
 - Sandro Munda
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2026-07-31 00:00:00.000000000 Z
+date: 2026-08-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails