Changeset 2768

Timestamp:

01/07/2011 09:43:40 AM (15 years ago)

Author:

johnjamesjacoby

Message:

Introduce tag management utilities. Fixes #1424. Props GautamGupta via Google Code-in

Location:

branches/plugin

Files:

• bbp-includes/bbp-functions.php (modified) (1 diff)
• bbp-includes/bbp-hooks.php (modified) (1 diff)
• bbp-includes/bbp-topic-template.php (modified) (1 diff)
• bbp-themes/bbp-twentyten/css/bbpress.css (modified) (3 diffs)
• bbp-themes/bbp-twentyten/form-bbp_reply.php (modified) (1 diff)
• bbp-themes/bbp-twentyten/form-bbp_topic_tag.php (added)
• bbp-themes/bbp-twentyten/taxonomy-bbp_topic_tag.php (modified) (1 diff)
• bbpress.php (modified) (1 diff)

branches/plugin/bbp-includes/bbp-functions.php

@@ -1310 +1310 @@
 
 /**
+ * Handles the front end tag management (renaming, merging, destroying)
+ *
+ * @since bbPress (r2768)
+ *
+ * @uses check_admin_referer() To verify the nonce and check the referer
+ * @uses current_user_can() To check if the current user can edit/delete tags
+ * @uses bbPress::errors::add() To log the error messages
+ * @uses wp_update_term() To update the topic tag
+ * @uses get_term_link() To get the topic tag url
+ * @uses term_exists() To check if the topic tag already exists
+ * @uses wp_insert_term() To insert a topic tag
+ * @uses wp_delete_term() To delete the topic tag
+ * @uses home_url() To get the blog's home page url
+ * @uses do_action() Calls actions based on the actions with associated args
+ * @uses is_wp_error() To check if the value retrieved is a {@link WP_Error}
+ * @uses wp_redirect() To redirect to the url
+ */
+function bbp_manage_topic_tag_handler() {
+
+    if ( 'POST' == $_SERVER['REQUEST_METHOD'] && !empty( $_POST['action'] ) && in_array( $_POST['action'], array( 'bbp-update-topic-tag', 'bbp-merge-topic-tag', 'bbp-delete-topic-tag' ) ) && !empty( $_POST['tag-id'] ) ) {
+
+        global $bbp;
+
+        $action = $_POST['action'];
+        $tag_id = (int) $_POST['tag-id'];
+        $tag    = get_term( $tag_id, $bbp->topic_tag_id );
+
+        if ( is_wp_error( $tag ) && $tag->get_error_message() ) {
+            $append_error = $tag->get_error_message() . ' ';
+            $bbp->errors->add( 'bbp_manage_topic_invalid_tag', __( '<strong>ERROR</strong>: The following problem(s) have been found while getting the tag:' . $append_error . 'Please try again.', 'bbpress' ) );
+            return;
+        }
+
+        switch ( $action ) {
+            case 'bbp-update-topic-tag' :
+                check_admin_referer( 'update-tag_' . $tag_id );
+
+                if ( !current_user_can( 'edit_topic_tags' ) ) {
+                    $bbp->errors->add( 'bbp_manage_topic_tag_update_permissions', __( '<strong>ERROR</strong>: You do not have the permissions to edit the topic tags!', 'bbpress' ) );
+                    return;
+                }
+
+                if ( empty( $_POST['tag-name'] ) || !$name = $_POST['tag-name'] ) {
+                    $bbp->errors->add( 'bbp_manage_topic_tag_update_name', __( '<strong>ERROR</strong>: You need to enter a tag name!', 'bbpress' ) );
+                    return;
+                }
+
+                $slug = !empty( $_POST['tag-slug'] ) ? $_POST['tag-slug'] : '';
+
+                $tag = wp_update_term( $tag_id, $bbp->topic_tag_id, array( 'name' => $name, 'slug' => $slug ) );
+
+                if ( is_wp_error( $tag ) && $tag->get_error_message() ) {
+                    $append_error = $tag->get_error_message() . ' ';
+                    $bbp->errors->add( 'bbp_manage_topic_tag_update_error', __( '<strong>ERROR</strong>: The following problem(s) have been found while updating the tag:' . $append_error . 'Please try again.', 'bbpress' ) );
+                    return;
+                }
+
+                $redirect = get_term_link( $tag_id, $bbp->topic_tag_id );
+
+                // Update counts, etc...
+                do_action( 'bbp_update_topic_tag', $tag_id, $tag, $name, $slug );
+
+                break;
+
+            case 'bbp-merge-topic-tag'  :
+                check_admin_referer( 'merge-tag_' . $tag_id );
+
+                if ( !current_user_can( 'edit_topic_tags' ) ) {
+                    $bbp->errors->add( 'bbp_manage_topic_tag_merge_permissions', __( '<strong>ERROR</strong>: You do not have the permissions to edit the topic tags!', 'bbpress' ) );
+                    return;
+                }
+
+                if ( empty( $_POST['tag-name'] ) || !$name = $_POST['tag-name'] ) {
+                    $bbp->errors->add( 'bbp_manage_topic_tag_merge_name', __( '<strong>ERROR</strong>: You need to enter a tag name!', 'bbpress' ) );
+                    return;
+                }
+
+                // Much part of merge tags functionality taken from Scribu's Term Management Tools WordPress Plugin
+
+                if ( !$tag = term_exists( $name, $bbp->topic_tag_id ) )
+                    $tag = wp_insert_term( $name, $bbp->topic_tag_id );
+
+                if ( is_wp_error( $tag ) && $tag->get_error_message() ) {
+                    $append_error = $tag->get_error_message() . ' ';
+                    $bbp->errors->add( 'bbp_manage_topic_tag_merge_error', __( '<strong>ERROR</strong>: The following problem(s) have been found while merging the tags:' . $append_error . 'Please try again.', 'bbpress' ) );
+                    return;
+                }
+
+                $to_tag = $tag['term_id'];
+
+                if ( $tag_id == $to_tag ) {
+                    $bbp->errors->add( 'bbp_manage_topic_tag_merge_same', __( '<strong>ERROR</strong>: The tags which are being merged can not be the same.', 'bbpress' ) );
+                    return;
+                }
+
+                $tag = wp_delete_term( $tag_id, $bbp->topic_tag_id, array( 'default' => $to_tag, 'force_default' => true ) );
+
+                if ( is_wp_error( $tag ) && $tag->get_error_message() ) {
+                    $append_error = $tag->get_error_message() . ' ';
+                    $bbp->errors->add( 'bbp_manage_topic_tag_merge_error', __( '<strong>ERROR</strong>: The following problem(s) have been found while merging the tags:' . $append_error . 'Please try again.', 'bbpress' ) );
+                    return;
+                }
+
+                $redirect = get_term_link( (int) $to_tag, $bbp->topic_tag_id );
+
+                // Update counts, etc...
+                do_action( 'bbp_merge_topic_tag', $tag_id, $to_tag, $tag );
+
+                break;
+
+            case 'bbp-delete-topic-tag' :
+                check_admin_referer( 'delete-tag_' . $tag_id );
+
+                if ( !current_user_can( 'delete_topic_tags' ) ) {
+                    $bbp->errors->add( 'bbp_manage_topic_tag_delete_permissions', __( '<strong>ERROR</strong>: You do not have the permissions to delete the topic tags!', 'bbpress' ) );
+                    return;
+                }
+
+                $tag = wp_delete_term( $tag_id, $bbp->topic_tag_id );
+
+                if ( is_wp_error( $tag ) && $tag->get_error_message() ) {
+                    $append_error = $tag->get_error_message() . ' ';
+                    $bbp->errors->add( 'bbp_manage_topic_tag_delete_error', __( '<strong>ERROR</strong>: The following problem(s) have been found while deleting the tag:' . $append_error . 'Please try again.', 'bbpress' ) );
+                    return;
+                }
+
+                // We don't have any other place to go other than home! Or we may die because of the 404 disease
+                $redirect = home_url();
+
+                // Update counts, etc...
+                do_action( 'bbp_delete_topic_tag', $tag_id, $tag );
+
+                break;
+        }
+
+        // Redirect back
+        $redirect = ( !empty( $redirect ) && !is_wp_error( $redirect ) ) ? $redirect : home_url();
+        wp_redirect( $redirect );
+
+        // For good measure
+        exit();
+
+    }
+}
+
+/**
  * Handles the front end user editing
  *

branches/plugin/bbp-includes/bbp-hooks.php

@@ -84 +84 @@
 add_action( 'bbp_deactivation', 'bbp_remove_caps',   1     );
 add_action( 'bbp_deactivation', 'bbp_remove_roles',  2     );
+
+// Topic Tag Page
+add_action( 'template_redirect', 'bbp_manage_topic_tag_handler', 1 );
 
 // Profile Page

branches/plugin/bbp-includes/bbp-topic-template.php

@@ -1674 +1674 @@
             return;
 
-        $uri = esc_url( add_query_arg( array( 'action' => 'merge' ), bbp_get_topic_edit_url( $topic->id ) ) );
+        $uri = esc_url( add_query_arg( array( 'action' => 'merge' ), bbp_get_topic_edit_url( $topic->ID ) ) );
 
         return apply_filters( 'bbp_get_topic_merge_link', $link_before . '<a href="' . $uri . '">' . $merge_text . '</a>' . $link_after, $args );

branches/plugin/bbp-themes/bbp-twentyten/css/bbpress.css

@@ -203 +203 @@
 
 .bbp-topic-form,
-.bbp-reply-form {
+.bbp-reply-form,
+.bbp-topic-tag-form {
     clear: left;
 }
 
-p#bbp_topic_submit_container {
+#merge_tag,
+#delete_tag {
+    display: inline;
+}
+
+p#bbp_topic_submit_container,
+p#bbp_reply_submit_container {
     float: right;
 }
@@ -294 +301 @@
         background-color: #fffbcc;
     }
-    div.bbp-template-notice.error {
+    div.bbp-template-notice.error,
+    div.bbp-template-notice.warning {
         background-color: #ffebe8;
         border-color: #c00;
     }
-    div.bbp-template-notice.error a {
+    div.bbp-template-notice.error a,
+    div.bbp-template-notice.warning a {
         color: #c00;
     }
@@ -307 +316 @@
         line-height: 140%;
     }
+    fieldset div.bbp-template-notice {
+        margin-top: -10px;
+    }
 
 /* =Stickies

branches/plugin/bbp-themes/bbp-twentyten/form-bbp_reply.php

@@ -62 +62 @@
                         <?php endif; ?>
 
-                        <p id="bbp_topic_submit_container">
+                        <p id="bbp_reply_submit_container">
                             <button type="submit" tabindex="14" id="bbp_reply_submit" name="bbp_reply_submit"><?php _e( 'Submit', 'bbpress' ); ?></button>
                         </p>

branches/plugin/bbp-themes/bbp-twentyten/taxonomy-bbp_topic_tag.php

@@ -29 +29 @@
                         <?php get_template_part( 'loop', 'bbp_topics' ); ?>
 
+                        <?php get_template_part( 'form', 'bbp_topic_tag' ); ?>
+
                     </div>
                 </div><!-- #topic-tag -->

branches/plugin/bbpress.php

@@ -641 +641 @@
         // Register the topic tag taxonomy
         register_taxonomy(
-            $this->topic_tag_id, // The topic tag ID
-            $this->topic_id,     // The topic content type
+            $this->topic_tag_id, // The topic tag id
+            $this->topic_id,     // The topic post type
             $bbp_tt
         );