Mirrors/CliFx
1
0
Fork 0
mirror of https://github.com/Tyrrrz/CliFx.git synced 2025-10-25 15:19:17 +00:00
Code Issues Packages Projects Releases Wiki Activity

Throw exception when an option has multiple values but the target type is not an array

Browse Source
This commit is contained in:
Alexey Golub
2019-06-09 22:14:01 +03:00
parent 63d798977d
commit 1543076bf4
1 changed files with 8 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
CliFx/Services/CommandOptionConverter.cs
View File

@@ -228,13 +228,19 @@ namespace CliFx.Services
throw new CommandOptionConvertException(
$"Can't convert sequence of values [{option.Values.JoinToString(", ")}] to type [{targetType}].");
}
else
else if (option.Values.Count <= 1)
{
// Take first value and ignore the rest
var value = option.Values.FirstOrDefault();
var value = option.Values.SingleOrDefault();
return ConvertValue(value, targetType);
}
else
{
// TODO: better exception
throw new CommandOptionConvertException(
$"Can't convert sequence of values [{option.Values.JoinToString(", ")}] to type [{targetType}].");
}
}
}
}