WordPress.org

Make WordPress Core

Opened 17 months ago

Closed 17 months ago

Last modified 17 months ago

#22569 closed defect (bug) (fixed)

Show a better message on link-manager.php when links are disabled

Reported by: nacin Owned by: ryan
Milestone: 3.5 Priority: normal
Severity: normal Version:
Component: Administration Keywords: has-patch
Focuses: Cc:

Description

We enforce the disabling of links via map_meta_cap(), since it's easy and clean. But it does mean that cap checks fail, which usually produces "You do not have sufficient permissions".

I meant to add a second message, but never did. Let's do that.

Attachments (4)

22569.diff (3.8 KB) - added by nacin 17 months ago.
If we went all out, here's how I'd do it.
22569.2.diff (3.7 KB) - added by nacin 17 months ago.
22569.3.diff (1.2 KB) - added by nacin 17 months ago.
22569.4.diff (1.8 KB) - added by nacin 17 months ago.

Download all attachments as: .zip

Change History (12)

nacin17 months ago

If we went all out, here's how I'd do it.

comment:1 nacin17 months ago

  • Keywords has-patch added

comment:2 TobiasBg17 months ago

Are those localhost URLs in the patch correct?

comment:3 nacin17 months ago

Hah. Clearly not. :)

nacin17 months ago

comment:4 ryan17 months ago

In multisite, a regular admin has manage_links but not install_plugins yet they still get a "please install" message that they cannot act upon themselves. Show the install message only if the user has both caps, otherwise the permissions error?

comment:5 ryan17 months ago

I think the call to user_can_access_admin_page() in wp-admin/includes/menu.php slams the door before these checks run.

nacin17 months ago

comment:6 nacin17 months ago

Good points. 22569.3.diff.

nacin17 months ago

comment:7 ryan17 months ago

  • Owner set to ryan
  • Resolution set to fixed
  • Status changed from new to closed

In 22855:

Show a better message ondmins on link-manager.php, link-add.php, link.php when links are disabled.

Props nacin
fixes #22569

comment:8 ryan17 months ago

Tested with super admin, normal admin, editor, and logged out.

Last edited 17 months ago by ryan (previous) (diff)
Note: See TracTickets for help on using tickets.