Changeset 12959

Timestamp:

10/31/2023 08:16:50 PM (2 years ago)

Author:

coffee2code

Message:

Photo Directory, Moderation: Improve flagging of high rejection users.

• Reword flag text to better reflect that it's based on a percentage of rejections against total submissions
• Increase flagging thresholds to 0.15 and 0.3 for warnings and alerts, respectively
• Add new critical threshold of 0.5
• Change how the rejection percentage is calculated
• Add new "More rejections than approvals" flag
• Change colors used for 'likely' and 'possible' flags

Fixes #7335.

Location:

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

Files:

• assets/css/admin.css (modified) (1 diff)
• inc/moderation.php (modified) (3 diffs)

sites/trunk/wordpress.org/public_html/wp-content/plugins/photo-directory/assets/css/admin.css

@@ -19 +19 @@
     margin-right: 4px;
 }
-.photos-flagged li.very_likely,
-.photos-flagged li.likely {
+.photos-flagged li.very_likely {
     background-color: #AA0000;
 }
+.photos-flagged li.likely {
+    background-color: darkorange;
+}
 .photos-flagged li.possible {
-    background-color: darkorange;
+    background-color: goldenrod;
 }
 

sites/trunk/wordpress.org/public_html/wp-content/plugins/photo-directory/inc/moderation.php

@@ -22 +22 @@
     /**
      * The threshold percentage of the number of rejections relative to the
-     * number of approvals at which the user should be flagged as a warning
-     * (in orange). Should be a lower value than
+     * number of approvals+rejections at which the user should be flagged as a warning
+     * (in yellow). Should be a lower value than
      * `FLAG_REJECTION_ALERT_THRESHOLD_PERCENTAGE`.
      *
      * @var float
      */
-    const FLAG_REJECTION_WARNING_THRESHOLD_PERCENTAGE = 0.1;
+    const FLAG_REJECTION_WARNING_THRESHOLD_PERCENTAGE = 0.15;
 
     /**
      * The threshold percentage of the number of rejections relative to the
-     * number of approvals at which the user should be flagged as an alert
+     * number of approvals+rejections at which the user should be flagged as an alert
      * (in red) rather than a warning (in orange). Should be a higher value than
      * `FLAG_REJECTION_WARNING_THRESHOLD_PERCENTAGE`.
@@ -38 +38 @@
      * @var float
      */
-    const FLAG_REJECTION_ALERT_THRESHOLD_PERCENTAGE = 0.25;
+    const FLAG_REJECTION_ALERT_THRESHOLD_PERCENTAGE = 0.30;
+
+    /**
+     * The threshold percentage of the number of rejections relative to the
+     * number of approvals+rejections at which the user should be flagged as
+     * critical rather than an alert. Should be a higher value than
+     * `FLAG_REJECTION_ALERT_THRESHOLD_PERCENTAGE`.
+     *
+     * @var float
+     */
+    const FLAG_REJECTION_CRITICAL_THRESHOLD_PERCENTAGE = 0.50;
 
     /**
@@ -619 +629 @@
 
             if ( $rejections_count > 0 ) {
-                $rejections_level = '';
+                $rejections_level = $message = '';
+
+                $rejections_percentage = $rejections_count / ( $rejections_count + $published_photos_count );
 
                 // A user with more rejections than approvals should be an alert.
-                if ( $rejections_count >= $published_photos_count ) {
+                if ( $rejections_count > $published_photos_count ) {
                     $rejections_level = 'very_likely';
+                    $message = __( 'more rejections than approvals', 'wporg-photos' );
                 }
                 // Specify as alert or warning based on count relative to alert threshold.
                 else {
-                    $reject_pct = $rejections_count / $published_photos_count;
-                    if ( $reject_pct >= self::FLAG_REJECTION_ALERT_THRESHOLD_PERCENTAGE ) {
+                    if ( $rejections_percentage >= self::FLAG_REJECTION_CRITICAL_THRESHOLD_PERCENTAGE ) {
                         $rejections_level = 'very_likely';
+                        $message = __( 'very high rejection rate (<strong>%d%%</strong>)', 'wporg-photos' );
                     }
-                    elseif ( $reject_pct >= self::FLAG_REJECTION_WARNING_THRESHOLD_PERCENTAGE ) {
+                    elseif ( $rejections_percentage >= self::FLAG_REJECTION_ALERT_THRESHOLD_PERCENTAGE ) {
+                        $rejections_level = 'likely';
+                        $message = __( 'high rejection rate (<strong>%d%%</strong>)', 'wporg-photos' );
+                    }
+                    elseif ( $rejections_percentage >= self::FLAG_REJECTION_WARNING_THRESHOLD_PERCENTAGE ) {
                         $rejections_level = 'possible';
+                        $message = __( 'rejection rate (<strong>%d%%</strong>)', 'wporg-photos' );
                     }
                 }
 
-                if ( $rejections_level ) {
-                    $flags[ sprintf( 'has rejections (<strong>%d</strong>)', $rejections_count ) ] = $rejections_level;
+                if ( $rejections_level && $message ) {
+                    $flags[ sprintf( $message, round( $rejections_percentage * 100 , 0 ) ) ] = $rejections_level;
                 }
             }