[wp-trac] [WordPress Trac] #47487: Classic Editor plugin link and link target

WordPress Trac noreply at wordpress.org
Fri Jun 7 17:32:24 UTC 2019


#47487: Classic Editor plugin link and link target
-------------------------------------+----------------------------
 Reporter:  derweili                 |       Owner:  (none)
     Type:  enhancement              |      Status:  new
 Priority:  normal                   |   Milestone:  5.3
Component:  Editor                   |     Version:  5.2.1
 Severity:  normal                   |  Resolution:
 Keywords:  has-patch needs-testing  |     Focuses:  accessibility
-------------------------------------+----------------------------
Changes (by garrett-eclipse):

 * keywords:  needs-patch => has-patch needs-testing
 * focuses:   => accessibility
 * milestone:  Future Release => 5.3


Comment:

 Thanks for flagging this @derweili and for the initial patch, made it easy
 to find what link you were referring to.

 @SergeyBiryukov I appreciate the input and direction, I've pulled the
 internal install link which was committed here;
 https://core.trac.wordpress.org/changeset/44264

 This links to the custom install screen made for the Classic Editor
 plugin. For reference here's the link code;
 `esc_url( wp_nonce_url( self_admin_url( 'plugin-
 install.php?tab=favorites&user=wordpressdotorg&save=0' ),
 'save_wporg_username_' . get_current_user_id() ) )`

 My patch
 [https://core.trac.wordpress.org/attachment/ticket/47487/47487.2.diff
 47487.2.diff] implements this URL replacing the translatable external
 plugin link.

 This loads the custom favorites tab of the plugin install screen for the
 wordpressdotorg user which is specifically favourited Classic Editor, also
 has nonce to restore the users favorites.

 Sergey can you review the link setup for CS I wasn't sure since it was
 inside an expanded sprintf if the esc_url, wp_nonce_url and self_admin_url
 calls should all be on individual lines indented or if the shortform I'm
 using passes formatting.

 Please test,
 Thank you

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/47487#comment:3>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list