Opened 10 years ago
Closed 10 years ago
#32376 closed defect (bug) (fixed)
Urls are encoded incorrectly in WP_Posts_List_Table::get_views()
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 4.4 | Priority: | normal |
Severity: | normal | Version: | 4.2.2 |
Component: | Posts, Post Types | Keywords: | has-patch needs-docs |
Focuses: | administration | Cc: |
Description
I stumbled upon this bug when I tried to fix something else: The current css class is not added to the 'All' link in the views. To solve this issue, I added the css class with DOMDocument. But when I called the method to read the html of
$views['all']
like so:
DOMDocument::loadHTML($views['all'])
, I received a parsing error. The parsing error is thrown because the ampersand in WP_Posts_List_Table::get_views() on line 185 is not encoded.
I will refrain from using my DOMDocument method of fixing the other bug, I will file another bug instead, because I now know why the 'current' css class is not added to the 'All' view.
Attachments (2)
Change History (7)
#2
@
10 years ago
- Milestone changed from Awaiting Review to 4.4
- Owner set to wonderboymusic
- Status changed from new to assigned
I did some work here. See 32376.diff
#4
@
10 years ago
- Keywords needs-docs added
- Resolution fixed deleted
- Status changed from closed to reopened
Documentation for this new method is incomplete:
- Need docs for all three parameters, including marking the third one optional
- The summary needs a period and should not contain markdown backticks
- Needs an
@access
tag
Note: See
TracTickets for help on using
tickets.
A patch fixing the issue.