Skip to:
Content

bbPress.org

Changeset 6136


Ignore:
Timestamp:
12/08/2016 03:34:29 PM (5 years ago)
Author:
johnjamesjacoby
Message:

Moderation: Move sub_action query var into a passable argument, so all actions can have reliable sub-action usage.

See #3032.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/includes/replies/functions.php

    r6134 r6136  
    16391639    }
    16401640
     1641    // Subaction?
     1642    $sub_action = in_array( $_GET['sub_action'], array( 'trash', 'untrash', 'delete' ), true )
     1643        ? sanitize_key( $_GET['sub_action'] )
     1644        : false;
     1645
    16411646    // Do the reply toggling
    16421647    $retval = bbp_toggle_reply( array(
    1643         'id'     => $reply_id,
    1644         'action' => $action,
    1645         'data'   => $post_data
     1648        'id'         => $reply_id,
     1649        'action'     => $action,
     1650        'sub_action' => $sub_action,
     1651        'data'       => $post_data
    16461652    ) );
    16471653
     
    16761682    // Parse the arguments
    16771683    $r = bbp_parse_args( $args, array(
    1678         'id'     => 0,
    1679         'action' => '',
    1680         'data'   => array()
     1684        'id'         => 0,
     1685        'action'     => '',
     1686        'sub_action' => '',
     1687        'data'       => array()
    16811688    ) );
    16821689
     
    17201727        case 'bbp_toggle_reply_trash' :
    17211728
    1722             // Subaction?
    1723             $sub_action = in_array( $_GET['sub_action'], array( 'trash', 'untrash', 'delete' ), true )
    1724                 ? $_GET['sub_action']
    1725                 : false;
    1726 
    17271729            // Bail if no subaction
    1728             if ( empty( $sub_action ) ) {
     1730            if ( empty( $r['sub_action'] ) ) {
    17291731                break;
    17301732            }
    17311733
    17321734            // Which subaction?
    1733             switch ( $sub_action ) {
     1735            switch ( $r['sub_action'] ) {
    17341736                case 'trash':
    17351737                    check_ajax_referer( "trash-{$nonce_suffix}" );
     
    17671769
    17681770    // Filter & return
    1769     return apply_filters( 'bbp_do_toggle_reply_handler', $retval, $r, $args );
     1771    return apply_filters( 'bbp_toggle_reply', $retval, $r, $args );
    17701772}
    17711773
     
    17911793
    17921794/**
    1793  * Return array of available reply handler actions
    1794  *
    1795  * @since 2.6.0 bbPress (rxxxx)
     1795 * Return array of available reply toggle actions
     1796 *
     1797 * @since 2.6.0 bbPress (r6133)
     1798 *
     1799 * @return array
    17961800 */
    17971801function bbp_get_reply_toggles() {
Note: See TracChangeset for help on using the changeset viewer.