Changeset 2777 for sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/class-tools.php

Timestamp:

03/22/2016 06:21:26 AM (10 years ago)

Author:

dd32

Message:

Plugin Directory: Introduce the Plugin Committer/Review & Admin roles. Use the capabilities throughout the core flows of the admin.
This also has a few hacks to make plugin committers/authors only see plugins which they can manage, although a few core bugs remain which cause non-owner committers not to be able to edit plugins properly.

See #1571

File:

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

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

@@ -41 +41 @@
         }
 
+        public static function get_users_write_access_plugins( $user ) {
+                global $wpdb;
+                if ( ! $user instanceof \WP_User ) {
+                        $user = new \WP_User( $user );
+                }
+                if ( ! $user->exists() ) {
+                        return false;
+                }
+
+                $plugins = $wpdb->get_col( $wpdb->prepare( 'SELECT path FROM `' . PLUGINS_TABLE_PREFIX . 'svn_access' . '` WHERE user = %s', $user->user_login ) );
+                $plugins = array_map( function( $plugin ) { return trim( $plugin, '/' ); }, $plugins );
+
+                return $plugins;
+
+        }
+
         /**
          * Grant a user RW access to a plugin.
@@ -55 +71 @@
                 }
 
-                if ( ! $user->exists() || ! Plugin_Directory::instance()->get_plugin_post( $plugin_slug ) ) {
+                if ( ! $user->exists() || ! Plugin_Directory::get_plugin_post( $plugin_slug ) ) {
                         return false;
                 }
 
-                $existing_committers = wp_list_pluck( self::get_plugin_committers( $plugin_slug ), 'user_login' );
+                $existing_committers = self::get_plugin_committers( $plugin_slug );
                 if ( in_array( $user->user_login, $existing_committers, true ) ) {
                         // User already has write access
@@ -89 +105 @@
                 }
 
-                if ( ! $user->exists() || ! Plugin_Directory::instance()->get_plugin_post( $plugin_slug ) ) {
+                if ( ! $user->exists() || ! Plugin_Directory::get_plugin_post( $plugin_slug ) ) {
                         return false;
                 }