From 38f6bec557d0e9e1a146aab939d150421fa29ea4 Mon Sep 17 00:00:00 2001 From: Kuba Suder Date: Fri, 14 Aug 2026 21:51:16 +0200 Subject: [PATCH] =?UTF-8?q?actually=20check=20if=20fields=20aren't=20nil?= =?UTF-8?q?=20in=20check=5Fif=5Fnot=5Fnil=20=F0=9F=AB=A0?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- lib/skyfall/firehose/message.rb | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/lib/skyfall/firehose/message.rb b/lib/skyfall/firehose/message.rb index 1882cdd..7575073 100644 --- a/lib/skyfall/firehose/message.rb +++ b/lib/skyfall/firehose/message.rb @@ -157,13 +157,13 @@ module Skyfall # Note: this method is written this way as an optimization def check_if_not_nil(a, b = nil, c = nil, d = nil, e = nil, f = nil, g = nil) - ok = @data_object.has_key?(a) - ok &&= @data_object.has_key?(b) if b - ok &&= @data_object.has_key?(c) if c - ok &&= @data_object.has_key?(d) if d - ok &&= @data_object.has_key?(e) if e - ok &&= @data_object.has_key?(f) if f - ok &&= @data_object.has_key?(g) if g + ok = !@data_object[a].nil? + ok &&= !@data_object[b].nil? if b + ok &&= !@data_object[c].nil? if c + ok &&= !@data_object[d].nil? if d + ok &&= !@data_object[e].nil? if e + ok &&= !@data_object[f].nil? if f + ok &&= !@data_object[g].nil? if g if !ok expected_fields = [a, b, c, d, e, f, g].compact -- 2.51.2