Changeset 12063 for sites/trunk/wordpress.org/public_html/wp-content/plugins/gp-translation-helpers/includes/class-wporg-notifications.php

Timestamp:

09/15/2022 03:13:17 PM (4 years ago)

Author:

amieiro

Message:

Translate. Add new status in the gp-translation-helpers plugin

File:

• sites/trunk/wordpress.org/public_html/wp-content/plugins/gp-translation-helpers/includes/class-wporg-notifications.php (modified) (4 diffs)

sites/trunk/wordpress.org/public_html/wp-content/plugins/gp-translation-helpers/includes/class-wporg-notifications.php

@@ -326 +326 @@
                         }
                 }
+                if ( isset( $comment_meta['reject_reason'][0] ) && ! empty( maybe_unserialize( $comment_meta['reject_reason'][0] ) ) ) {
+                        $reasons         = array();
+                        $comment_reasons = Helper_Translation_Discussion::get_comment_reasons();
+                        $reasons         = array_map(
+                                function( $reason ) use ( $comment_reasons ) {
+                                        if ( array_key_exists( $reason, $comment_reasons ) ) {
+                                                return $comment_reasons[ $reason ]['name'];
+                                        }
+                                },
+                                maybe_unserialize( $comment_meta['reject_reason'][0] )
+                        );
+                        $output         .= '- <strong>' . esc_html__( 'Reason(s): ' ) . '</strong>' . esc_html( implode( ', ', $reasons ) ) . '<br>';
+                }
                 $output .= '- <strong>' . esc_html__( 'Comment: ' ) . '</strong>' . esc_html( $comment->comment_content ) . '</pre>';
                 $output .= '<br><br>';
@@ -430 +443 @@
          */
         private static function get_opted_in_email_addresses( array $email_addresses ): array {
-                foreach ( $email_addresses as $email_address ) {
-                        if ( self::is_global_optout_email_address( $email_address ) ) {
-                                $index = array_search( $email_address, $email_addresses, true );
-                                if ( false !== $index ) {
-                                        unset( $email_addresses[ $index ] );
-                                }
+                foreach ( $email_addresses as $index => $email_address ) {
+                        if ( ! is_string( $email_address ) || empty( $email_address ) || self::is_global_optout_email_address( $email_address ) ) {
+                                unset( $email_addresses[ $index ] );
                         }
                 }
@@ -451 +461 @@
          */
         private static function is_global_optout_email_address( string $email_address ): bool {
+                if ( empty( $email_address ) || ! is_email( $email_address ) ) {
+                        return false;
+                }
                 $user            = get_user_by( 'email', $email_address );
                 $gp_default_sort = get_user_option( 'gp_default_sort', $user->ID );
@@ -602 +615 @@
                 $user = wp_get_current_user();
 
+                if ( ! $user->user_email ) {
+                        return __( "You will not receive notifications because you don't have an e-mail address set." );
+                }
                 if ( self::is_global_optout_email_address( $user->user_email ) ) {
-                        $output  = __( 'You will not receive notifications because you have not yet opted-in. ' );
+                        $output  = __( 'You will not receive notifications because you have not yet opted-in.' );
                         $output .= ' <a href="https://translate.wordpress.org/settings/">' . __( 'Start receiving notifications.' ) . '</a>';
                         return $output;