prettier / prettier/plugin-php

Wrong indentation in match statements with multiple options

Đang mở
#2,435 1 bình luận 0 reaction 0 người được giao Xem trên GitHub

Chưa có ai nhận issue này.

bug
Ngôn ngữ chính
PHP
Star
1.9k
Fork
139
Chỉ số merge pull request
Không có pull request nào được merge trong 30 ngày

Mô tả

@prettier/plugin-php v0.22.4
Playground link

Input:

<?php

echo match ($operation) {
      "short", "really very long option 2" => SomeClass::fairlyLongMethodNameThatRunsOn($variable, ["short", "short"]),
}

Output:

<?php

echo match ($operation) {
    "short",
    "really very long option 2"
        => SomeClass::fairlyLongMethodNameThatRunsOn($variable, [
        "short",
        "short",
    ]),
};

As you can see, the indentation in the output is off. There's arguments for each individual line being ok, but at the very least, the array entries should by indented one more than their braces (now: same as opening, 1 more than closing), and the closing brace for the array should be indented as much as the line with the opening brace.

As far as I can tell, the length of the individual parts is irrelevant, only that they are in total long enough for a line break to be forced.

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

Mở hướng dẫn đóng góp

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Hướng nghiên cứu

Bắt đầu với bản tái hiện Playground được liên kết và so sánh đầu vào và đầu ra được báo cáo cho câu lệnh match có nhiều tùy chọn và mảng lồng nhau của nó. Theo dõi hành vi của formatter chịu trách nhiệm về ngắt dòng và thụt lề, sau đó xác minh rằng các phần tử mảng được thụt lề sâu hơn dấu ngoặc nhọn mở của chúng và dấu ngoặc nhọn đóng được căn chỉnh với dấu ngoặc đó mà không làm ảnh hưởng đến định dạng match xung quanh.

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

Đánh giá

Công nghệ
php
Lĩnh vực
tooling
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
45/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.