Changeset 3324

Timestamp:

06/09/2016 07:34:35 PM (8 years ago)

Author:

obenland

Message:

Plugin Directory: Limit the amount of assignable categories to 3.

See #1573.

Location:

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

Files:

• class-customizations.php (modified) (2 diffs)
• metabox/class-plugin-categories.php (modified) (3 diffs)

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

@@ -29 +29 @@
         add_action( 'pre_get_posts', array( $this, 'pre_get_posts' ) );
         add_action( 'save_post_plugin', array( $this, 'save_plugin_post' ) );
+        add_filter( 'tax_input_pre', array( $this, 'filter_tax_input' ) );
 
         add_action( 'load-edit.php', array( $this, 'bulk_reject_plugins' ) );
@@ -448 +449 @@
             update_post_meta( $post_id, 'tested', wp_slash( wp_unslash( $_POST['tested_with'] ) ) );
         }
+    }
+
+    /**
+     * Filters the value of tax_inputs before saving.
+     *
+     * @param array $tax_input Array of taxonomies with selected terms.
+     * @return array
+     */
+    public function filter_tax_input( $tax_input ) {
+
+        // Limit the amount of assignable categories to 3.
+        if ( isset( $tax_input['plugin_category'] ) ) {
+            $tax_input['plugin_category'] = array_slice( array_filter( $tax_input['plugin_category'] ), 0, 3 );
+        }
+
+        return $tax_input;
     }
 

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

@@ -8 +8 @@
  */
 class Plugin_Categories {
+
     /**
      * Displays the categories metabox for plugins.
@@ -17 +18 @@
         ?>
         <div id="taxonomy-plugin_category" class="categorydiv">
+            <div class="notice notice-info inline">
+                <p><?php _e( 'You can assign up to 3 categories.', 'wporg-plugins' ); ?></p>
+            </div>
+
             <div id="plugin_category-all" class="tabs-panel">
                 <?php // Allows for an empty term set to be sent. 0 is an invalid Term ID and will be ignored by empty() checks. ?>
@@ -28 +33 @@
     }
 }
-