Skip to content

Commit e3365f2

Browse files
committed
style: negate the TryGetValue check instead of comparing it to false
S1125, introduced by the previous commit — `is false` on a bool method call reads as a puzzle where `!` reads as the sentence.
1 parent c2dc8a3 commit e3365f2

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

src/UI/NeoReports.UI/Services/BuilderConfigMapper.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -442,7 +442,7 @@ private static JsonArray BuildOutputs(BuilderState state, JsonArray? original)
442442
// size from 90 to 120 must not turn the property into the string "120". Only untouched rows
443443
// were protected before, so the very coercion the round-trip fixed came back the moment
444444
// someone edited the value.
445-
if (stored is JsonValue previous && previous.TryGetValue(out string? _) is false)
445+
if (stored is JsonValue previous && !previous.TryGetValue(out string? _))
446446
{
447447
if (previous.TryGetValue(out long _) && long.TryParse(row.Value, NumberStyles.Integer, CultureInfo.InvariantCulture, out long asLong))
448448
return JsonValue.Create(asLong);

0 commit comments

Comments
 (0)