Changeset 2752 for sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/admin/class-customizations.php

Timestamp:

03/15/2016 08:02:51 PM (9 years ago)

Author:

obenland

Message:

Plugin Directory: Allow plugin committers to be added and removed.

Still needs cap checks.
Uses wpList to seemlessly add/remove committers. Works similar to the
category meta box.

See #1571.

File:

• sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/admin/class-customizations.php (modified) (4 diffs)

sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/admin/class-customizations.php

@@ -30 +30 @@
 
         add_action( 'admin_enqueue_scripts', array( $this, 'enqueue_assets' ) );
+
         add_action( 'wp_ajax_replyto-comment', array( $this, 'save_custom_comment' ), 0 );
-
-        add_filter( 'postbox_classes_plugin_internal-notes', array( $this, 'postbox_classes' ) );
         add_filter( 'comment_row_actions', array( $this, 'custom_comment_row_actions' ), 10, 2 );
+
+        add_filter( 'postbox_classes_plugin_internal-notes',    array( __NAMESPACE__ . '\Metabox\Internal_Notes', 'postbox_classes' ) );
+        add_filter( 'postbox_classes_plugin_plugin-committers', array( __NAMESPACE__ . '\Metabox\Committers',     'postbox_classes' ) );
+        add_filter( 'wp_ajax_add-committer',    array( __NAMESPACE__ . '\Metabox\Committers', 'add_committer'    ) );
+        add_filter( 'wp_ajax_delete-committer', array( __NAMESPACE__ . '\Metabox\Committers', 'remove_committer' ) );
     }
 
@@ -61 +65 @@
         global $post_type;
 
-        if ( 'post.php' == $hook_suffix && 'plugin' == $post_type ) {
+        if ( 'post.php' === $hook_suffix && 'plugin' === $post_type ) {
             wp_enqueue_style( 'plugin-admin-edit-css', plugins_url( 'css/edit-form.css', Plugin_Directory\PLUGIN_FILE ), array( 'edit' ), 1 );
-            wp_enqueue_script( 'plugin-admin-edit-js', plugins_url( 'js/edit-form.js', Plugin_Directory\PLUGIN_FILE ), array( 'wp-util' ), 1 );
+            wp_enqueue_script( 'plugin-admin-edit-js', plugins_url( 'js/edit-form.js', Plugin_Directory\PLUGIN_FILE ), array( 'wp-util', 'wp-lists' ), 1 );
+            wp_localize_script( 'plugin-admin-edit-js', 'pluginDirectory', array(
+                'removeCommitterAYS' => __( 'Are you sure you want to remove this committer?', 'wporg-plugins' ),
+            ) );
         }
     }
@@ -133 +140 @@
             update_post_meta( $post_id, 'tested', wp_slash( wp_unslash( $_POST['tested_with'] ) ) );
         }
-    }
-
-    /**
-     * Filters the postbox classes for custom comment meta boxes.
-     *
-     * @param array $classes An array of postbox classes.
-     * @return array
-     */
-    public function postbox_classes( $classes ) {
-        $classes[] = 'comments-meta-box';
-
-        return array_filter( $classes );
     }
 
@@ -160 +155 @@
      */
     public function custom_comment_row_actions( $actions, $comment ) {
-        if ( 'internal-note' === $comment->comment_type ) {
+        if ( 'internal-note' === $comment->comment_type && isset( $_REQUEST['mode'] ) && 'single' === $_REQUEST['mode'] ) {
             $allowed_actions = array( 'reply' => true );