Automattic / Automattic/VIP-Coding-Standards

Catch incorrect bloginfo() usage

Đang mở
#396 1 bình luận 0 reaction 0 người được giao Xem trên GitHub
Type: Enhancement
Ngôn ngữ chính
PHP
Star
261
Fork
44
Merge trung bình
19 phút
Pull request đã merge (30 ngày)
1

Mô tả

## What problem would the enhancement address for VIP?
When using `bloginfo()`, the caller should not echo the output, as `bloginfo()` does this internally (see [here](https://developer.wordpress.org/reference/functions/bloginfo/#source)). Callers should simply call `bloginfo()` without any `echo`, but if the callers want to control when the output is echoed, they should use `get_bloginfo()` ([#](https://developer.wordpress.org/reference/functions/get_bloginfo/)). Our standards should catch these kinds of issues in code.

## Describe the solution you'd like
The usage of `echo bloginfo()` should be caught and advice given on using `get_bloginfo()` instead or to skip the echo.

## What code should be reported as a violation?
echo bloginfo( 'name' );

## What code should *not* be reported as a violation?
echo get_bloginfo( 'name' );

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

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

Hướng nghiên cứu

Bắt đầu với các quy tắc PHP_CodeSniffer kiểm tra việc sử dụng hàm, sử dụng bloginfo() và get_bloginfo() làm các điểm đầu vào. Xác nhận rằng echo bloginfo( 'name' ) được báo cáo kèm khuyến nghị bỏ echo hoặc sử dụng get_bloginfo(), trong khi echo get_bloginfo( 'name' ) thì không; chạy các bài kiểm thử quy tắc liên quan nếu 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ệ
php
Lĩnh vực
tooling
Loại issue
Tính năng
Độ khó
3/5
Thời gian dự kiến
1-2 ngày
Mức độ hoạt động
Đình trệ
Độ rõ ràng
Đặc tả 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.