Changeset 6683 for trunk/src/includes/replies/template.php

Timestamp:

09/09/2017 05:27:44 AM (9 years ago)

Author:

johnjamesjacoby

Message:

Template: rework post class assignments.

This change avoids multiple reassignments to the same $classes variable name, and instead tries to name variables logically and merge them together when necessary. The performance difference is nil, as 'array_merge()` will perform similarly to how each array was reshaped when new classes would be added, but the human difference is only positive, from having clearer and easier to understand logic.

File:

• trunk/src/includes/replies/template.php (modified) (1 diff)

trunk/src/includes/replies/template.php

@@ -2176 +2176 @@
                 $bbp       = bbpress();
                 $reply_id  = bbp_get_reply_id( $reply_id );
-                $count     = isset( $bbp->reply_query->current_post ) ? $bbp->reply_query->current_post : 1;
-                $classes   = (array) $classes;
-                $classes[] = ( (int) $count % 2 ) ? 'even' : 'odd';
-                $classes[] = 'bbp-parent-forum-'   . bbp_get_reply_forum_id( $reply_id );
-                $classes[] = 'bbp-parent-topic-'   . bbp_get_reply_topic_id( $reply_id );
-                $classes[] = 'bbp-reply-position-' . bbp_get_reply_position( $reply_id, true );
-                $classes[] = 'user-id-' . bbp_get_reply_author_id( $reply_id );
-                $classes[] = ( bbp_get_reply_author_id( $reply_id ) === bbp_get_topic_author_id( bbp_get_reply_topic_id( $reply_id ) ) ? 'topic-author' : '' );
-                $classes   = array_filter( $classes );
-                $classes   = get_post_class( $classes, $reply_id );
-                $classes   = apply_filters( 'bbp_get_reply_class', $classes, $reply_id );
-                $retval    = 'class="' . implode( ' ', $classes ) . '"';
-
-                return $retval;
+                $topic_id  = bbp_get_reply_topic_id( $reply_id );
+                $author_id = bbp_get_reply_author_id( $reply_id );
+                $classes   = array_filter( (array) $classes );
+                $count     = isset( $bbp->reply_query->current_post )
+                        ? (int) $bbp->reply_query->current_post
+                        : 1;
+
+                // Get reply classes
+                $reply_classes = array(
+                        'loop-item-' . $count,
+                        ( $count % 2                                        )   ? 'even'         : 'odd',
+                        ( $author_id === bbp_get_topic_author_id( $topic_id ) ) ? 'topic-author' : '',
+                        'user-id-' . $author_id,
+                        'bbp-parent-forum-'   . bbp_get_reply_forum_id( $reply_id ),
+                        'bbp-parent-topic-'   . $topic_id,
+                        'bbp-reply-position-' . bbp_get_reply_position( $reply_id, true )
+                );
+
+                // Run the topic classes through the post-class filters, which also
+                // handles the escaping of each individual class.
+                $post_classes = get_post_class( array_merge( $classes, $reply_classes ), $reply_id );
+
+                // Filter
+                $new_classes  = apply_filters( 'bbp_get_reply_class', $post_classes, $reply_id, $classes );
+
+                // Return
+                return 'class="' . implode( ' ', $new_classes ) . '"';
         }