WordPress.org

Make WordPress Core

Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#38868 closed defect (bug) (wontfix)

Merge two option managment error messages

Reported by: ramiy Owned by:
Milestone: Priority: normal
Severity: normal Version: 4.7
Component: I18N Keywords: has-patch has-screenshots close
Focuses: administration Cc:

Description

See the attached patch.

Attachments (2)

manage-options.png (37.3 KB) - added by ramiy 4 years ago.
38868.patch (3.8 KB) - added by ramiy 4 years ago.

Download all attachments as: .zip

Change History (7)

@ramiy
4 years ago

@ramiy
4 years ago

#1 @ramiy
4 years ago

  • Keywords has-patch has-screenshots added

#2 @ramiy
4 years ago

The patch unifies 7 translation strings that checks if ( ! current_user_can( 'manage_options' ) ).

#3 @SergeyBiryukov
4 years ago

  • Keywords close added

Most of those strings refer to the general manage_options capability, whereas the one in wp-admin/options.php refers to the capability passed through the option_page_capability_{$option_page} filter, so I think the more specific "Sorry, you are not allowed to manage these options" string from [39308] is more appropriate there.

#4 @ramiy
4 years ago

  • Resolution set to wontfix
  • Status changed from new to closed

#5 @netweb
4 years ago

  • Milestone Awaiting Review deleted
Note: See TracTickets for help on using tickets.