Allow form feed characters in attributes
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 2/5
- Thời gian dự kiến
- 1-3 giờ
- Mức phù hợp với người mới
- 74/100
Hướng nghiên cứu
Bắt đầu tại Tokenizer::quotedAttributeValue() và tái hiện vấn đề với một thuộc tính alt của img chứa ký tự ngắt trang, đồng thời so sánh đầu ra HTML5 với DOMDocument. Truy vết lý do văn bản sau ký tự ngắt trang bị loại bỏ, sau đó bổ sung coverage cho thấy ký tự này được giữ lại như trong đầu ra DOM native.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Tokenizer::quotedAttributeValue() has special handling for form feed characters.
(This used to also include newlines and tabs but was removed in https://github.com/Masterminds/html5-php/commit/8785fc5775df52dd4a6b63677ebd08f8301ffdcc.)
Because of this, if an attribute contains a form feed, everything after this character is ignored.
This creates a discrepancy between masterminds/html5 and the native DOM extension:
$html = "<img src='foo.png' alt='before \f after'>";
$html5 = new HTML5();
$dom = $html5->loadHTML($html);
echo $html5->saveHTML($dom);
// <img src="foo.png" alt="before ">
$dom = new DOMDocument();
$dom->loadHTML($html);
echo $dom->saveHTML();
// <img src="foo.png" alt="before after">
This HTML containing a form feed character in the alt attribute is valid according to https://validator.w3.org/:
<!doctype html>
<html lang="en">
<head>
<meta charset="UTF-8">
<title>Document</title>
</head>
<body>
<img src="foo.png" alt=" foo">
</body>
</html>
- Ngôn ngữ chính
- HTML
- Star
- 1.8k
- Fork
- 123
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Chuẩn bị môi trường
Dự án này không cung cấp dev container, Dockerfile hay hướng dẫn đóng góp, nên bạn cần tự thiết lập môi trường: hãy bắt đầu từ README và xem hướng dẫn đóng góp lần đầu của chúng tôi để biết các bước chung.
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 Masterminds/html5-php
-
Template contents participate in the DOM treeCó thể đã có người làm @eyupcanakman đã nhận 204 ngày trước. Đang mởhelp-wanted
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 35/100
Masterminds/html5-php#252 · 1 bình luận · 1 reaction ·
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 35/100
Masterminds/html5-php#251 ·
-
Parser remove the single < (less than) character from given html stringCó thể đã có người làm @bsweeney đã nhận 789 ngày trước. Đang mở
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 35/100
Masterminds/html5-php#250 · 5 bình luận · 2 reaction ·
-
multiple <html> <body>Đang mở
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 28/100
Masterminds/html5-php#246 · 1 bình luận ·
-
<noscript> with html insideĐang mở
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 25/100
Masterminds/html5-php#245 · 1 bình luận ·
Tất cả issue của Masterminds/html5-php
Issue tương tự
-
bug llm translation
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
Maintainer thường phản hồi trong vòng 1 ngày
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
jellyfin/jellyfin-mpv-shim#800 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Michigan 2025 senior interest, dividends and capital gains deduction limits are uprated, not the published amountsCó thể đã có người làm @PavelMakarchuk đã nhận hôm nay. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
PolicyEngine/policyengine-us#9826 ·
Maintainer thường phản hồi trong vòng 2 ngày
-
enhancement
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
Maintainer thường phản hồi trong vòng 1 ngày
-
P4: low query
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
jeffknupp/association#336 ·