Changeset 14427

Timestamp:

04/16/2025 08:51:45 PM (5 months ago)

Author:

coffee2code

Message:

Photo Directory, Flagged: Update approach to amend flagged stats to author column.

A custom author column has replaced the default author column and a new hook is available for cleaner insertion.

File:

• sites/trunk/wordpress.org/public_html/wp-content/plugins/photo-directory/inc/flagged.php (modified) (6 diffs)

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

@@ -1 +1 @@
 <?php
 /**
- * Post handling customizations.
+ * Flagged photo (aka potentially sensitive content) functionality.
  *
  * @package WordPressdotorg\Photo_Directory
@@ -48 +48 @@
         add_filter( 'display_post_states',                   [ __CLASS__, 'display_post_states' ], 10, 2 );
 
-        // Add count of user's flagged photos in author column.
-        //     Note: Priority after Admin::add_published_photos_count_to_author()
-        add_filter( 'the_author',                            [ __CLASS__, 'add_flagged_photos_count_to_author' ], 11 );
+        // Output count of user's flagged photos in author column.
+        add_filter( 'photo_author_column_data_end',          [ __CLASS__, 'output_flagged_photos_count_to_author' ] );
 
         // Add support to the post edit page.
@@ -240 +239 @@
             ? "<span class=\"update-plugins count-{$flagged_count}\"><span class=\"plugin-count\">{$flagged_count}</span></span>"
             : '';
- 
+
         // Add 'Flagged' link if user can read flagged photos.
         add_submenu_page(
@@ -265 +264 @@
 
     /**
-     * Appends the count of the published photos to author names in photo post
-     * listings.
-     *
-     * @param string $display_name The author's display name.
-     * @return string
-     */
-    public static function add_flagged_photos_count_to_author( $display_name ) {
-        global $authordata;
-
-        if ( ! is_admin() || ! Admin::should_include_photo_column() ) {
-            return $display_name;
-        }
-
-        // Close link to contributor's listing of photos.
-        $display_name .= '</a>';
+     * Outputs the count of the flagged photos for the post's author.
+     *
+     * @param WP_Post $post Post object.
+     */
+    public static function output_flagged_photos_count_to_author( $post ) {
+        $author_id = (int) get_post_field( 'post_author', $post );
+        $output = '';
 
         // Show number of flagged photos.
-        $flagged_count = User::count_flagged_photos( $authordata->ID );
+        $flagged_count = User::count_flagged_photos( $author_id );
         $flagged_link = '';
         if ( $flagged_count ) {
@@ -289 +280 @@
                     'post_type'   => Registrations::get_post_type(),
                     'post_status' => self::get_post_status(),
-                    'author'      => $authordata->ID,
+                    'author'      => $author_id,
                 ], 'edit.php' );
             }
-            $display_name .= '<div class="user-flagged-count">'
+            $output .= '<div class="user-flagged-count">'
                 . sprintf(
                     /* translators: %s: Count of user's flagged photos possibly linked to listing of their flagged photos. */
@@ -301 +292 @@
         }
 
-        // Prevent unbalanced tag.
-        $display_name .= '<a>';
-
-        return $display_name;
+        if ( $output ) {
+            echo $output;
+        }
     }