prettier / prettier/plugin-php

`{` and `}` for variable property

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

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

discussion enhancement
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 1.14.3

PHP Plugin 0.8.0

Input:

$var = $this->{$key};
$var = $this->value->{$method}();

Output:

$var = $this->$key;
$var = $this->value->$method();

Expected behavior:

Need vote

👍 for $this->{$key}
❤️ for $this->$key

I found what laravel, symfony and a lot of other projects use { and }, looks it is popular.

One note - we should respect { and } in source code, because it can be break php5 code http://php.net/manual/en/migration70.incompatible.php#migration70.incompatible.variable-handling.indirect.

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

Tái hiện input được báo cáo bằng Prettier 1.14.3 và PHP Plugin 0.8.0, sau đó kiểm tra cách các thuộc tính biến được định dạng. Issue không nêu file hoặc test nào và chưa có output mong đợi được thống nhất; hãy xác nhận quyết định của dự án về việc giữ lại các dấu ngoặc nhọn trước khi xác định việc hoàn tất.

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ó
4/5
Thời gian dự kiến
3-5 ngày
Mức độ hoạt động
Đình trệ
Độ rõ ràng
Cần làm rõ
Mức phù hợp với người mới
25/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.