Changeset 3100

Timestamp:

05/08/2016 10:26:26 PM (7 years ago)

Author:

obenland

Message:

Plugin Directory: Allow for third party content to be displayed.

Having an action here allows for more flexibility to enhance the author card in
the future.

H/t coffee2code.
See #1695.

File:

• sites/trunk/wordpress.org/public_html/wp-content/plugins/plugin-directory/admin/metabox/class-author-card.php (modified) (1 diff)

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

@@ -182 +182 @@
         <?php
 
-        self::show_warning_flags( $post, $author, $all_plugins );
-    }
-
-    /**
-     * Displays listing of warning flags for the plugin and its authors.
-     *
-     * @param \WP_Post $plugin           The plugin object.
-     * @param \WP_User $author           The plugin author.
-     * @param array    $all_plugins      Array of plugin objects for all of user's plugins.
-     */
-    public static function show_warning_flags( $plugin, $author, $all_plugins ) {
-        $flagged = array(
-            'critical' => array(),
-            'med'      => array(),
-            'low'      => array(),
-            'info'     => array(),
-        );
-
-        $approved_plugins = wp_list_filter( $all_plugins, array( 'post_status' => 'publish' ) );
-        $rejected_plugins = wp_list_filter( $all_plugins, array( 'post_status' => 'rejected' ) );
-
-        // More than one instance of a spammer coming from one of these IPs or IP blocks (critical)
-        $post_ip       = get_post_meta( $plugin->ID, 'post_ip', true );
-        $is_spammer_ip = false;
-
-        $suspected_spammer_ip_blocks = array(
-            '2.240.101.121',
-            '2.240.163.90',
-            '2.240.118.188',
-            '2.241.60.160',
-            '2.241.66.20',
-            '2.241.124.187',
-            '5.102.170.',
-            '5.102.171.',
-            '38.78.',
-            '49.50.124.',
-            '65.33.104.38',
-            '71.41.77.202',
-            '76.73.108.',
-            '80.131.192.168',
-            '87.188.67.',
-            '87.188.75.',
-            '87.188.82.',
-            '91.228.',
-            '94.103.41.',
-            '109.123.',
-            '110.55.1.251',
-            '110.55.4.248',
-            '116.193.162.',
-            '119.235.251.',
-            '159.253.145.183',
-            '173.171.9.190',
-            '173.234.140.18',
-            '188.116.36.',
-            '217.87.249.',
-            '217.87.251.',
-            '217.87.252.',
-        );
-
-        foreach ( $suspected_spammer_ip_blocks as $spammer_ip ) {
-            if ( 0 === strpos( $post_ip, $spammer_ip ) ) {
-                $flagged['critical'][] = "spammer IP '$spammer_ip" . ( substr( $spammer_ip, - 1 ) == '.' ? '*' : '' ) . "'";
-                $is_spammer_ip         = true;
-                break;
-            }
-        }
-
-        /*
-         * These IPs or IP blocks have instances of being used by spammers, but aren't concrete
-         * (or are fairly broad) that we don't want to auto-reject them. But we want to be wary. (high)
+        /**
+         * Fires at the end of a plugin's author card.
+         *
+         * @param \WP_Post $plugin           The plugin object.
+         * @param \WP_User $author           The plugin author.
+         * @param array    $all_plugins      Array of plugin objects for all of user's plugins.
          */
-        $possible_spammer_ip_blocks = array(
-            '2.240.',
-            '2.241.',
-            '91.238.',
-            '182.68.',
-        );
-        if ( ! $is_spammer_ip ) {
-            foreach ( $possible_spammer_ip_blocks as $spammer_ip ) {
-                if ( 0 === strpos( $post_ip, $spammer_ip ) ) {
-                    $flagged['med'][] = "possible spammer IP '$spammer_ip" . ( substr( $spammer_ip, - 1 ) == '.' ? '*' : '' ) . "'";
-                    break;
-                }
-            }
-        }
-
-        /*
-         * If user is banned from logging into WP.org (critical)
-         *
-         * This is pretty rare. They would have to have been banned after having
-         * submitted the plugin.
-         */
-        if ( property_exists( $author, 'capabilities' ) && isset( $author->capabilities['blocked'] ) && '1' == $author->capabilities['blocked'] ) {
-            $flagged['critical'][] = 'user has been banned from logging into WP.org';
-        }
-
-        // If user < 2 days old, extra red-flaggy (high). ElseIf user is < 2 weeks old, consider them new. (med)
-        $user_date    = new \DateTime( $author->user_registered );
-        $user_date    = $user_date->format( 'U' );
-        $request_date = new \DateTime( $plugin->post_date );
-        $request_date = $request_date->format( 'U' );
-
-        if ( $user_date > strtotime( '-3 days', $request_date ) ) {
-            $flagged['med'][] = 'user &lt; 3 days old at request';
-        } elseif ( $user_date > strtotime( '-2 weeks', $request_date ) ) {
-            $flagged['low'][] = 'user &lt; 2 weeks old at request';
-        }
-
-        // If username ends in numbers and the user doesn't have any approved plugins.
-        if ( preg_match( '/\d{3,}$/', $author->user_login ) && 0 === count( $approved_plugins ) ) {
-            $flagged['med'][] = 'username ends in numbers';
-        }
-
-        // If username contains spammer-used words.
-        $spam_username_substrings = array(
-            'design',
-            'develop',
-            'html',
-            'market',
-            'seo',
-        );
-        foreach ( $spam_username_substrings as $spam ) {
-            if ( false !== strpos( $author->user_login, $spam ) ) {
-                $flagged['med'][] = "spammer-used username substring ($spam)";
-                break;
-            }
-        }
-
-        // If user's email is @yahoo.* or @mail.com (med).
-        $suspicious_email_hosts = array( '@yahoo.', '@mail.com' );
-        foreach ( $suspicious_email_hosts as $email_host ) {
-            if ( false !== strpos( $author->user_email, $email_host ) ) {
-                $flagged['med'][] = 'spammer-used email host';
-                break;
-            }
-        }
-
-        // If the plugin is for a typically spammed genre (med).
-        $spam_names   = array();
-        $spam_targets = array(
-            'bookmark',
-            'cookie',
-            'facebook',
-            'gallery',
-            'google',
-            'lightbox',
-            'seo',
-            'sitemap',
-            'slide',
-            'social',
-            'twitter',
-            'youtube',
-        );
-        foreach ( $spam_targets as $spam_target ) {
-            if ( false !== strpos( $plugin->post_name, $spam_target ) || false !== strpos( $plugin->post_title, $spam_target ) ) {
-                $spam_names[] = $spam_target;
-            }
-        }
-        if ( ! empty( $spam_names ) ) {
-            $flagged['low'][] = "plugin name/slug contains '" . implode( "', '", $spam_names ) . "'";
-        }
-
-        // If the plugin's name contains undesirable terms.
-        $undesirables      = array();
-        $undesirable_terms = array( 'autoblog', 'auto-blog', 'booking', 'plugin', 'spinning' );
-        foreach ( $undesirable_terms as $undesirable ) {
-            if ( false !== strpos( $plugin->post_name, $undesirable ) || false !== strpos( $plugin->post_title, $undesirable ) ) {
-                $undesirables[] = $undesirable;
-            }
-        }
-        if ( ! empty( $undesirables ) ) {
-            $flagged['med'][] = "plugin name/slug contains potentially undesirable term(s) '" . implode( "', '", $undesirables ) . "'";
-        }
-
-        // Home URL is at weebly.com.
-        if ( false !== strpos( $author->user_url, 'weebly.com' ) ) {
-            $flagged['med'][] = 'spammer-used web host for user URL (weebly.com)';
-        }
-
-        // User's first plugin (low).
-        if ( 0 === count( $approved_plugins ) ) {
-            $flagged['low'][] = 'user has no open plugins';
-        }
-
-        // User was rejected for this plugin before.
-        if ( ! empty( $rejected_plugins ) && in_array( $plugin->post_name, $rejected_plugins ) ) {
-            $flagged['med'][] = 'user was previously rejected for this plugin';
-        }
-
-        // User has previously rejected plugins (med).
-        if ( count( $rejected_plugins ) > 0 ) {
-            $flagged['med'][] = 'user has rejected plugins';
-        }
-
-        // User is blocked from posting to the support forums (med).
-        if ( property_exists( $author, 'elf_not_trusted' ) && '1' == $author->elf_not_trusted ) {
-            $flagged['med'][] = 'user is blocked from posting to the support forums';
-        }
-
-        // User is marked as a bozo in the support forums (low).
-        if ( property_exists( $author, 'is_bozo' ) && '1' == $author->is_bozo ) {
-            $flagged['low'][] = 'user is a bozo in the support forums';
-        }
-
-        // No home URL (low).
-        if ( empty( $author->user_url ) ) {
-            $flagged['low'][] = 'no URL for user';
-        } elseif ( false !== strpos( $author->user_url, 'blogspot.com' ) ) {
-            $flagged['med'][] = 'user URL at blogspot.com';
-        } elseif ( false !== strpos( $author->user_url, 'wordpress.com' ) ) {
-            $flagged['low'][] = 'user URL at WordPress.com';
-        }
-
-        // User has submitted this plugin before (info).
-        if ( in_array( $plugin->post_name, wp_list_pluck( $all_plugins, 'post_name' ) ) ) {
-            $flagged['info'][] = 'user has submitted this plugin before';
-        }
-
-        $flagged = array_filter( $flagged );
-
-        if ( empty( $flagged ) ) {
-            echo '<span class="plugin-flagged-status plugin-queue-unflagged" style="display:none;" title="This plugin has no warning flags">&nbsp;</span>';
-        } else {
-            if ( isset( $flagged['critical'] ) ) {
-                echo '<span class="plugin-flagged-status plugin-queue-flagged-critical" style="display:none;" title="This plugin should be rejected">&nbsp;</span>';
-            }
-            echo '<div class="plugin-queue-flagged">';
-            echo '<h4>FLAGGED!</h4>';
-            echo '<ul class="plugin-flagged">';
-
-            foreach ( $flagged as $flag_level => $flag ) {
-                $flag_name = 'critical' == $flag_level ? 'DO NOT APPROVE' : strtoupper( $flag_level );
-
-                echo '<li class="plugin-flagged-' . $flag_level . '"><strong>' . $flag_name . ' (' . count( $flagged[ $flag_level ] ) . '):</strong> ';
-                echo implode( '; ', $flagged[ $flag_level ] );
-
-                // Critically flagged plugins should sit in queue for at least a week to give spammer
-                // the impression that we're reviewing it
-                if ( 'critical' == $flag_level ) {
-                    $reject_on = strftime( '%h. %e', strtotime( '+1 week', $request_date ) );
-                    echo '<br />Reject this plugin after ' . $reject_on . ' (to give impression we\'re reviewing it).';
-                }
-
-                echo '</li>';
-            }
-            echo '</ul>';
-            echo '</div>';
-        }
-
-        return;
+        do_action( 'wporg_plugins_author_card', $post, $author, $all_plugins );
     }
 }