Skip to content

Commit fb6b812

Browse files
committed
Fix for bug in CheckExcludes() and CheckIncludes()
1 parent 42092cb commit fb6b812

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

Source/Menu/Notifications/NotificationManager.cs

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -101,6 +101,8 @@ private string GetRemoteJson()
101101
// Helpful to supply server with data for its log file.
102102
client.Headers[HttpRequestHeader.UserAgent] = $"{System.Windows.Forms.Application.ProductName}/{VersionInfo.VersionOrBuild}";
103103

104+
// Trial for glTD and PBR graphics
105+
//return client.DownloadString(new Uri("https://wepp.co.uk/openrails/notifications/menu_testing_direct3d.json"));
104106
return client.DownloadString(new Uri("https://wepp.co.uk/openrails/notifications/menu.json"));
105107
}
106108

@@ -241,7 +243,7 @@ private Check CheckExcludes(Check check)
241243
{
242244
if (c is Contains)
243245
{
244-
CheckContains(check, c);
246+
return CheckContains(check, c);
245247
}
246248
}
247249
return null;
@@ -259,7 +261,7 @@ private Check CheckIncludes(Check check)
259261

260262
foreach (var c in check.IncludesAnyOf)
261263
{
262-
CheckContains(check, c);
264+
return CheckContains(check, c);
263265
}
264266
return null;
265267
}

0 commit comments

Comments
 (0)