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

Timestamp:

10/11/2016 08:27:15 AM (10 years ago)

Author:

dd32

Message:

Plugin Directory: Add a plugin_committer taxonomy and sync it with the svn_access table.
The taxonomy is only used to ease querying, and allows for /author/$self to display both plugins you're listed as a contributor for and those you have commit access for.

See #2111

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

@@ -122 +122 @@
 
         /**
+         * Syncs the list of committers from the svn_access table to a taxonomy.
+         *
+         * @static
+         * @param string $plugin_slug The Plugin Slug to sync.
+         */
+        public static function sync_plugin_committers_with_taxonomy( $plugin_slug ) {
+                $post = Plugin_Directory::get_plugin_post( $plugin_slug );
+                if ( ! $post ) {
+                        return false;
+                }
+
+                $committer_slugs = array();
+                foreach ( Tools::get_plugin_committers( $plugin_slug ) as $committer ) {
+                        $user = get_user_by( 'login', $committer );
+                        if ( $user ) {
+                                $committer_slugs[] = $user->user_nicename;
+                        }
+                }
+
+                wp_set_post_terms( $post->ID, $committer_slugs, 'plugin_committers' );
+        }
+
+        /**
          * Grant a user RW access to a plugin.
          *
@@ -149 +172 @@
                 }
 
-                wp_cache_delete( "{$plugin_slug}_committer", 'wporg-plugins' );
-                wp_cache_delete( "{$user->user_login}_committer", 'wporg-plugins' );
-
-                return (bool) $wpdb->insert( PLUGINS_TABLE_PREFIX . 'svn_access', array(
+                $result = (bool) $wpdb->insert( PLUGINS_TABLE_PREFIX . 'svn_access', array(
                         'path'   => "/{$plugin_slug}",
                         'user'   => $user->user_login,
                         'access' => 'rw',
                 ) );
+
+                wp_cache_delete( "{$plugin_slug}_committer", 'wporg-plugins' );
+                wp_cache_delete( "{$user->user_login}_committer", 'wporg-plugins' );
+                Tools::sync_plugin_committers_with_taxonomy( $plugin_slug );
+
+                return $result;
         }
 
@@ -180 +206 @@
                 }
 
-                wp_cache_delete( "{$plugin_slug}_committer", 'wporg-plugins' );
-                wp_cache_delete( "{$user->user_login}_committer", 'wporg-plugins' );
-
-                return $wpdb->delete( PLUGINS_TABLE_PREFIX . 'svn_access', array(
+                $result = (bool) $wpdb->delete( PLUGINS_TABLE_PREFIX . 'svn_access', array(
                         'path' => "/{$plugin_slug}",
                         'user' => $user->user_login,
                 ) );
+
+                wp_cache_delete( "{$plugin_slug}_committer", 'wporg-plugins' );
+                wp_cache_delete( "{$user->user_login}_committer", 'wporg-plugins' );
+                Tools::sync_plugin_committers_with_taxonomy( $plugin_slug );
+
+                return $result;
         }