Changeset 2567

Timestamp:

10/19/2010 06:51:32 AM (16 years ago)

Author:

johnjamesjacoby

Message:

First pass at Voices. Fixes #1330. Props GautamGupta

Location:

branches/plugin

Files:

• bbp-filters.php (modified) (1 diff)
• bbp-templatetags.php (modified) (12 diffs)
• bbp-themes/bbp-twentyten/loop-bbp_topics.php (modified) (1 diff)

branches/plugin/bbp-filters.php

@@ -5 +5 @@
 add_filter( 'bbp_get_forum_topic_reply_count', 'bbp_number_format' );
 
+// Add hooks to insert and delete post functions to update our voice counts
+add_action( 'wp_insert_post', 'bbp_update_topic_voice_count' );
+add_action( 'wp_delete_post', 'bbp_update_topic_voice_count' );
+
 ?>

branches/plugin/bbp-templatetags.php

@@ -244 +244 @@
          */
         function bbp_get_forum_last_active ( $forum_id = 0 ) {
-                if ( !$forum_id )
+                if ( empty( $forum_id ) )
                         $forum_id = bbp_get_forum_id();
 
@@ -283 +283 @@
          */
         function bbp_get_forum_topic_count ( $forum_id = 0 ) {
-                if ( !$forum_id )
+                if ( empty( $forum_id ) )
                         $forum_id = bbp_get_forum_id();
 
@@ -309 +309 @@
  */
 function bbp_update_forum_topic_count ( $new_topic_count, $forum_id = 0 ) {
-        if ( !$forum_id )
+        if ( empty( $forum_id ) )
                 $forum_id = bbp_get_forum_id();
 
@@ -348 +348 @@
          */
         function bbp_get_forum_topic_reply_count ( $forum_id = 0 ) {
-                if ( !$forum_id )
+                if ( empty( $forum_id ) )
                         $forum_id = bbp_get_forum_id();
 
@@ -378 +378 @@
  */
 function bbp_update_forum_topic_reply_count ( $new_topic_reply_count, $forum_id = 0 ) {
-        if ( !$forum_id )
+        if ( empty( $forum_id ) )
                 $forum_id = bbp_get_forum_id();
 
@@ -673 +673 @@
                         global $bbp_topics_template;
 
-                        if ( !$topic_id )
+                        if ( empty( $topic_id ) )
                                 $topic_id = bbp_get_topic_id();
 
@@ -711 +711 @@
          */
         function bbp_get_topic_last_active ( $topic_id = '' ) {
-                if ( !$topic_id )
+                if ( empty( $topic_id ) )
                         $topic_id = bbp_get_topic_id();
 
@@ -733 +733 @@
 }
         /**
-         * bbp_topic_reply_count ()
+         * bbp_get_topic_reply_count ()
          *
          * Return total post count of a topic
@@ -750 +750 @@
          */
         function bbp_get_topic_reply_count ( $topic_id = '' ) {
-                if ( !$topic_id )
+                if ( empty( $topic_id ) )
                         $topic_id = bbp_get_topic_id();
 
@@ -771 +771 @@
  * @todo make this not suck
  *
- * @uses bbp_get_topic_id(0
+ * @uses bbp_get_topic_id()
  * @uses apply_filters
  *
@@ -780 +780 @@
  */
 function bbp_update_topic_reply_count ( $new_topic_reply_count, $topic_id = '' ) {
-        if ( !$topic_id )
+        if ( empty( $topic_id ) )
                 $topic_id = bbp_get_topic_id();
 
         return apply_filters( 'bbp_update_topic_reply_count', (int)update_post_meta( $topic_id, 'bbp_topic_reply_count', $new_topic_reply_count ) );
+}
+
+/**
+ * bbp_topic_voice_count ()
+ *
+ * Output total voice count of a topic
+ *
+ * @package bbPress
+ * @subpackage Template Tags
+ * @since bbPress (1.2-r2565)
+ *
+ * @uses bbp_get_topic_voice_count()
+ * @uses apply_filters
+ *
+ * @param int $topic_id
+ */
+function bbp_topic_voice_count ( $topic_id = 0 ) {
+        echo bbp_get_topic_voice_count( $topic_id );
+}
+        /**
+         * bbp_get_topic_voice_count ()
+         *
+         * Return total voice count of a topic
+         *
+         * @package bbPress
+         * @subpackage Template Tags
+         * @since bbPress (1.2-r2565)
+         *
+         * @uses bbp_get_topic_id()
+         * @uses apply_filters
+         *
+         * @param int $topic_id
+         *
+         * @return int Voice count of the topic
+         */
+        function bbp_get_topic_voice_count ( $topic_id = 0 ) {
+                if ( empty( $topic_id ) )
+                        $topic_id = bbp_get_topic_id();
+
+                if ( !$voices = get_post_meta( $topic_id, 'bbp_topic_voice_count', true ) )
+                        $voices = bbp_update_topic_voice_count( $topic_id );
+
+                return apply_filters( 'bbp_get_topic_voice_count', (int)$voices, $topic_id );
+        }
+
+/**
+ * bbp_update_topic_voice_count ()
+ *
+ * Adjust the total voice count of a topic
+ *
+ * @package bbPress
+ * @subpackage Template Tags
+ * @since bbPress (1.2-r2565)
+ *
+ * @uses bbp_get_topic_id()
+ * @uses wpdb
+ * @uses apply_filters
+ *
+ * @todo cache
+ *
+ * @param int $topic_id optional Topic ID to update
+ *
+ * @return bool false on failure, voice count on success
+ */
+function bbp_update_topic_voice_count ( $topic_id = 0 ) {
+        global $wpdb;
+
+        if ( empty( $topic_id ) )
+                $topic_id = bbp_get_topic_id();
+
+        if ( !in_array( get_post_field( 'post_type', $topic_id ), array( BBP_TOPIC_POST_TYPE_ID, BBP_REPLY_POST_TYPE_ID ) ) ) /* If it is not a topic or reply, then we don't need it */
+                return false;
+
+        // If it's a reply, then get the parent (topic id)
+        if ( BBP_REPLY_POST_TYPE_ID == get_post_field( 'post_type', $topic_id ) )
+                $topic_id = get_post_field( 'post_parent', $topic_id );
+
+        // There should always be at least 1 voice
+        if ( !$voices = count( $wpdb->get_col( $wpdb->prepare( "SELECT DISTINCT post_author FROM $wpdb->posts WHERE ( post_parent = %d AND post_status = 'publish' AND post_type = '" . BBP_REPLY_POST_TYPE_ID . "' ) OR ( ID = %d AND post_type = '" . BBP_TOPIC_POST_TYPE_ID . "' );", $topic_id, $topic_id ) ) ) )
+                $voices = 1;
+
+        update_post_meta( $topic_id, 'bbp_topic_voice_count', $voices );
+
+        return apply_filters( 'bbp_update_topic_voice_count', (int)$voices );
 }
 
@@ -1189 +1273 @@
          */
         function bbp_get_reply_topic_id ( $reply_id = 0 ) {
-                if ( !$reply_id )
+                if ( empty( $reply_id ) )
                         $reply_id = bbp_get_reply_id();
 

branches/plugin/bbp-themes/bbp-twentyten/loop-bbp_topics.php

@@ -36 +36 @@
                                         <td class="bbp-topic-replies"><?php bbp_topic_reply_count(); ?></td>
 
-                                        <td class="bbp-topic-voices"><?php // @todo - bbp_topic_voice_count(); ?></td>
+                                        <td class="bbp-topic-voices"><?php bbp_topic_voice_count(); ?></td>
 
                                         <td class="bbp-topic-freshness">