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

Don't ignore case in short names

Browse Source
This commit is contained in:
Alexey Golub
2019-08-13 18:38:24 +03:00
parent 86fdf72d9c
commit 384482a47c
1 changed files with 6 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
CliFx/Models/Extensions.cs
View File

@@ -14,8 +14,8 @@ namespace CliFx.Models
var firstOptionAlias = commandInput.Options.FirstOrDefault()?.Alias;
return string.Equals(firstOptionAlias, "help", StringComparison.OrdinalIgnoreCase) ||
string.Equals(firstOptionAlias, "h", StringComparison.OrdinalIgnoreCase) ||
string.Equals(firstOptionAlias, "?", StringComparison.OrdinalIgnoreCase);
string.Equals(firstOptionAlias, "h", StringComparison.Ordinal) ||
string.Equals(firstOptionAlias, "?", StringComparison.Ordinal);
}
public static bool IsVersionRequested(this CommandInput commandInput)
@@ -55,13 +55,16 @@ namespace CliFx.Models
{
foreach (var optionSchema in optionSchemas)
{
// Compare against name. Case is ignored.
var matchesByName =
!optionSchema.Name.IsNullOrWhiteSpace() &&
string.Equals(optionSchema.Name, alias, StringComparison.OrdinalIgnoreCase);
// Compare against short name. Case is NOT ignored.
var matchesByShortName =
optionSchema.ShortName != null &&
string.Equals(optionSchema.ShortName.Value.AsString(), alias, StringComparison.OrdinalIgnoreCase);
alias.Length == 1 &&
alias[0] == optionSchema.ShortName;
if (matchesByName || matchesByShortName)
return optionSchema;