Automattic / Automattic/VIP-Coding-Standards
Flag incorrectly constructed mailto links
- 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?
Some developers are not aware that `esc_url()` supports more than just the `http` and `https` protocols. The default list also includes ftp, ftps, mailto, news, irc, gopher, nntp, feed, and telnet as well.
of those extra ones, the most common is `mailto`, and a common mistake is to split a URL into a static `'mailto:'` and a email address variable/string escaped with something that isn't `esc_url()`.
## Describe the solution you'd like
Add a new sniff, or consider improving ProperEscapingFunction, so that we look for `'mailto:' string before an escaping function.
## What code should be reported as a violation?
```php
Email us
Email us
```
There are likely other ways to get a similar output.
## What code should *not* be reported as a violation?
```php
Email us
">Email us
```
Hướng dẫn đóng góp
Hướng nghiên cứu
Bắt đầu với sniff ProperEscapingFunction được đề cập trong issue và so sánh cách xử lý các ví dụ PHP vi phạm và không vi phạm được liệt kê. Hoàn thành khi các liên kết mailto được tạo không chính xác bị báo cáo, trong khi các giá trị mailto được truyền qua esc_url() được chấp nhận.
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, wordpress
- 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
- Khá rõ ràng
- Mức phù hợp với người mới
- 35/100