Opened 3 years ago
Last modified 6 months ago
#56599 new defect (bug)
"All" view not selected for all relevant views for custom post type
| Reported by: |
|
Owned by: | |
|---|---|---|---|
| Milestone: | Awaiting Review | Priority: | normal |
| Severity: | trivial | Version: | 4.2 |
| Component: | Posts, Post Types | Keywords: | has-patch has-test-info needs-testing |
| Focuses: | administration | Cc: |
Description
When the screen options get saved, the parameter mode=list gets appended to the URL. For the posts table, the “All” view gets the .current HTML class, since \WP_Posts_List_Table::is_base_request() returns true. For all other post types—including pages—it returns false.
Change History (8)
This ticket was mentioned in PR #3275 on WordPress/wordpress-develop by alpipego.
3 years ago
#1
- Keywords has-patch added
#2
@
3 years ago
We noticed the same happens for order and orderby—even for the posts table. I have updated the patch to consider order and orderby as well.
#3
@
3 years ago
- Component changed from General to Posts, Post Types
- Focuses administration added
- Keywords has-testing-info added
- Version set to 4.2
Reproduction Report
This report validates that the issue can be reproduced.
Steps to Reproduce or Test
- Navigate to
Posts. - Open the
Screen Optionstab and make a change. - 🐞 Click
Apply.
Expected Results
- ❌ The
Allview is not bold.
Environment
- OS: Windows 10
- Web Server: Apache (WSL2 - Ubuntu)
- PHP: 7.4
- WordPress: 6.1-alpha-54198
- Browser: Chrome 105.0.5195.125
- Theme: Twenty Twenty-Two
- Active Plugins: None activated.
Actual Results
- ✅ The
Allview is not bold.
Additional Notes
- This began happening from WordPress 4.2 onwards.
#4
@
3 years ago
- Keywords needs-testing added
Per notes in the test report above:
This may benefit from additional testing for plugins that extend the
WP_Posts_List_Tableclass. Addingneeds-testinguntil this is done.
This ticket was mentioned in PR #8056 on WordPress/wordpress-develop by @alpipego.
11 months ago
#5
Analogous to the paged parameter, unset mode whet checking whether this is the base request for the \WP_Posts_List_Table.
https://core.trac.wordpress.org/ticket/56599
Trac ticket:
#6
@
11 months ago
After accidentally closing the old PR, I have created a new one and rebased it on the current trunk.
Analogous to the
pagedparameter, unsetmodewhet checking whetherthis is the base request for the
\WP_Posts_List_Table.Trac ticket: https://core.trac.wordpress.org/ticket/56599