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 (8 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;
     }