aj/MoneyMap
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix: add null checks in AIReceiptParser to prevent null reference warnings

Browse Source 
Added validation to ensure messageContent is not null or whitespace before
deserializing JSON responses from OpenAI and Claude vision APIs. This fixes
CS8604 warnings about possible null reference arguments.
This commit is contained in:
AJ
2025-11-16 11:53:28 -05:00
parent 56089cc437
commit ea14466f57
1 changed files with 10 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
MoneyMap/Services/AIReceiptParser.cs
View File

@@ -306,6 +306,11 @@ namespace MoneyMap.Services
messageContent = messageContent.Replace("```json", "").Replace("```", "").Trim(); messageContent = messageContent.Replace("```json", "").Replace("```", "").Trim();
} }
if (string.IsNullOrWhiteSpace(messageContent))
{
return new ParsedReceiptData();
}
var parsedData = JsonSerializer.Deserialize<ParsedReceiptData>(messageContent, new JsonSerializerOptions var parsedData = JsonSerializer.Deserialize<ParsedReceiptData>(messageContent, new JsonSerializerOptions
{ {
PropertyNameCaseInsensitive = true PropertyNameCaseInsensitive = true
@@ -388,6 +393,11 @@ namespace MoneyMap.Services
messageContent = messageContent.Replace("```json", "").Replace("```", "").Trim(); messageContent = messageContent.Replace("```json", "").Replace("```", "").Trim();
} }
if (string.IsNullOrWhiteSpace(messageContent))
{
return new ParsedReceiptData();
}
var parsedData = JsonSerializer.Deserialize<ParsedReceiptData>(messageContent, new JsonSerializerOptions var parsedData = JsonSerializer.Deserialize<ParsedReceiptData>(messageContent, new JsonSerializerOptions
{ {
PropertyNameCaseInsensitive = true PropertyNameCaseInsensitive = true