commandlineparser / commandlineparser/commandline

Multi instance setting ignored when default verb is parsed.

Đang mở
#922 0 bình luận 0 reaction 0 người được giao Xem trên GitHub
Ngôn ngữ chính
C#
Star
4.8k
Fork
478
Chỉ số merge pull request
Không có pull request nào được merge trong 30 ngày

Mô tả

**Describe the bug**
When using the function "MapResult" to parse multiple verbs, the parser ignores the setting "AllowMultiInstance" while parsing the default verb.

Lib version: 2.9.1
Language: C#

**To Reproduce**
Test the following demo code to see the result.

Demo Class Options:
```
[Verb("log", isDefault:true, HelpText = "Log command")]
class LogOptions
{
[Option('a', HelpText = "Option A for log command")]
public IEnumerable OptionA { get; set; }

[Option('b', HelpText = "Option B for log command")]
public IEnumerable OptionB { get; set; }
}

[Verb("save", HelpText = "Save command")]
class SaveOptions
{
[Option('d', HelpText = "Option D for save command")]
public IEnumerable OptionD { get; set; }

[Option('e', HelpText = "Option E for save command")]
public IEnumerable OptionE { get; set; }
}
```
Demo code:
```
Parser parser = new(x =>
{
x.HelpWriter = Console.Out;
x.IgnoreUnknownArguments = true;
x.AllowMultiInstance = true;
x.AutoHelp = true;
});

var argTest1 = "log -a valueA -b valueB" .Split(' '); // This works.
var argTest2 = "save -d valueD -e valueE".Split(' '); // This works.
var argTest3 = "-a valueA -b valueB".Split(' '); // This works.
var argTest4 = "log -a valueA -b valueB -b valueB2".Split(' '); // This works.
var argTest5 = " -a valueA -b valueB -b valueB2".Split(' '); // This don't work.

var res = parser.ParseArguments(argTest5).MapResult(
(LogOptions opts) => ProcessLogCommand(opts),
(SaveOptions opts) => ProcessSaveCommand(opts),
errs => 1);
}
```

When trying to parse argTest5 this error is shown:

```
ERROR(S):
Option 'b' is defined multiple times.

-a Option A for log command

-b Option B for log command

--help Display this help screen.

--version Display version information.
```

**Expected behavior**
Option "SaveOptions" correctly parsed with multiple arguments for argument b.

Hướng dẫn đóng góp

Chưa lập chỉ mục được hướng dẫn đóng góp cho kho mã nguồn này

Hướng nghiên cứu

Bắt đầu bằng cách tái hiện vấn đề với ví dụ ParseArguments và MapResult được cung cấp, đặc biệt là argTest5, rồi theo dõi cách verb mặc định được phân tích khi AllowMultiInstance được bật. Được xem là hoàn tất khi verb mặc định chấp nhận các giá trị tùy chọn lặp lại mà không báo tùy chọn được định nghĩa nhiều lần, trong khi các ví dụ khác vẫn tiếp tục được phân tích chính xác.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Đánh giá

Công nghệ
csharp
Lĩnh vực
cli
Loại issue
Lỗi
Độ khó
3/5
Thời gian dự kiến
1-2 ngày
Mức độ hoạt động
Đình trệ
Độ rõ ràng
Khá rõ ràng
Mức phù hợp với người mới
35/100

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.