Akatsuki-Misaki/ArchiSteamFarm
1
0
Fork 0
mirror of https://github.com/JustArchiNET/ArchiSteamFarm.git synced 2026-01-01 22:20:52 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix parsing tags in GetActiveTradeOffers (#2060)

Browse Source 
* Fix parsing tags in GetActiveTradeOffers

* Accept empty values in Tag constructor

* Throw on null value in constructor

* Fix mindless copypaste
This commit is contained in:
Vitaliy
2020-11-21 17:42:19 +03:00
committed by GitHub
parent 6a8a3b9c8a
commit 3ac04c6aaf
2 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
ArchiSteamFarm/ArchiWebHandler.cs
View File

@@ -1527,7 +1527,8 @@ namespace ArchiSteamFarm {
string? value = tag["internal_name"].AsString();
if (string.IsNullOrEmpty(value)) {
// Apparently, name can be empty, but not null
if (value == null) {
Bot.ArchiLogger.LogNullError(nameof(value));
return null;

2
ArchiSteamFarm/Json/Steam.cs
View File

@@ -261,7 +261,7 @@ namespace ArchiSteamFarm.Json {
internal Tag(string identifier, string value) {
Identifier = !string.IsNullOrEmpty(identifier) ? identifier : throw new ArgumentNullException(nameof(identifier));
Value = !string.IsNullOrEmpty(value) ? value : throw new ArgumentNullException(nameof(value));
Value = value ?? throw new ArgumentNullException(nameof(value));
}
[JsonConstructor]