Opened 3 years ago
Last modified 12 months ago
#14060 new defect (bug)
Misleading "You do not have sufficient permissions to access this page."
| Reported by: |
|
Owned by: | |
|---|---|---|---|
| Priority: | normal | Milestone: | Future Release |
| Component: | Plugins | Version: | 3.0 |
| Severity: | normal | Keywords: | has-patch |
| Cc: |
Description
When accessing a page of a plugin in the admin that does not exists any longer, you're told by an error message, that:
You do not have sufficient permissions to access this page.
This message is a little misleading, because there is no such page.
A message that better reflects the decision to display it would be less misleading, e.g.:
The requested page is not accessible.
Attachments (3)
Change History (15)
comment:1
hakre
— 3 years ago
FYI: at the end of Plugin.php there are a lot of functions undocumented and in addition to that important global variables for the admin like $parent_file or $plugin_page are not documented as well.
comment:3
dd32
— 3 years ago
- Keywords needs-patch added; has-patch removed
Both of those proposed wordings are not appropriate IMO.
The major case of that block is to prevent users accessing pages which they do not have access to, The current wording seems appropriate for that (A straight up, Permission denied message).
"The requested page is not accessible." - Doesn't convey the reason for the page being inaccessible.
An alternate solution could be such as follows to give non-existant pages a specific error:
if ( !admin_page_exists() ) {
do_action...
wp_die "non-existant page!"
} elseif ( !user_can_access_admin_page() ) {
do_action...
wp_die "You dont have access!...";
}
comment:5
hakre
— 3 years ago
For my scenario to check whether or not a page exists, would haven been helpful already.
SergeyBiryukov
— 2 years ago
comment:9
SergeyBiryukov
— 2 years ago
- Keywords has-patch added; needs-patch removed
14060.2.patch implements dd32's suggestion.
comment:10
SergeyBiryukov
— 21 months ago
Closed #18662 as a duplicate.
comment:11
jane
— 19 months ago
"The requested page no longer exists."
comment:12
SergeyBiryukov
— 12 months ago
- Component changed from Accessibility to Plugins
Simple Patch