Skip to:
Content

BuddyPress.org

Changeset 11858


Ignore:
Timestamp:
02/15/2018 03:52:40 PM (7 years ago)
Author:
espellcaste
Message:

Make use of bp_is_post_request() instead of hardcoding POST verifications directly.

BuddyPress is not making use of the bp_is_post_request() in several ajax scenarios to confirm if the post request is indeed a POST request. Instead, it is hardcoding the check directly. This change updates those places making use of this function.

Props DjPaul

Fixes #7684

Location:
trunk/src
Files:
9 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/bp-core/bp-core-attachments.php

    r11841 r11858  
    11901190 */
    11911191function bp_attachments_cover_image_ajax_upload() {
    1192     if ( 'POST' !== strtoupper( $_SERVER['REQUEST_METHOD'] ) ) {
     1192    if ( ! bp_is_post_request() ) {
    11931193        wp_die();
    11941194    }
    … …  
    13791379 */
    13801380function bp_attachments_cover_image_ajax_delete() {
    1381     if ( 'POST' !== strtoupper( $_SERVER['REQUEST_METHOD'] ) ) {
     1381    if ( ! bp_is_post_request() ) {
    13821382        wp_send_json_error();
    13831383    }

  • trunk/src/bp-core/bp-core-avatars.php

    r11670 r11858  
    810810 */
    811811function bp_avatar_ajax_delete() {
    812     // Bail if not a POST action.
    813     if ( 'POST' !== strtoupper( $_SERVER['REQUEST_METHOD'] ) ) {
     812    if ( ! bp_is_post_request() ) {
    814813        wp_send_json_error();
    815814    }
    … …  
    953952 */
    954953function bp_avatar_ajax_upload() {
    955     // Bail if not a POST action.
    956     if ( 'POST' !== strtoupper( $_SERVER['REQUEST_METHOD'] ) ) {
     954    if ( ! bp_is_post_request() ) {
    957955        wp_die();
    958956    }
    … …  
    12381236 */
    12391237function bp_avatar_ajax_set() {
    1240     // Bail if not a POST action.
    1241     if ( 'POST' !== strtoupper( $_SERVER['REQUEST_METHOD'] ) ) {
     1238    if ( ! bp_is_post_request() ) {
    12421239        wp_send_json_error();
    12431240    }

  • trunk/src/bp-settings/bp-settings-actions.php

    r11705 r11858  
    2828 */
    2929function bp_settings_action_general() {
    30 
    31     // Bail if not a POST action.
    32     if ( 'POST' !== strtoupper( $_SERVER['REQUEST_METHOD'] ) )
    33         return;
     30    if ( ! bp_is_post_request() ) {
     31        return;
     32    }
    3433
    3534    // Bail if no submit action.
    36     if ( ! isset( $_POST['submit'] ) )
    37         return;
     35    if ( ! isset( $_POST['submit'] ) ) {
     36        return;
     37    }
    3838
    3939    // Bail if not in settings.
    40     if ( ! bp_is_settings_component() || ! bp_is_current_action( 'general' ) )
    41         return;
     40    if ( ! bp_is_settings_component() || ! bp_is_current_action( 'general' ) ) {
     41        return;
     42    }
    4243
    4344    // 404 if there are any additional action variables attached
    … …  
    260261 */
    261262function bp_settings_action_notifications() {
    262 
    263     // Bail if not a POST action.
    264     if ( 'POST' !== strtoupper( $_SERVER['REQUEST_METHOD'] ) )
    265         return;
     263    if ( ! bp_is_post_request() ) {
     264        return;
     265    }
    266266
    267267    // Bail if no submit action.
    268     if ( ! isset( $_POST['submit'] ) )
    269         return;
     268    if ( ! isset( $_POST['submit'] ) ) {
     269        return;
     270    }
    270271
    271272    // Bail if not in settings.
    272     if ( ! bp_is_settings_component() || ! bp_is_current_action( 'notifications' ) )
     273    if ( ! bp_is_settings_component() || ! bp_is_current_action( 'notifications' ) ) {
    273274        return false;
     275    }
    274276
    275277    // 404 if there are any additional action variables attached
    … …  
    307309 */
    308310function bp_settings_action_capabilities() {
    309 
    310     // Bail if not a POST action.
    311     if ( 'POST' !== strtoupper( $_SERVER['REQUEST_METHOD'] ) )
    312         return;
     311    if ( ! bp_is_post_request() ) {
     312        return;
     313    }
    313314
    314315    // Bail if no submit action.
    315     if ( ! isset( $_POST['capabilities-submit'] ) )
    316         return;
     316    if ( ! isset( $_POST['capabilities-submit'] ) ) {
     317        return;
     318    }
    317319
    318320    // Bail if not in settings.
    319     if ( ! bp_is_settings_component() || ! bp_is_current_action( 'capabilities' ) )
     321    if ( ! bp_is_settings_component() || ! bp_is_current_action( 'capabilities' ) ) {
    320322        return false;
     323    }
    321324
    322325    // 404 if there are any additional action variables attached
    … …  
    381384 */
    382385function bp_settings_action_delete_account() {
    383 
    384     // Bail if not a POST action.
    385     if ( 'POST' !== strtoupper( $_SERVER['REQUEST_METHOD'] ) )
    386         return;
     386    if ( ! bp_is_post_request() ) {
     387        return;
     388    }
    387389
    388390    // Bail if no submit action.
    389     if ( ! isset( $_POST['delete-account-understand'] ) )
    390         return;
     391    if ( ! isset( $_POST['delete-account-understand'] ) ) {
     392        return;
     393    }
    391394
    392395    // Bail if not in settings.
    393     if ( ! bp_is_settings_component() || ! bp_is_current_action( 'delete-account' ) )
     396    if ( ! bp_is_settings_component() || ! bp_is_current_action( 'delete-account' ) ) {
    394397        return false;
     398    }
    395399
    396400    // 404 if there are any additional action variables attached

  • trunk/src/bp-templates/bp-legacy/buddypress-functions.php

    r11821 r11858  
    780780 */
    781781function bp_legacy_theme_object_template_loader() {
    782     // Bail if not a POST action.
    783     if ( 'POST' !== strtoupper( $_SERVER['REQUEST_METHOD'] ) )
    784         return;
     782    if ( ! bp_is_post_request() ) {
     783        return;
     784    }
    785785
    786786    // Bail if no object passed.
    787     if ( empty( $_POST['object'] ) )
    788         return;
     787    if ( empty( $_POST['object'] ) ) {
     788        return;
     789    }
    789790
    790791    // Sanitize the object.
    … …  
    792793
    793794    // Bail if object is not an active component to prevent arbitrary file inclusion.
    794     if ( ! bp_is_active( $object ) )
    795         return;
     795    if ( ! bp_is_active( $object ) ) {
     796        return;
     797    }
    796798
    797799    /**
    … …  
    860862 */
    861863function bp_legacy_theme_activity_template_loader() {
    862     // Bail if not a POST action.
    863     if ( 'POST' !== strtoupper( $_SERVER['REQUEST_METHOD'] ) )
    864         return;
     864    if ( ! bp_is_post_request() ) {
     865        return;
     866    }
    865867
    866868    $scope = '';
    … …  
    921923    $bp = buddypress();
    922924
    923     // Bail if not a POST action.
    924     if ( 'POST' !== strtoupper( $_SERVER['REQUEST_METHOD'] ) )
    925         return;
     925    if ( ! bp_is_post_request() ) {
     926        return;
     927    }
    926928
    927929    // Check the nonce.
    … …  
    10101012    $bp = buddypress();
    10111013
    1012     // Bail if not a POST action.
    1013     if ( 'POST' !== strtoupper( $_SERVER['REQUEST_METHOD'] ) ) {
     1014    if ( ! bp_is_post_request() ) {
    10141015        return;
    10151016    }
    … …  
    10791080 */
    10801081function bp_legacy_theme_delete_activity() {
    1081     // Bail if not a POST action.
    1082     if ( 'POST' !== strtoupper( $_SERVER['REQUEST_METHOD'] ) )
    1083         return;
     1082    if ( ! bp_is_post_request() ) {
     1083        return;
     1084    }
    10841085
    10851086    // Check the nonce.
    … …  
    11171118 */
    11181119function bp_legacy_theme_delete_activity_comment() {
    1119     // Bail if not a POST action.
    1120     if ( 'POST' !== strtoupper( $_SERVER['REQUEST_METHOD'] ) )
    1121         return;
     1120    if ( ! bp_is_post_request() ) {
     1121        return;
     1122    }
    11221123
    11231124    // Check the nonce.
    … …  
    11591160    $bp = buddypress();
    11601161
    1161     // Bail if not a POST action.
    1162     if ( 'POST' !== strtoupper( $_SERVER['REQUEST_METHOD'] ) )
    1163         return;
     1162    if ( ! bp_is_post_request() ) {
     1163        return;
     1164    }
    11641165
    11651166    // Check that user is logged in, Activity Streams are enabled, and Akismet is present.
    … …  
    12041205function bp_legacy_theme_mark_activity_favorite() {
    12051206    // Bail if not a POST action.
    1206     if ( 'POST' !== strtoupper( $_SERVER['REQUEST_METHOD'] ) )
    1207         return;
     1207    if ( ! bp_is_post_request() ) {
     1208        return;
     1209    }
    12081210
    12091211    if ( ! isset( $_POST['nonce'] ) ) {
    … …  
    12331235 */
    12341236function bp_legacy_theme_unmark_activity_favorite() {
    1235     // Bail if not a POST action.
    1236     if ( 'POST' !== strtoupper( $_SERVER['REQUEST_METHOD'] ) )
    1237         return;
     1237    if ( ! bp_is_post_request() ) {
     1238        return;
     1239    }
    12381240
    12391241    if ( ! isset( $_POST['nonce'] ) ) {
    … …  
    12641266 */
    12651267function bp_legacy_theme_get_single_activity_content() {
    1266     // Bail if not a POST action.
    1267     if ( 'POST' !== strtoupper( $_SERVER['REQUEST_METHOD'] ) )
    1268         return;
     1268    if ( ! bp_is_post_request() ) {
     1269        return;
     1270    }
    12691271
    12701272    $activity_array = bp_activity_get_specific( array(
    … …  
    13041306 */
    13051307function bp_legacy_theme_ajax_invite_user() {
    1306     // Bail if not a POST action.
    1307     if ( 'POST' !== strtoupper( $_SERVER['REQUEST_METHOD'] ) )
    1308         return;
     1308    if ( ! bp_is_post_request() ) {
     1309        return;
     1310    }
    13091311
    13101312    check_ajax_referer( 'groups_invite_uninvite_user' );
    … …  
    13871389 */
    13881390function bp_legacy_theme_ajax_addremove_friend() {
    1389 
    1390     // Bail if not a POST action.
    1391     if ( 'POST' !== strtoupper( $_SERVER['REQUEST_METHOD'] ) )
    1392         return;
     1391    if ( ! bp_is_post_request() ) {
     1392        return;
     1393    }
    13931394
    13941395    // Cast fid as an integer.
    … …  
    14411442 */
    14421443function bp_legacy_theme_ajax_accept_friendship() {
    1443     // Bail if not a POST action.
    1444     if ( 'POST' !== strtoupper( $_SERVER['REQUEST_METHOD'] ) )
    1445         return;
     1444    if ( ! bp_is_post_request() ) {
     1445        return;
     1446    }
    14461447
    14471448    check_admin_referer( 'friends_accept_friendship' );
    … …  
    14611462 */
    14621463function bp_legacy_theme_ajax_reject_friendship() {
    1463     // Bail if not a POST action.
    1464     if ( 'POST' !== strtoupper( $_SERVER['REQUEST_METHOD'] ) )
    1465         return;
     1464    if ( ! bp_is_post_request() ) {
     1465        return;
     1466    }
    14661467
    14671468    check_admin_referer( 'friends_reject_friendship' );
    … …  
    14811482 */
    14821483function bp_legacy_theme_ajax_joinleave_group() {
    1483     // Bail if not a POST action.
    1484     if ( 'POST' !== strtoupper( $_SERVER['REQUEST_METHOD'] ) )
    1485         return;
     1484    if ( ! bp_is_post_request() ) {
     1485        return;
     1486    }
    14861487
    14871488    // Cast gid as integer.
    … …  
    15521553 */
    15531554function bp_legacy_theme_ajax_close_notice() {
    1554     // Bail if not a POST action.
    1555     if ( 'POST' !== strtoupper( $_SERVER['REQUEST_METHOD'] ) )
    1556         return;
     1555    if ( ! bp_is_post_request() ) {
     1556        return;
     1557    }
    15571558
    15581559    $nonce_check = isset( $_POST['nonce'] ) && wp_verify_nonce( wp_unslash( $_POST['nonce'] ), 'bp_messages_close_notice' );
    … …  
    15841585 */
    15851586function bp_legacy_theme_ajax_messages_send_reply() {
    1586     // Bail if not a POST action.
    1587     if ( 'POST' !== strtoupper( $_SERVER['REQUEST_METHOD'] ) )
    1588         return;
     1587    if ( ! bp_is_post_request() ) {
     1588        return;
     1589    }
    15891590
    15901591    check_ajax_referer( 'messages_send_message' );

  • trunk/src/bp-templates/bp-nouveau/includes/activity/ajax.php

    r11856 r11858  
    8686 */
    8787function bp_nouveau_ajax_mark_activity_favorite() {
    88     // Bail if not a POST action.
    89     if ( 'POST' !== strtoupper( $_SERVER['REQUEST_METHOD'] ) ) {
     88    if ( ! bp_is_post_request() ) {
    9089        wp_send_json_error();
    9190    }
    … …  
    127126 */
    128127function bp_nouveau_ajax_unmark_activity_favorite() {
    129     // Bail if not a POST action.
    130     if ( 'POST' !== strtoupper( $_SERVER['REQUEST_METHOD'] ) ) {
     128    if ( ! bp_is_post_request() ) {
    131129        wp_send_json_error();
    132130    }
    … …  
    164162 */
    165163function bp_nouveau_ajax_clear_new_mentions() {
    166     // Bail if not a POST action.
    167     if ( 'POST' !== strtoupper( $_SERVER['REQUEST_METHOD'] ) ) {
     164    if ( ! bp_is_post_request() ) {
    168165        wp_send_json_error();
    169166    }
    … …  
    194191
    195192    // Bail if not a POST action.
    196     if ( 'POST' !== strtoupper( $_SERVER['REQUEST_METHOD'] ) ) {
     193    if ( ! bp_is_post_request() ) {
    197194        wp_send_json_error( $response );
    198195    }
    … …  
    266263
    267264    // Bail if not a POST action.
    268     if ( 'POST' !== strtoupper( $_SERVER['REQUEST_METHOD'] ) ) {
     265    if ( ! bp_is_post_request() ) {
    269266        wp_send_json_error( $response );
    270267    }
    … …  
    327324
    328325    // Bail if not a POST action.
    329     if ( 'POST' !== strtoupper( $_SERVER['REQUEST_METHOD'] ) ) {
     326    if ( ! bp_is_post_request() ) {
    330327        wp_send_json_error( $response );
    331328    }
    … …  
    564561
    565562    // Bail if not a POST action.
    566     if ( 'POST' !== strtoupper( $_SERVER['REQUEST_METHOD'] ) ) {
     563    if ( ! bp_is_post_request() ) {
    567564        wp_send_json_error( $response );
    568565    }

  • trunk/src/bp-templates/bp-nouveau/includes/ajax.php

    r11855 r11858  
    1616 */
    1717function bp_nouveau_ajax_object_template_loader() {
    18     if ( 'POST' !== strtoupper( $_SERVER['REQUEST_METHOD'] ) ) {
     18    if ( ! bp_is_post_request() ) {
    1919        wp_send_json_error();
    2020    }

  • trunk/src/bp-templates/bp-nouveau/includes/friends/ajax.php

    r11856 r11858  
    6363
    6464    // Bail if not a POST action.
    65     if ( 'POST' !== strtoupper( $_SERVER['REQUEST_METHOD'] ) ) {
     65    if ( ! bp_is_post_request() ) {
    6666        wp_send_json_error( $response );
    6767    }

  • trunk/src/bp-templates/bp-nouveau/includes/groups/ajax.php

    r11856 r11858  
    4040
    4141    // Bail if not a POST action.
    42     if ( 'POST' !== strtoupper( $_SERVER['REQUEST_METHOD'] ) || empty( $_POST['action'] ) ) {
     42    if ( ! bp_is_post_request() || empty( $_POST['action'] ) ) {
    4343        wp_send_json_error( $response );
    4444    }

  • trunk/src/bp-xprofile/bp-xprofile-actions.php

    r10825 r11858  
    5555
    5656    // Bail if not a POST action.
    57     if ( 'POST' !== strtoupper( $_SERVER['REQUEST_METHOD'] ) ) {
     57    if ( ! bp_is_post_request() ) {
    5858        return;
    5959    }
Note: See TracChangeset for help on using the changeset viewer.