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 (10 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 );