DocBlocks for filters inside a conditional expression are not imported
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 3/5
- Thời gian dự kiến
- 1-2 ngày
- Mức phù hợp với người mới
- 45/100
- Loại issue
- Lỗi
- Độ rõ ràng
- Khá rõ ràng
- Mức độ hoạt động
- Đình trệ
- Công nghệ
- php
- Lĩnh vực
- documentation
Hướng nghiên cứu
Bắt đầu bằng cách tái hiện hai ví dụ PHP và theo dõi cách các lệnh gọi apply_filters bên trong các biểu thức điều kiện được liên kết với các DocBlocks. So sánh kết quả của parser đối với cả hai ví dụ với tài liệu bộ lọc dự kiến, bao gồm wp_save_post_revision_check_for_changes. Được xem là hoàn tất khi DocBlock 2 được gán cho in_conditional_expression và DocBlock chính xác được gán cho in_conditional.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
DocBlocks outside a conditional are not imported if there are other filters (with DocBlocks) within the conditional.
Example 1
In this example DocBlock 2 is not imported for the filter in_conditional_expression
/**
* DocBlock 1
*/
function test_docblocks() {
/**
* DocBlock 2
*/
if ( apply_filters( 'in_conditional_expression', true ) ) {
/**
* DocBlock 3
*/
$value = apply_filters( 'in_conditional', true );
}
}
Example 2
In this example the (wrong) DocBlock 2 is imported for the filter 'in_conditional`.
/**
* DocBlock 1
*/
function test_docblocks() {
/**
* DocBlock 2
*/
if ( apply_filters( 'in_conditional_expression', true ) ) {
$value = apply_filters( 'in_conditional', true );
}
}
The wp_save_post_revision_check_for_changes filter doesn't get its DocBlock imported because of this bug.
See this filter in trac
- Ngôn ngữ chính
- PHP
- Star
- 239
- Fork
- 81
- Merge trung bình
- 16 giờ 29 phút
- Pull request đã merge (30 ngày)
- 8
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
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- 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.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của WordPress/phpdoc-parser
-
Include constants in exports Đang mở
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 55/100
WordPress/phpdoc-parser#269 ·
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 64/100
WordPress/phpdoc-parser#268 · 1 bình luận ·
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 45/100
WordPress/phpdoc-parser#254 ·
-
Compatibility updates required Đang mở
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 35/100
WordPress/phpdoc-parser#246 · 9 bình luận ·
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 25/100
WordPress/phpdoc-parser#235 · 1 bình luận ·
Tất cả issue của WordPress/phpdoc-parser
Issue tương tự
-
tooling
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
-
UX
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
ProfessionalWiki/NeoWiki#1525 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 90/100
OpenConext/OpenConext-engineblock#2122 ·
-
Bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
Automattic/safe-publish#594 ·