Opened 11 years ago
Closed 11 years ago
#26290 closed defect (bug) (fixed)
wp_kses_normalize_entities destroys some valid entities that ent2ncr supports
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 3.8 | Priority: | normal |
Severity: | normal | Version: | 3.7 |
Component: | Formatting | Keywords: | has-patch |
Focuses: | Cc: |
Description
We have a list of common html entities we convert for feeds.
We have a different list of entities kses allows.
They don't overlap and you can't extend the kses config to allow for all of them because it doesn't support entities like ¼
Attachments (1)
Change History (6)
#3
follow-up:
↓ 4
@
11 years ago
26290.diff looks good. In the future let's save unit tests for the same commit as the patch. Takes some adjusting to, for sure, but it's much kinder to our future selves.
#4
in reply to:
↑ 3
@
11 years ago
Replying to nacin:
26290.diff looks good. In the future let's save unit tests for the same commit as the patch. Takes some adjusting to, for sure, but it's much kinder to our future selves.
I'm not convinced by that I actually prefer them to be separate.
The ticket as a reference links them together.
In 26430: