Changeset 12071 for sites/trunk/wordpress.org/public_html/wp-content/plugins/gp-translation-helpers

Timestamp:

09/16/2022 10:15:08 AM (4 years ago)

Author:

akirk

Message:

gp-translation-helpers: Fix notices

Location:

sites/trunk/wordpress.org/public_html/wp-content/plugins/gp-translation-helpers

Files:

• . (modified) (1 prop)
• helpers/helper-translation-discussion.php (modified) (2 diffs)
• templates/discussions-dashboard.php (modified) (2 diffs)

sites/trunk/wordpress.org/public_html/wp-content/plugins/gp-translation-helpers

@@ -8 +8 @@
 .eslintrc.json
 tmp
+phpunit.xml.dist
+tests

@@ -8 +8 @@
 .eslintrc.json
 tmp
+phpunit.xml.dist
+tests

sites/trunk/wordpress.org/public_html/wp-content/plugins/gp-translation-helpers/helpers/helper-translation-discussion.php

@@ -559 +559 @@
                 'translation_id'       => isset( $this->data['translation_id'] ) ? $this->data['translation_id'] : null,
                 'locale_slug'          => $this->data['locale_slug'],
+                'original_permalink'   => $this->data['original_permalink'],
                 'original_id'          => $this->data['original_id'],
                 'project'              => $this->data['project'],
@@ -672 +673 @@
      * @param      string  $link     The link.
      * @param      array   $args     The arguments.
-     * @param      string  $comment  The comment.
+     * @param      object  $comment  The comment.
      * @param      WP_Post $post     The post.
      *

sites/trunk/wordpress.org/public_html/wp-content/plugins/gp-translation-helpers/templates/discussions-dashboard.php

@@ -49 +49 @@
     }
 
-
     $comments_by_post_id[ $_comment->comment_post_ID ][] = $_comment;
 
@@ -60 +59 @@
 
 // If the referenced comment is not in the current batch of comments we need to re-add it.
-foreach ( $bulk_comments as $original_id => $_post_id ) {
-    if ( ! isset( $comments_by_post_id[ $_comment->comment_post_ID ] ) ) {
-        $linked_comment = $_comment->comment_content;
-        $parts          = wp_parse_url( $linked_comment );
-        $comment_id     = intval( str_replace( 'comment-', '', $parts['fragment'] ) );
-        if ( $comment_id ) {
-            $comments_by_post_id[ $_comment->comment_post_ID ][] = get_comment( $comment_id );
+foreach ( $bulk_comments as $original_id => $_comments ) {
+    foreach ( $_comments as $_comment ) {
+        if ( ! isset( $comments_by_post_id[ $_comment->comment_post_ID ] ) ) {
+            $linked_comment = $_comment->comment_content;
+            $parts          = wp_parse_url( $linked_comment );
+            $comment_id     = intval( str_replace( 'comment-', '', $parts['fragment'] ) );
+            if ( $comment_id ) {
+                $comments_by_post_id[ $_comment->comment_post_ID ][] = get_comment( $comment_id );
+                if ( ! isset( $latest_comment_date_by_post_id[ $_comment->comment_post_ID ] ) ) {
+                    $latest_comment_date_by_post_id[ $_comment->comment_post_ID ] = $_comment->comment_date;
+                }
+            }
         }
     }
 }
 
-uasort(
+uksort(
     $comments_by_post_id,
     function( $a, $b ) use ( $latest_comment_date_by_post_id ) {
-        return $latest_comment_date_by_post_id[ $b->comment_post_ID ] <=> $latest_comment_date_by_post_id[ $a->comment_post_ID ];
+        return $latest_comment_date_by_post_id[ $b ] <=> $latest_comment_date_by_post_id[ $a ];
     }
 );

@@ -8 +8 @@
 .eslintrc.json
 tmp
+phpunit.xml.dist
+tests