Skip to:
Content

bbPress.org


Ignore:
Timestamp:
07/29/2013 01:41:36 AM (13 years ago)
Author:
johnjamesjacoby
Message:

Switch some wp_insert_post() usages to wp_update_post(). Use bbp_get_forum(), bbp_get_topic(), and bbp_get_reply() where appropriate, and use objects instead of arrays to avoid escaping issues with wp_update_post().

File:
1 edited

Legend:

Unmodified
Added
Removed

  • trunk/includes/topics/functions.php

    r5036 r5055  
    215215
    216216                        // Forum does not exist
    217                         } elseif ( ! get_post( $posted_forum_id ) ) {
     217                        } elseif ( ! bbp_get_forum( $posted_forum_id ) ) {
    218218                                bbp_add_error( 'bbp_topic_forum_id', __( '<strong>ERROR</strong>: Forum does not exist.', 'bbpress' ) );
    219219
    … …  
    26622662 *
    26632663 * @param int $topic_id Topic id
    2664  * @uses get_post() To get the topic
     2664 * @uses bbp_get_topic() To get the topic
    26652665 * @uses do_action() Calls 'bbp_close_topic' with the topic id
    26662666 * @uses add_post_meta() To add the previous status to a meta
    2667  * @uses wp_insert_post() To update the topic with the new status
     2667 * @uses wp_update_post() To update the topic with the new status
    26682668 * @uses do_action() Calls 'bbp_opened_topic' with the topic id
    26692669 * @return mixed False or {@link WP_Error} on failure, topic id on success
    … …  
    26722672
    26732673        // Get topic
    2674         if ( !$topic = get_post( $topic_id, ARRAY_A ) )
     2674        $topic = bbp_get_topic( $topic_id );
     2675        if ( empty( $topic ) )
    26752676                return $topic;
    26762677
    26772678        // Bail if already closed
    2678         if ( bbp_get_closed_status_id() === $topic['post_status'] )
     2679        if ( bbp_get_closed_status_id() === $topic->post_status )
    26792680                return false;
    26802681
    … …  
    26832684
    26842685        // Add pre close status
    2685         add_post_meta( $topic_id, '_bbp_status', $topic['post_status'] );
     2686        add_post_meta( $topic_id, '_bbp_status', $topic->post_status );
    26862687
    26872688        // Set closed status
    2688         $topic['post_status'] = bbp_get_closed_status_id();
     2689        $topic->post_status = bbp_get_closed_status_id();
    26892690
    26902691        // No revisions
    … …  
    26922693
    26932694        // Update topic
    2694         $topic_id = wp_insert_post( $topic );
     2695        $topic_id = wp_update_post( $topic );
    26952696
    26962697        // Execute post close code
    … …  
    27072708 *
    27082709 * @param int $topic_id Topic id
    2709  * @uses get_post() To get the topic
     2710 * @uses bbp_get_topic() To get the topic
    27102711 * @uses do_action() Calls 'bbp_open_topic' with the topic id
    27112712 * @uses get_post_meta() To get the previous status
    27122713 * @uses delete_post_meta() To delete the previous status meta
    2713  * @uses wp_insert_post() To update the topic with the new status
     2714 * @uses wp_update_post() To update the topic with the new status
    27142715 * @uses do_action() Calls 'bbp_opened_topic' with the topic id
    27152716 * @return mixed False or {@link WP_Error} on failure, topic id on success
    … …  
    27182719
    27192720        // Get topic
    2720         if ( !$topic = get_post( $topic_id, ARRAY_A ) )
     2721        $topic = bbp_get_topic( $topic_id );
     2722        if ( empty( $topic ) )
    27212723                return $topic;
    27222724
    27232725        // Bail if already open
    2724         if ( bbp_get_closed_status_id() !== $topic['post_status'])
     2726        if ( bbp_get_closed_status_id() !== $topic->post_status )
    27252727                return false;
    27262728
    … …  
    27292731
    27302732        // Get previous status
    2731         $topic_status         = get_post_meta( $topic_id, '_bbp_status', true );
     2733        $topic_status       = get_post_meta( $topic_id, '_bbp_status', true );
    27322734
    27332735        // Set previous status
    2734         $topic['post_status'] = $topic_status;
     2736        $topic->post_status = $topic_status;
    27352737
    27362738        // Remove old status meta
    … …  
    27412743
    27422744        // Update topic
    2743         $topic_id = wp_insert_post( $topic );
     2745        $topic_id = wp_update_post( $topic );
    27442746
    27452747        // Execute post open code
    … …  
    27562758 *
    27572759 * @param int $topic_id Topic id
    2758  * @uses get_post() To get the topic
     2760 * @uses bbp_get_topic() To get the topic
    27592761 * @uses do_action() Calls 'bbp_spam_topic' with the topic id
    27602762 * @uses add_post_meta() To add the previous status to a meta
    2761  * @uses wp_insert_post() To update the topic with the new status
     2763 * @uses wp_update_post() To update the topic with the new status
    27622764 * @uses do_action() Calls 'bbp_spammed_topic' with the topic id
    27632765 * @return mixed False or {@link WP_Error} on failure, topic id on success
    … …  
    27662768
    27672769        // Get the topic
    2768         if ( !$topic = get_post( $topic_id, ARRAY_A ) )
     2770        $topic = bbp_get_topic( $topic_id );
     2771        if ( empty( $topic ) )
    27692772                return $topic;
    27702773
    27712774        // Bail if topic is spam
    2772         if ( bbp_get_spam_status_id() === $topic['post_status'] )
     2775        if ( bbp_get_spam_status_id() === $topic->post_status )
    27732776                return false;
    27742777
    … …  
    28152818
    28162819        // Add the original post status as post meta for future restoration
    2817         add_post_meta( $topic_id, '_bbp_spam_meta_status', $topic['post_status'] );
     2820        add_post_meta( $topic_id, '_bbp_spam_meta_status', $topic->post_status );
    28182821
    28192822        // Get topic tags
    … …  
    28382841
    28392842                        // Empty the topic of its tags
    2840                         $topic['tax_input'] = array( bbp_get_topic_tag_tax_id() => '' );
     2843                        $topic->tax_input = array( bbp_get_topic_tag_tax_id() => '' );
    28412844                }
    28422845        }
    28432846
    28442847        // Set post status to spam
    2845         $topic['post_status'] = bbp_get_spam_status_id();
     2848        $topic->post_status = bbp_get_spam_status_id();
    28462849
    28472850        // No revisions
    … …  
    28492852
    28502853        // Update the topic
    2851         $topic_id = wp_insert_post( $topic );
     2854        $topic_id = wp_update_post( $topic );
    28522855
    28532856        // Execute post spam code
    … …  
    28642867 *
    28652868 * @param int $topic_id Topic id
    2866  * @uses get_post() To get the topic
     2869 * @uses bbp_get_topic() To get the topic
    28672870 * @uses do_action() Calls 'bbp_unspam_topic' with the topic id
    28682871 * @uses get_post_meta() To get the previous status
    28692872 * @uses delete_post_meta() To delete the previous status meta
    2870  * @uses wp_insert_post() To update the topic with the new status
     2873 * @uses wp_update_post() To update the topic with the new status
    28712874 * @uses do_action() Calls 'bbp_unspammed_topic' with the topic id
    28722875 * @return mixed False or {@link WP_Error} on failure, topic id on success
    … …  
    28752878
    28762879        // Get the topic
    2877         if ( !$topic = get_post( $topic_id, ARRAY_A ) )
     2880        $topic = bbp_get_topic( $topic_id );
     2881        if ( empty( $topic ) )
    28782882                return $topic;
    28792883
    28802884        // Bail if already not spam
    2881         if ( bbp_get_spam_status_id() !== $topic['post_status'] )
     2885        if ( bbp_get_spam_status_id() !== $topic->post_status )
    28822886                return false;
    28832887
    … …  
    29122916
    29132917                // Set the tax_input of the topic
    2914                 $topic['tax_input'] = array( bbp_get_topic_tag_tax_id() => $terms );
     2918                $topic->tax_input = array( bbp_get_topic_tag_tax_id() => $terms );
    29152919
    29162920                // Delete pre-spam topic tag meta
    … …  
    29292933
    29302934        // Set post status to pre spam
    2931         $topic['post_status'] = $topic_status;
     2935        $topic->post_status = $topic_status;
    29322936
    29332937        // Delete pre spam meta
    … …  
    29382942
    29392943        // Update the topic
    2940         $topic_id = wp_insert_post( $topic );
     2944        $topic_id = wp_update_post( $topic );
    29412945
    29422946        // Execute post unspam code
Note: See TracChangeset for help on using the changeset viewer.