From 26777963220dd13baf0a8756c21df07e593c1b19 Mon Sep 17 00:00:00 2001 From: Jason Kulatunga Date: Sun, 6 Nov 2022 07:54:32 -0800 Subject: [PATCH] fixing bug. Null value for temperatures should be ignored. --- .../backend/pkg/database/scrutiny_repository_temperature.go | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/webapp/backend/pkg/database/scrutiny_repository_temperature.go b/webapp/backend/pkg/database/scrutiny_repository_temperature.go index fb3be6a..9edec87 100644 --- a/webapp/backend/pkg/database/scrutiny_repository_temperature.go +++ b/webapp/backend/pkg/database/scrutiny_repository_temperature.go @@ -17,6 +17,10 @@ func (sr *scrutinyRepository) SaveSmartTemperature(ctx context.Context, wwn stri if len(collectorSmartData.AtaSctTemperatureHistory.Table) > 0 { for ndx, temp := range collectorSmartData.AtaSctTemperatureHistory.Table { + //temp value may be null, we must skip/ignore them. See #393 + if temp == 0 { + continue + } minutesOffset := collectorSmartData.AtaSctTemperatureHistory.LoggingIntervalMinutes * int64(ndx) * 60 smartTemp := measurements.SmartTemperature{