Skip to:
Content

BuddyPress.org

Changeset 13891


Ignore:
Timestamp:
06/02/2024 01:12:39 AM (2 years ago)
Author:
espellcaste
Message:

WPCS: Part V: miscellaneous fixes for some of the files of the core component.

Follow-up to [13883], [13886], [13887], and [13888]

See #9164 and #7228

Location:
trunk/src/bp-core
Files:
3 edited

Legend:

Unmodified
Added
Removed

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

    r13741 r13891  
    3232
    3333                // Add secondary parent item for all BuddyPress components.
    34                 $wp_admin_bar->add_node( array(
    35                         'parent'    => 'my-account',
    36                         'id'        => 'my-account-buddypress',
    37                         'title'     => __( 'My Account', 'buddypress' ),
    38                         'group'     => true,
    39                         'meta'      => array(
    40                                 'class' => 'ab-sub-secondary'
     34                $wp_admin_bar->add_node(
     35                        array(
     36                                'parent' => 'my-account',
     37                                'id'     => 'my-account-buddypress',
     38                                'title'  => __( 'My Account', 'buddypress' ),
     39                                'group'  => true,
     40                                'meta'   => array(
     41                                        'class' => 'ab-sub-secondary',
     42                                ),
    4143                        )
    42                 ) );
     44                );
    4345        }
    4446}
    … …  
    5254function bp_core_load_admin_bar() {
    5355        // Show the Toolbar for logged out users.
    54         if ( ! is_user_logged_in() && (int) bp_get_option( 'hide-loggedout-adminbar' ) != 1 ) {
     56        if ( ! is_user_logged_in() && (int) bp_get_option( 'hide-loggedout-adminbar' ) !== 1 ) {
    5557                show_admin_bar( true );
    5658        }
    … …  
    6870 */
    6971function bp_core_load_admin_bar_css() {
    70         add_action( 'bp_enqueue_scripts',       'bp_core_enqueue_admin_bar_css', 1 );
     72        add_action( 'bp_enqueue_scripts', 'bp_core_enqueue_admin_bar_css', 1 );
    7173        add_action( 'bp_admin_enqueue_scripts', 'bp_core_enqueue_admin_bar_css', 1 );
    7274}

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

    r13890 r13891  
    7474 * @since 3.0.0
    7575 *
    76  * @return array See wp_upload_dir().
     76 * @param array $args {
     77 *    @type int    $object_id        The ID of the object (Required).
     78 *    @type string $object_directory The object directory (eg: members, groups) (Required).
     79 * }
     80 * @return array
    7781 */
    7882function bp_attachments_cover_image_upload_dir( $args = array() ) {
    … …  
    102106
    103107        // Set the subdir.
    104         $subdir  = '/' . $r['object_directory'] . '/' . $r['object_id'] . '/cover-image';
     108        $subdir = '/' . $r['object_directory'] . '/' . $r['object_id'] . '/cover-image';
    105109
    106110        $upload_dir = bp_attachments_uploads_dir_get();
    … …  
    111115         * @since 2.4.0
    112116         *
    113          * @param array $value      Array containing the path, URL, and other helpful settings.
    114          * @param array $upload_dir The original Uploads dir.
     117         * @param array $image_upload_data Array containing the path, URL, and other helpful settings.
     118         * @param array $upload_dir        The original Uploads dir.
    115119         */
    116         return apply_filters( 'bp_attachments_cover_image_upload_dir', array(
    117                 'path'    => $upload_dir['basedir'] . $subdir,
    118                 'url'     => set_url_scheme( $upload_dir['baseurl'] ) . $subdir,
    119                 'subdir'  => $subdir,
    120                 'basedir' => $upload_dir['basedir'],
    121                 'baseurl' => set_url_scheme( $upload_dir['baseurl'] ),
    122                 'error'   => false,
    123         ), $upload_dir );
     120        return apply_filters(
     121                'bp_attachments_cover_image_upload_dir',
     122                array(
     123                        'path'    => $upload_dir['basedir'] . $subdir,
     124                        'url'     => set_url_scheme( $upload_dir['baseurl'] ) . $subdir,
     125                        'subdir'  => $subdir,
     126                        'basedir' => $upload_dir['basedir'],
     127                        'baseurl' => set_url_scheme( $upload_dir['baseurl'] ),
     128                        'error'   => false,
     129                ),
     130                $upload_dir
     131        );
    124132}
    125133
    … …  
    197205                        // Loop to explode keys using '|'.
    198206                        foreach ( $allowed_types as $allowed_type ) {
    199                                 $t = explode( '|', $allowed_type );
     207                                $t    = explode( '|', $allowed_type );
    200208                                $exts = array_merge( $exts, (array) $t );
    201209                        }
    … …  
    417425                                'crop_h'        => $r['crop_h'],
    418426                                'crop_x'        => $r['crop_x'],
    419                                 'crop_y'        => $r['crop_y']
     427                                'crop_y'        => $r['crop_y'],
    420428                        )
    421429                );
    422430
    423         // It's a cover image we need to fit it to feature's dimensions.
     431                // It's a cover image we need to fit it to feature's dimensions.
    424432        } elseif ( 'cover_image' === $type ) {
    425433                $cover_image = bp_attachments_cover_image_generate_file(
    … …  
    427435                                'file'            => $image_file_path,
    428436                                'component'       => $r['component'],
    429                                 'cover_image_dir' => $attachment_data['path']
     437                                'cover_image_dir' => $attachment_data['path'],
    430438                        )
    431439                );
    … …  
    520528                        $attachment_data = trailingslashit( $type_dir ) . $r['file'];
    521529                }
    522 
    523530        } else {
    524531                $file = false;
    525532
    526533                // Open the directory and get the first file.
    527                 if ( $att_dir = opendir( $type_dir ) ) {
     534                $att_dir = opendir( $type_dir );
     535                if ( $att_dir ) {
    528536
    529537                        while ( false !== ( $attachment_file = readdir( $att_dir ) ) ) {
    … …  
    604612                'file_data_name'      => 'file',
    605613                'multipart_params'    => array(
    606                         'action'          => 'bp_upload_attachment',
    607                         '_wpnonce'        => wp_create_nonce( 'bp-uploader' ),
     614                        'action'   => 'bp_upload_attachment',
     615                        '_wpnonce' => wp_create_nonce( 'bp-uploader' ),
    608616                ),
    609617                'url'                 => admin_url( 'admin-ajax.php', 'relative' ),
    610618                'flash_swf_url'       => includes_url( 'js/plupload/plupload.flash.swf' ),
    611619                'silverlight_xap_url' => includes_url( 'js/plupload/plupload.silverlight.xap' ),
    612                 'filters' => array(
    613                         'max_file_size'   => $max_upload_size . 'b',
     620                'filters'             => array(
     621                        'max_file_size' => $max_upload_size . 'b',
    614622                ),
    615623                'multipart'           => true,
    … …  
    625633
    626634        $settings = array(
    627                 'defaults' => $defaults,
    628                 'browser'  => array(
     635                'defaults'      => $defaults,
     636                'browser'       => array(
    629637                        'mobile'    => wp_is_mobile(),
    630638                        'supported' => _device_can_upload(),
    … …  
    692700                        /* translators: %s: File name. */
    693701                        'error_uploading'           => __( '“%s” has failed to upload.', 'buddypress' ),
    694                         'has_avatar_warning'        => __( 'If you'd like to delete the existing profile photo but not upload a new one, please use the delete tab.', 'buddypress' )
     702                        'has_avatar_warning'        => __( 'If you'd like to delete the existing profile photo but not upload a new one, please use the delete tab.', 'buddypress' ),
    695703                )
    696704        );
    … …  
    705713 * to set specific script data.
    706714 *
    707  * @param string $class Name of the class extending BP_Attachment (eg: BP_Attachment_Avatar).
     715 * @param string $class_name Name of the class extending BP_Attachment (eg: BP_Attachment_Avatar).
    708716 * @return null|WP_Error
    709717 */
    710 function bp_attachments_enqueue_scripts( $class = '' ) {
     718function bp_attachments_enqueue_scripts( $class_name = '' ) {
     719        $class = $class_name;
     720
    711721        // Enqueue me just once per page, please.
    712722        if ( did_action( 'bp_attachments_enqueue_scripts' ) ) {
    … …  
    719729
    720730        // Get an instance of the class and get the script data.
    721         $attachment  = new $class;
     731        $attachment  = new $class();
    722732        $script_data = $attachment->script_data();
    723733
    … …  
    766776
    767777        // Merge other arguments.
    768         $ui_args = array_intersect_key( $args, array(
    769                 'file_data_name' => true,
    770                 'browse_button'  => true,
    771                 'container'      => true,
    772                 'drop_element'   => true,
    773         ) );
     778        $ui_args = array_intersect_key(
     779                $args,
     780                array(
     781                        'file_data_name' => true,
     782                        'browse_button'  => true,
     783                        'container'      => true,
     784                        'drop_element'   => true,
     785                )
     786        );
    774787
    775788        $defaults = array_merge( $defaults, $ui_args );
    … …  
    780793
    781794        if ( isset( $args['mime_types'] ) && $args['mime_types'] ) {
    782                 $defaults['filters']['mime_types'] =  array( array( 'extensions' => $args['mime_types'] ) );
     795                $defaults['filters']['mime_types'] = array( array( 'extensions' => $args['mime_types'] ) );
    783796        }
    784797
    … …  
    793806                // Include the cropping informations for avatars.
    794807                $settings['crop'] = array(
    795                         'full_h'  => bp_core_avatar_full_height(),
    796                         'full_w'  => bp_core_avatar_full_width(),
     808                        'full_h' => bp_core_avatar_full_height(),
     809                        'full_w' => bp_core_avatar_full_width(),
    797810                );
    798811
    … …  
    811824                // Init the Avatar nav.
    812825                $avatar_nav = array(
    813                         'upload'  => array(
     826                        'upload' => array(
    814827                                'id'      => 'upload',
    815828                                'caption' => __( 'Upload', 'buddypress' ),
    816                                 'order'   => 0
     829                                'order'   => 0,
    817830                        ),
    818                         'delete'  => array(
     831                        'delete' => array(
    819832                                'id'      => 'delete',
    820833                                'caption' => __( 'Delete', 'buddypress' ),
    821834                                'order'   => 100,
    822                                 'hide'    => (int) ! $has_avatar
     835                                'hide'    => (int) ! $has_avatar,
    823836                        ),
    824837                );
    … …  
    851864                                'id'      => 'camera',
    852865                                'caption' => __( 'Take Photo', 'buddypress' ),
    853                                 'order'   => 10
     866                                'order'   => 10,
    854867                        );
    855868
    856869                        // Set warning messages.
    857870                        $strings['camera_warnings'] = array(
    858                                 'requesting'  => __( 'Please allow us to access to your camera.', 'buddypress'),
    859                                 'loading'     => __( 'Please wait as we access your camera.', 'buddypress' ),
    860                                 'loaded'      => __( 'Camera loaded. Click on the "Capture" button to take your photo.', 'buddypress' ),
    861                                 'noaccess'    => __( 'It looks like you do not have a webcam or we were unable to get permission to use your webcam. Please upload a photo instead.', 'buddypress' ),
    862                                 'errormsg'    => __( 'Your browser is not supported. Please upload a photo instead.', 'buddypress' ),
    863                                 'videoerror'  => __( 'Video error. Please upload a photo instead.', 'buddypress' ),
    864                                 'ready'       => __( 'Your profile photo is ready. Click on the "Save" button to use this photo.', 'buddypress' ),
    865                                 'nocapture'   => __( 'No photo was captured. Click on the "Capture" button to take your photo.', 'buddypress' ),
     871                                'requesting' => __( 'Please allow us to access to your camera.', 'buddypress' ),
     872                                'loading'    => __( 'Please wait as we access your camera.', 'buddypress' ),
     873                                'loaded'     => __( 'Camera loaded. Click on the "Capture" button to take your photo.', 'buddypress' ),
     874                                'noaccess'   => __( 'It looks like you do not have a webcam or we were unable to get permission to use your webcam. Please upload a photo instead.', 'buddypress' ),
     875                                'errormsg'   => __( 'Your browser is not supported. Please upload a photo instead.', 'buddypress' ),
     876                                'videoerror' => __( 'Video error. Please upload a photo instead.', 'buddypress' ),
     877                                'ready'      => __( 'Your profile photo is ready. Click on the "Save" button to use this photo.', 'buddypress' ),
     878                                'nocapture'  => __( 'No photo was captured. Click on the "Capture" button to take your photo.', 'buddypress' ),
    866879                        );
    867880                }
    … …  
    886899                $settings['nav'] = bp_sort_by_key( apply_filters( 'bp_attachments_avatar_nav', $avatar_nav, $object ), 'order', 'num' );
    887900
    888         // Specific to BuddyPress cover images.
     901                // Specific to BuddyPress cover images.
    889902        } elseif ( 'bp_cover_image_upload' === $defaults['multipart_params']['action'] ) {
    890903
    … …  
    908921
    909922                // Set warning messages.
    910                 $strings['cover_image_warnings'] = apply_filters( 'bp_attachments_cover_image_ui_warnings', array(
    911                         'dimensions'  => sprintf(
    912                                 /* translators: 1: the advised width size in pixels. 2: the advised height size in pixels. */
    913                                 __( 'For better results, make sure to upload an image that is larger than %1$spx wide, and %2$spx tall.', 'buddypress' ),
    914                                 (int) $cover_dimensions['width'],
    915                                 (int) $cover_dimensions['height']
    916                         ),
    917                 ) );
     923                $strings['cover_image_warnings'] = apply_filters(
     924                        'bp_attachments_cover_image_ui_warnings',
     925                        array(
     926                                'dimensions' => sprintf(
     927                                        /* translators: 1: the advised width size in pixels. 2: the advised height size in pixels. */
     928                                        __( 'For better results, make sure to upload an image that is larger than %1$spx wide, and %2$spx tall.', 'buddypress' ),
     929                                        (int) $cover_dimensions['width'],
     930                                        (int) $cover_dimensions['height']
     931                                ),
     932                        )
     933                );
    918934        }
    919935
    … …  
    936952        }
    937953
    938         wp_enqueue_script ( 'bp-plupload' );
    939         wp_localize_script( 'bp-plupload', 'BP_Uploader', array( 'strings' => $strings, 'settings' => $settings ) );
     954        wp_enqueue_script( 'bp-plupload' );
     955        wp_localize_script(
     956                'bp-plupload',
     957                'BP_Uploader',
     958                array(
     959                        'strings'  => $strings,
     960                        'settings' => $settings,
     961                )
     962        );
    940963
    941964        /**
    … …  
    9871010                                        $can = (bool) groups_is_user_admin( bp_loggedin_user_id(), $args['item_id'] ) || bp_current_user_can( 'bp_moderate' );
    9881011                                }
    989                         // User profile photo.
     1012                                // User profile photo.
    9901013                        } elseif ( bp_is_active( 'members' ) && 'user' === $args['object'] ) {
    9911014                                $can = bp_loggedin_user_id() === (int) $args['item_id'] || bp_current_user_can( 'bp_moderate' );
    9921015                        }
    993                 /**
    994                  * No avatar arguments, fallback to bp_user_can_create_groups()
    995                  * or bp_is_item_admin()
    996                  */
     1016                        /**
     1017                         * No avatar arguments, fallback to bp_user_can_create_groups()
     1018                         * or bp_is_item_admin()
     1019                         */
     1020                } elseif ( bp_is_group_create() ) {
     1021                                $can = bp_user_can_create_groups();
    9971022                } else {
    998                         if ( bp_is_group_create() ) {
    999                                 $can = bp_user_can_create_groups();
    1000                         } else {
    1001                                 $can = bp_is_item_admin();
    1002                         }
     1023                        $can = bp_is_item_admin();
    10031024                }
    10041025        }
    … …  
    10271048                wp_send_json( $response );
    10281049
    1029         /**
    1030         * Send specific json response
    1031         * the html4 Plupload handler requires a text/html content-type for older IE.
    1032         * See https://core.trac.wordpress.org/ticket/31037
    1033         */
     1050                /**
     1051                * Send specific json response
     1052                * the html4 Plupload handler requires a text/html content-type for older IE.
     1053                * See https://core.trac.wordpress.org/ticket/31037
     1054                */
    10341055        } else {
    10351056                echo wp_json_encode( $response );
    … …  
    10451066 *
    10461067 * @param string $slug Template part slug. eg 'uploader' for 'uploader.php'.
    1047  * @return bool
    10481068 */
    10491069function bp_attachments_get_template_part( $slug ) {
    … …  
    11371157
    11381158        // Current component is not supported.
    1139         if ( ! in_array( $component, $settings['components'] ) ) {
     1159        if ( ! in_array( $component, $settings['components'], true ) ) {
    11401160                return false;
    11411161        }
    … …  
    11551175function bp_attachments_get_cover_image_dimensions( $component = 'members' ) {
    11561176        // Let's prevent notices when setting the warning strings.
    1157         $default = array( 'width' => 0, 'height' => 0 );
     1177        $default = array(
     1178                'width'  => 0,
     1179                'height' => 0,
     1180        );
    11581181
    11591182        $settings = bp_attachments_get_cover_image_settings( $component );
    … …  
    12011224        }
    12021225
    1203         if ( ( bp_is_group_admin_page() && 'group-cover-image' == bp_get_group_current_admin_tab() )
     1226        if ( ( bp_is_group_admin_page() && 'group-cover-image' === bp_get_group_current_admin_tab() )
    12041227                || ( bp_is_group_create() && bp_is_group_creation_step( 'group-cover-image' ) ) ) {
    12051228                $retval = ! bp_disable_group_cover_image_uploads();
    … …  
    12221245        }
    12231246
    1224         $cover_src = bp_attachments_get_attachment( 'url', array(
    1225                 'item_id'   => $user_id,
    1226         ) );
     1247        $cover_src = bp_attachments_get_attachment(
     1248                'url',
     1249                array(
     1250                        'item_id' => $user_id,
     1251                )
     1252        );
    12271253
    12281254        return (bool) apply_filters( 'bp_attachments_get_user_has_cover_image', $cover_src, $user_id );
    … …  
    12431269        }
    12441270
    1245         $cover_src = bp_attachments_get_attachment( 'url', array(
    1246                 'object_dir' => 'groups',
    1247                 'item_id'    => $group_id,
    1248         ) );
     1271        $cover_src = bp_attachments_get_attachment(
     1272                'url',
     1273                array(
     1274                        'object_dir' => 'groups',
     1275                        'item_id'    => $group_id,
     1276                )
     1277        );
    12491278
    12501279        return (bool) apply_filters( 'bp_attachments_get_group_has_cover_image', $cover_src, $group_id );
    … …  
    13301359                'cover_file'     => $cover_file,
    13311360                'cover_basename' => $cover_basename,
    1332                 'is_too_small'   => $is_too_small
     1361                'is_too_small'   => $is_too_small,
    13331362        );
    13341363}
    … …  
    13381367 *
    13391368 * @since 2.4.0
    1340  *
    1341  * @return string|null A json object containing success data if the upload succeeded,
    1342  *                     error message otherwise.
    13431369 */
    13441370function bp_attachments_cover_image_ajax_upload() {
     1371
    13451372        if ( ! bp_is_post_request() ) {
    13461373                wp_die();
    … …  
    13501377
    13511378        // Sending the json response will be different if the current Plupload runtime is html4.
    1352         $is_html4 = ! empty( $_POST['html4' ] );
     1379        $is_html4 = ! empty( $_POST['html4'] );
    13531380
    13541381        if ( empty( $_POST['bp_params'] ) ) {
    … …  
    13831410        // Member's cover image.
    13841411        if ( 'user' === $bp_params['object'] ) {
    1385                 $object_data = array( 'dir' => 'members', 'component' => 'members' );
     1412                $object_data = array(
     1413                        'dir'       => 'members',
     1414                        'component' => 'members',
     1415                );
    13861416
    13871417                if ( ! bp_displayed_user_id() && ! empty( $bp_params['item_id'] ) ) {
    1388                         $needs_reset = array( 'key' => 'displayed_user', 'value' => $bp->displayed_user );
     1418                        $needs_reset            = array(
     1419                                'key'   => 'displayed_user',
     1420                                'value' => $bp->displayed_user,
     1421                        );
    13891422                        $bp->displayed_user->id = $bp_params['item_id'];
    13901423                }
    13911424
    1392         // Group's cover image.
     1425                // Group's cover image.
    13931426        } elseif ( 'group' === $bp_params['object'] ) {
    1394                 $object_data = array( 'dir' => 'groups', 'component' => 'groups' );
     1427                $object_data = array(
     1428                        'dir'       => 'groups',
     1429                        'component' => 'groups',
     1430                );
    13951431
    13961432                if ( ! bp_get_current_group_id() && ! empty( $bp_params['item_id'] ) ) {
    1397                         $needs_reset = array( 'component' => 'groups', 'key' => 'current_group', 'value' => $bp->groups->current_group );
     1433                        $needs_reset               = array(
     1434                                'component' => 'groups',
     1435                                'key'       => 'current_group',
     1436                                'value'     => $bp->groups->current_group,
     1437                        );
    13981438                        $bp->groups->current_group = groups_get_group( $bp_params['item_id'] );
    13991439                }
    14001440
    1401         // Other object's cover image.
     1441                // Other object's cover image.
    14021442        } else {
    14031443                $object_data = apply_filters( 'bp_attachments_cover_image_object_dir', array(), $bp_params['object'] );
    … …  
    14271467
    14281468        $cover_image_attachment = new BP_Attachment_Cover_Image();
    1429         $uploaded = $cover_image_attachment->upload( $_FILES );
     1469        $uploaded               = $cover_image_attachment->upload( $_FILES );
    14301470
    14311471        // Reset objects.
    … …  
    14401480        if ( ! empty( $uploaded['error'] ) ) {
    14411481                // Upload error response.
    1442                 bp_attachments_json_response( false, $is_html4, array(
    1443                         'type'    => 'upload_error',
    1444                         'message' => sprintf(
    1445                                 /* translators: %s: the upload error message */
    1446                                 __( 'Upload Failed! Error was: %s', 'buddypress' ),
    1447                                 $uploaded['error']
    1448                         ),
    1449                 ) );
     1482                bp_attachments_json_response(
     1483                        false,
     1484                        $is_html4,
     1485                        array(
     1486                                'type'    => 'upload_error',
     1487                                'message' => sprintf(
     1488                                        /* translators: %s: the upload error message */
     1489                                        __( 'Upload Failed! Error was: %s', 'buddypress' ),
     1490                                        $uploaded['error']
     1491                                ),
     1492                        )
     1493                );
    14501494        }
    14511495
    … …  
    14561500        // The BP Attachments Uploads Dir is not set, stop.
    14571501        if ( ! $bp_attachments_uploads_dir ) {
    1458                 bp_attachments_json_response( false, $is_html4, array(
    1459                         'type'    => 'upload_error',
    1460                         'message' => $error_message,
    1461                 ) );
     1502                bp_attachments_json_response(
     1503                        false,
     1504                        $is_html4,
     1505                        array(
     1506                                'type'    => 'upload_error',
     1507                                'message' => $error_message,
     1508                        )
     1509                );
    14621510        }
    14631511
    … …  
    14671515        if ( 1 === validate_file( $cover_dir ) || ! is_dir( $cover_dir ) ) {
    14681516                // Upload error response.
    1469                 bp_attachments_json_response( false, $is_html4, array(
    1470                         'type'    => 'upload_error',
    1471                         'message' => $error_message,
    1472                 ) );
     1517                bp_attachments_json_response(
     1518                        false,
     1519                        $is_html4,
     1520                        array(
     1521                                'type'    => 'upload_error',
     1522                                'message' => $error_message,
     1523                        )
     1524                );
    14731525        }
    14741526
    … …  
    14801532         * class, let's use it.
    14811533         */
    1482         $cover = bp_attachments_cover_image_generate_file( array(
    1483                 'file'            => $uploaded['file'],
    1484                 'component'       => $object_data['component'],
    1485                 'cover_image_dir' => $cover_dir
    1486         ), $cover_image_attachment );
     1534        $cover = bp_attachments_cover_image_generate_file(
     1535                array(
     1536                        'file'            => $uploaded['file'],
     1537                        'component'       => $object_data['component'],
     1538                        'cover_image_dir' => $cover_dir,
     1539                ),
     1540                $cover_image_attachment
     1541        );
    14871542
    14881543        if ( ! $cover ) {
    1489                 bp_attachments_json_response( false, $is_html4, array(
    1490                         'type'    => 'upload_error',
    1491                         'message' => $error_message,
    1492                 ) );
     1544                bp_attachments_json_response(
     1545                        false,
     1546                        $is_html4,
     1547                        array(
     1548                                'type'    => 'upload_error',
     1549                                'message' => $error_message,
     1550                        )
     1551                );
    14931552        }
    14941553
    … …  
    15531612
    15541613        // Finally return the cover image url to the UI.
    1555         bp_attachments_json_response( true, $is_html4, array(
    1556                 'name'          => $name,
    1557                 'url'           => $cover_url,
    1558                 'feedback_code' => $feedback_code,
    1559         ) );
     1614        bp_attachments_json_response(
     1615                true,
     1616                $is_html4,
     1617                array(
     1618                        'name'          => $name,
     1619                        'url'           => $cover_url,
     1620                        'feedback_code' => $feedback_code,
     1621                )
     1622        );
    15601623}
    15611624add_action( 'wp_ajax_bp_cover_image_upload', 'bp_attachments_cover_image_ajax_upload' );
    … …  
    15651628 *
    15661629 * @since 2.4.0
    1567  *
    1568  * @return string|null A json object containing success data if the cover image was deleted
    1569  *                     error message otherwise.
    15701630 */
    15711631function bp_attachments_cover_image_ajax_delete() {
     1632
    15721633        if ( ! bp_is_post_request() ) {
    15731634                wp_send_json_error();
    … …  
    15941655                $dir       = 'members';
    15951656
    1596         // Set it for any other cases.
     1657                // Set it for any other cases.
    15971658        } else {
    15981659                $component = $args['object'] . 's';
    … …  
    16011662
    16021663        // Handle delete.
    1603         if ( bp_attachments_delete_file( array( 'item_id' => $args['item_id'], 'object_dir' => $dir, 'type' => 'cover-image' ) ) ) {
     1664        if ( bp_attachments_delete_file(
     1665                array(
     1666                        'item_id'    => $args['item_id'],
     1667                        'object_dir' => $dir,
     1668                        'type'       => 'cover-image',
     1669                )
     1670        ) ) {
    16041671                $item_id = (int) $args['item_id'];
    16051672
    … …  
    16401707
    16411708        } else {
    1642                 wp_send_json_error( array(
    1643                         'feedback_code' => 2,
    1644                 ) );
     1709                wp_send_json_error(
     1710                        array(
     1711                                'feedback_code' => 2,
     1712                        )
     1713                );
    16451714        }
    16461715}
    … …  
    17291798                $supported_file = bp_attachments_get_file_object( $file );
    17301799
    1731                 if ( is_null( $supported_file) ) {
     1800                if ( is_null( $supported_file ) ) {
    17321801                        continue;
    17331802                }
    … …  
    17621831                $supported_file = bp_attachments_get_file_object( $file );
    17631832
    1764                 if ( is_null( $supported_file) ) {
     1833                if ( is_null( $supported_file ) ) {
    17651834                        continue;
    17661835                }

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

    r13890 r13891  
    5656        $bp = buddypress();
    5757
    58         $bp->avatar        = new stdClass;
    59         $bp->avatar->thumb = new stdClass;
    60         $bp->avatar->full  = new stdClass;
     58        $bp->avatar        = new stdClass();
     59        $bp->avatar->thumb = new stdClass();
     60        $bp->avatar->full  = new stdClass();
    6161
    6262        // Dimensions.
    … …  
    7676        // These have to be set on page load in order to avoid infinite filter loops at runtime.
    7777        $bp->avatar->upload_path = bp_core_avatar_upload_path();
    78         $bp->avatar->url = bp_core_avatar_url();
     78        $bp->avatar->url         = bp_core_avatar_url();
    7979
    8080        // Cache the root blog's show_avatars setting, to avoid unnecessary
    … …  
    8383
    8484        // Backpat for pre-1.5.
    85         if ( ! defined( 'BP_AVATAR_UPLOAD_PATH' ) )
     85        if ( ! defined( 'BP_AVATAR_UPLOAD_PATH' ) ) {
    8686                define( 'BP_AVATAR_UPLOAD_PATH', $bp->avatar->upload_path );
     87        }
    8788
    8889        // Backpat for pre-1.5.
    89         if ( ! defined( 'BP_AVATAR_URL' ) )
     90        if ( ! defined( 'BP_AVATAR_URL' ) ) {
    9091                define( 'BP_AVATAR_URL', $bp->avatar->url );
     92        }
    9193
    9294        /**
    … …  
    112114        }
    113115
    114         /** this filter is documented in wp-admin/options-discussion.php */
     116        /** This filter is documented in wp-admin/options-discussion.php */
    115117        $gravatar_defaults = apply_filters(
    116118                'avatar_defaults',
    … …  
    264266                switch ( $params['object'] ) {
    265267
    266                         case 'blog'  :
     268                        case 'blog':
    267269                                $params['item_id'] = get_current_blog_id();
    268270                                break;
    269271
    270                         case 'group' :
     272                        case 'group':
    271273                                if ( bp_is_active( 'groups' ) ) {
    272274                                        $params['item_id'] = $bp->groups->current_group->id;
    … …  
    277279                                break;
    278280
    279                         case 'user'  :
    280                         default      :
     281                        case 'user':
     282                        default:
    281283                                $params['item_id'] = bp_displayed_user_id();
    282284                                break;
    … …  
    305307                switch ( $params['object'] ) {
    306308
    307                         case 'blog'  :
     309                        case 'blog':
    308310                                $params['avatar_dir'] = 'blog-avatars';
    309311                                break;
    310312
    311                         case 'group' :
     313                        case 'group':
    312314                                if ( bp_is_active( 'groups' ) ) {
    313315                                        $params['avatar_dir'] = 'group-avatars';
    … …  
    318320                                break;
    319321
    320                         case 'user'  :
    321                         default      :
     322                        case 'user':
     323                        default:
    322324                                $params['avatar_dir'] = 'avatars';
    323325                                break;
    … …  
    346348                switch ( $params['object'] ) {
    347349
    348                         case 'blog'  :
     350                        case 'blog':
    349351                                $item_name = get_blog_option( $params['item_id'], 'blogname' );
    350352                                break;
    351353
    352                         case 'group' :
     354                        case 'group':
    353355                                $item_name = bp_get_group_name( groups_get_group( $params['item_id'] ) );
    354356                                break;
    355357
    356                         case 'user'  :
    357                         default :
     358                        case 'user':
     359                        default:
    358360                                $item_name = bp_core_get_user_displayname( $params['item_id'] );
    359361                                break;
    … …  
    370372                 * @param array  $params Array of parameters for the request.
    371373                 */
    372                 $item_name = apply_filters( 'bp_core_avatar_alt', $item_name, $params['item_id'], $params['object'], $params );
     374                $item_name     = apply_filters( 'bp_core_avatar_alt', $item_name, $params['item_id'], $params['object'], $params );
    373375                $params['alt'] = sprintf( $params['alt'], $item_name );
    374376        }
    … …  
    429431        if ( false !== $params['width'] ) {
    430432                // Width has been specified. No modification necessary.
    431         } elseif ( 'thumb' == $params['type'] ) {
     433        } elseif ( 'thumb' === $params['type'] ) {
    432434                $params['width'] = bp_core_avatar_thumb_width();
    433435        } else {
    … …  
    439441        if ( false !== $params['height'] ) {
    440442                // Height has been specified. No modification necessary.
    441         } elseif ( 'thumb' == $params['type'] ) {
     443        } elseif ( 'thumb' === $params['type'] ) {
    442444                $params['height'] = bp_core_avatar_thumb_height();
    443445        } else {
    … …  
    467469
    468470        // Merge classes.
    469         $avatar_classes = array_merge( $avatar_classes, array(
    470                 $params['object'] . '-' . $params['item_id'] . '-avatar',
    471                 'avatar-' . $params['width'],
    472         ) );
     471        $avatar_classes = array_merge(
     472                $avatar_classes,
     473                array(
     474                        $params['object'] . '-' . $params['item_id'] . '-avatar',
     475                        'avatar-' . $params['width'],
     476                )
     477        );
    473478
    474479        // Sanitize each class.
    … …  
    479484
    480485        // Set img URL and DIR based on prepopulated constants.
    481         $avatar_loc        = new stdClass();
    482         $avatar_loc->path  = trailingslashit( bp_core_avatar_upload_path() );
    483         $avatar_loc->url   = trailingslashit( bp_core_avatar_url() );
    484 
    485         $avatar_loc->dir   = trailingslashit( $params['avatar_dir'] );
     486        $avatar_loc       = new stdClass();
     487        $avatar_loc->path = trailingslashit( bp_core_avatar_upload_path() );
     488        $avatar_loc->url  = trailingslashit( bp_core_avatar_url() );
     489
     490        $avatar_loc->dir = trailingslashit( $params['avatar_dir'] );
    486491
    487492        /**
    … …  
    495500         * @param string $value Subdirectory where the requested avatar should be found.
    496501         */
    497         $avatar_folder_url = apply_filters( 'bp_core_avatar_folder_url', ( $avatar_loc->url  . $avatar_loc->dir . $params['item_id'] ), $params['item_id'], $params['object'], $params['avatar_dir'] );
     502        $avatar_folder_url = apply_filters( 'bp_core_avatar_folder_url', ( $avatar_loc->url . $avatar_loc->dir . $params['item_id'] ), $params['item_id'], $params['object'], $params['avatar_dir'] );
    498503
    499504        /**
    … …  
    514519         * or thumbnail image.
    515520         */
    516         $avatar_size              = ( 'full' == $params['type'] ) ? '-bpfull' : '-bpthumb';
    517         $legacy_user_avatar_name  = ( 'full' == $params['type'] ) ? '-avatar2' : '-avatar1';
    518         $legacy_group_avatar_name = ( 'full' == $params['type'] ) ? '-groupavatar-full' : '-groupavatar-thumb';
     521        $avatar_size              = ( 'full' === $params['type'] ) ? '-bpfull' : '-bpthumb';
     522        $legacy_user_avatar_name  = ( 'full' === $params['type'] ) ? '-avatar2' : '-avatar1';
     523        $legacy_group_avatar_name = ( 'full' === $params['type'] ) ? '-groupavatar-full' : '-groupavatar-thumb';
    519524
    520525        // Check for directory.
    … …  
    538543                                // Check for current avatar.
    539544                                foreach ( $avatar_files as $key => $value ) {
    540                                         if ( strpos ( $value, $avatar_size )!== false ) {
    541                                                 $avatar_url = $avatar_folder_url . '/' . $avatar_files[$key];
     545                                        if ( strpos( $value, $avatar_size ) !== false ) {
     546                                                $avatar_url = $avatar_folder_url . '/' . $avatar_files[ $key ];
    542547                                        }
    543548                                }
    544549
    545550                                // Legacy avatar check.
    546                                 if ( !isset( $avatar_url ) ) {
     551                                if ( ! isset( $avatar_url ) ) {
    547552                                        foreach ( $avatar_files as $key => $value ) {
    548                                                 if ( strpos ( $value, $legacy_user_avatar_name )!== false ) {
    549                                                         $avatar_url = $avatar_folder_url . '/' . $avatar_files[$key];
     553                                                if ( strpos( $value, $legacy_user_avatar_name ) !== false ) {
     554                                                        $avatar_url = $avatar_folder_url . '/' . $avatar_files[ $key ];
    550555                                                }
    551556                                        }
    552557
    553558                                        // Legacy group avatar check.
    554                                         if ( !isset( $avatar_url ) ) {
     559                                        if ( ! isset( $avatar_url ) ) {
    555560                                                foreach ( $avatar_files as $key => $value ) {
    556                                                         if ( strpos ( $value, $legacy_group_avatar_name )!== false ) {
    557                                                                 $avatar_url = $avatar_folder_url . '/' . $avatar_files[$key];
     561                                                        if ( strpos( $value, $legacy_group_avatar_name ) !== false ) {
     562                                                                $avatar_url = $avatar_folder_url . '/' . $avatar_files[ $key ];
    558563                                                        }
    559564                                                }
    … …  
    589594                                 * @param string $avatar_folder_dir Avatar DIR path.
    590595                                 */
    591                                 return apply_filters( 'bp_core_fetch_avatar', '<img src="' . $avatar_url . '"' . $html_class . $html_css_id  . $html_width . $html_height . $html_alt . $html_title . $extra_attr . ' />', $params, $params['item_id'], $params['avatar_dir'], $html_css_id, $html_width, $html_height, $avatar_folder_url, $avatar_folder_dir );
    592 
    593                         // ...or only the URL
     596                                return apply_filters( 'bp_core_fetch_avatar', '<img src="' . $avatar_url . '"' . $html_class . $html_css_id . $html_width . $html_height . $html_alt . $html_title . $extra_attr . ' />', $params, $params['item_id'], $params['avatar_dir'], $html_css_id, $html_width, $html_height, $avatar_folder_url, $avatar_folder_dir );
     597
     598                                // ...or only the URL
    594599                        } else {
    595600
    … …  
    625630                if ( empty( $bp->grav_default->{$params['object']} ) ) {
    626631                        $default_grav = 'wavatar';
    627                 } elseif ( 'mystery' == $bp->grav_default->{$params['object']} ) {
     632                } elseif ( 'mystery' === $bp->grav_default->{$params['object']} ) {
    628633
    629634                        /**
    … …  
    642647                // Set gravatar object.
    643648                if ( empty( $params['email'] ) ) {
    644                         if ( 'user' == $params['object'] ) {
     649                        if ( 'user' === $params['object'] ) {
    645650                                $params['email'] = bp_core_get_user_email( $params['item_id'] );
    646                         } elseif ( 'group' == $params['object'] || 'blog' == $params['object'] ) {
     651                        } elseif ( 'group' === $params['object'] || 'blog' === $params['object'] ) {
    647652                                $params['email'] = $params['item_id'] . '-' . $params['object'] . '@' . bp_get_domain();
    648653                        }
    … …  
    670675
    671676                // Append email hash to Gravatar.
    672                 $gravatar .=  md5( strtolower( $params['email'] ) );
     677                $gravatar .= md5( strtolower( $params['email'] ) );
    673678
    674679                // Main Gravatar URL args.
    675680                $url_args = array(
    676                         's' => $params['width']
     681                        's' => $params['width'],
    677682                );
    678683
    … …  
    716721
    717722                // Set up the Gravatar URL.
    718                 $gravatar = esc_url( add_query_arg(
    719                         rawurlencode_deep( array_filter( $url_args ) ),
    720                         $gravatar
    721                 ) );
    722 
    723         // No avatar was found, and we've been told not to use a gravatar.
     723                $gravatar = esc_url(
     724                        add_query_arg(
     725                                rawurlencode_deep( array_filter( $url_args ) ),
     726                                $gravatar
     727                        )
     728                );
     729
     730                // No avatar was found, and we've been told not to use a gravatar.
    724731        } else {
    725732
    … …  
    771778                'item_id'    => false,
    772779                'object'     => 'user', // User OR group OR blog OR custom type (if you use filters).
    773                 'avatar_dir' => false
     780                'avatar_dir' => false,
    774781        );
    775782
    … …  
    851858        if ( $av_dir = opendir( $avatar_folder_dir ) ) {
    852859                while ( false !== ( $avatar_file = readdir( $av_dir ) ) ) {
    853                         if ( ( preg_match( "/-bpfull/", $avatar_file ) || preg_match( "/-bpthumb/", $avatar_file ) ) && '.' != $avatar_file && '..' != $avatar_file ) {
     860                        if ( ( preg_match( '/-bpfull/', $avatar_file ) || preg_match( '/-bpthumb/', $avatar_file ) ) && '.' !== $avatar_file && '..' !== $avatar_file ) {
    854861                                @unlink( $avatar_folder_dir . '/' . $avatar_file );
    855862                        }
    … …  
    905912
    906913        // Handle delete.
    907         if ( bp_core_delete_existing_avatar( array( 'item_id' => $avatar_data['item_id'], 'object' => $avatar_data['object'] ) ) ) {
     914        if ( bp_core_delete_existing_avatar(
     915                array(
     916                        'item_id' => $avatar_data['item_id'],
     917                        'object'  => $avatar_data['object'],
     918                )
     919        ) ) {
    908920                $return = array(
    909                         'avatar' => esc_url( bp_core_fetch_avatar( array(
    910                                 'object'  => $avatar_data['object'],
    911                                 'item_id' => $avatar_data['item_id'],
    912                                 'html'    => false,
    913                                 'type'    => 'full',
    914                         ) ) ),
     921                        'avatar'        => esc_url(
     922                                bp_core_fetch_avatar(
     923                                        array(
     924                                                'object'  => $avatar_data['object'],
     925                                                'item_id' => $avatar_data['item_id'],
     926                                                'html'    => false,
     927                                                'type'    => 'full',
     928                                        )
     929                                )
     930                        ),
    915931                        'feedback_code' => 4,
    916932                        'item_id'       => $avatar_data['item_id'],
    … …  
    919935                wp_send_json_success( $return );
    920936        } else {
    921                 wp_send_json_error( array(
    922                         'feedback_code' => 3,
    923                 ) );
     937                wp_send_json_error(
     938                        array(
     939                                'feedback_code' => 3,
     940                        )
     941                );
    924942        }
    925943}
    … …  
    966984
    967985        // Upload the file.
    968         $avatar_attachment = new BP_Attachment_Avatar();
     986        $avatar_attachment          = new BP_Attachment_Avatar();
    969987        $bp->avatar_admin->original = $avatar_attachment->upload( $file, $upload_dir_filter );
    970988
    … …  
    10301048 *
    10311049 * @since 2.3.0
    1032  *
    1033  * @return string|null A JSON object containing success data if the upload succeeded
    1034  *                     error message otherwise.
    10351050 */
    10361051function bp_avatar_ajax_upload() {
     1052
    10371053        if ( ! bp_is_post_request() ) {
    10381054                wp_die();
    … …  
    10441060         */
    10451061        $is_html4 = false;
    1046         if ( ! empty( $_POST['html4' ] ) ) {
     1062        if ( ! empty( $_POST['html4'] ) ) {
    10471063                $is_html4 = true;
    10481064        }
    … …  
    10551071
    10561072        // We need it to carry on.
    1057         if ( ! empty( $_POST['bp_params' ] ) ) {
    1058                 $bp_params = $_POST['bp_params' ];
     1073        if ( ! empty( $_POST['bp_params'] ) ) {
     1074                $bp_params = $_POST['bp_params'];
    10591075        } else {
    10601076                bp_attachments_json_response( false, $is_html4 );
    … …  
    10711087        }
    10721088
    1073         $bp = buddypress();
     1089        $bp                             = buddypress();
    10741090        $bp_params['upload_dir_filter'] = '';
    1075         $needs_reset = array();
     1091        $needs_reset                    = array();
    10761092
    10771093        if ( 'user' === $bp_params['object'] && bp_is_active( 'members' ) ) {
    … …  
    10791095
    10801096                if ( ! bp_displayed_user_id() && ! empty( $bp_params['item_id'] ) ) {
    1081                         $needs_reset = array( 'key' => 'displayed_user', 'value' => $bp->displayed_user );
     1097                        $needs_reset            = array(
     1098                                'key'   => 'displayed_user',
     1099                                'value' => $bp->displayed_user,
     1100                        );
    10821101                        $bp->displayed_user->id = $bp_params['item_id'];
    10831102                }
    … …  
    10861105
    10871106                if ( ! bp_get_current_group_id() && ! empty( $bp_params['item_id'] ) ) {
    1088                         $needs_reset = array( 'component' => 'groups', 'key' => 'current_group', 'value' => $bp->groups->current_group );
     1107                        $needs_reset               = array(
     1108                                'component' => 'groups',
     1109                                'key'       => 'current_group',
     1110                                'value'     => $bp->groups->current_group,
     1111                        );
    10891112                        $bp->groups->current_group = groups_get_group( $bp_params['item_id'] );
    10901113                }
    … …  
    11091132         */
    11101133        if ( isset( $bp_params['ui_available_width'] ) ) {
    1111                 $bp->avatar_admin->ui_available_width =  (int) $bp_params['ui_available_width'];
     1134                $bp->avatar_admin->ui_available_width = (int) $bp_params['ui_available_width'];
    11121135        }
    11131136
    … …  
    11481171
    11491172                // Upload error reply.
    1150                 bp_attachments_json_response( false, $is_html4, array(
    1151                         'type'    => 'upload_error',
    1152                         'message' => $message,
    1153                 ) );
     1173                bp_attachments_json_response(
     1174                        false,
     1175                        $is_html4,
     1176                        array(
     1177                                'type'    => 'upload_error',
     1178                                'message' => $message,
     1179                        )
     1180                );
    11541181        }
    11551182
    … …  
    11611188
    11621189        // Set the name of the file.
    1163         $name = $_FILES['file']['name'];
     1190        $name       = $_FILES['file']['name'];
    11641191        $name_parts = pathinfo( $name );
    1165         $name = trim( substr( $name, 0, - ( 1 + strlen( $name_parts['extension'] ) ) ) );
     1192        $name       = trim( substr( $name, 0, - ( 1 + strlen( $name_parts['extension'] ) ) ) );
    11661193
    11671194        // Finally return the avatar to the editor.
    1168         bp_attachments_json_response( true, $is_html4, array(
    1169                 'name'      => $name,
    1170                 'url'       => $bp->avatar_admin->image->url,
    1171                 'width'     => $uploaded_image[0],
    1172                 'height'    => $uploaded_image[1],
    1173                 'feedback'  => $feedback_message,
    1174         ) );
     1195        bp_attachments_json_response(
     1196                true,
     1197                $is_html4,
     1198                array(
     1199                        'name'     => $name,
     1200                        'url'      => $bp->avatar_admin->image->url,
     1201                        'width'    => $uploaded_image[0],
     1202                        'height'   => $uploaded_image[1],
     1203                        'feedback' => $feedback_message,
     1204                )
     1205        );
    11751206}
    11761207add_action( 'wp_ajax_bp_avatar_upload', 'bp_avatar_ajax_upload' );
    … …  
    11821213 * @since 10.0.0 Adds the `$return` param to eventually return the crop result.
    11831214 *
    1184  * @param string $data    Base64 encoded image.
    1185  * @param int    $item_id Item to associate.
    1186  * @param string $return  Whether to get the crop `array` or a `boolean`. Defaults to `boolean`.
    1187  * @return array|bool True on success, false on failure.
    1188  */
    1189 function bp_avatar_handle_capture( $data = '', $item_id = 0, $return = 'boolean' ) {
     1215 * @param string $data    Optional. Base64 encoded image.
     1216 * @param int    $item_id Optional. Item to associate.
     1217 * @param string $retval  Optional. Whether to get the crop `array` or a `boolean`. Defaults to `boolean`.
     1218 * @return array|bool
     1219 */
     1220function bp_avatar_handle_capture( $data = '', $item_id = 0, $retval = 'boolean' ) {
     1221        $return = $retval;
     1222
    11901223        if ( empty( $data ) || empty( $item_id ) ) {
    11911224                return false;
    … …  
    12291262
    12301263        // It's not a regular upload, we may need to create this folder.
    1231         if( ! is_dir( $avatar_folder_dir ) ) {
     1264        if ( ! is_dir( $avatar_folder_dir ) ) {
    12321265                if ( ! wp_mkdir_p( $avatar_folder_dir ) ) {
    12331266                        return false;
    … …  
    12411274
    12421275                // Crop to default values.
    1243                 $crop_args = array( 'item_id' => $item_id, 'original_file' => $avatar_to_crop, 'crop_x' => 0, 'crop_y' => 0 );
     1276                $crop_args = array(
     1277                        'item_id'       => $item_id,
     1278                        'original_file' => $avatar_to_crop,
     1279                        'crop_x'        => 0,
     1280                        'crop_y'        => 0,
     1281                );
    12441282
    12451283                if ( 'array' === $return ) {
    … …  
    12481286
    12491287                return bp_core_avatar_handle_crop( $crop_args );
    1250         } else {
    1251                 return false;
    1252         }
     1288        }
     1289
     1290        return false;
    12531291}
    12541292
    … …  
    12761314 *     @type int         $crop_y        The vertical starting point of the crop. Default: 0.
    12771315 * }
    1278  * @param string       $return Whether to get the crop `array` or a `boolean`. Defaults to `boolean`.
    1279  * @return array|bool True or the crop result on success, false on failure.
    1280  */
    1281 function bp_core_avatar_handle_crop( $args = '', $return = 'boolean' ) {
     1316 * @param string       $retval Optional. Whether to get the crop `array` or a `boolean`. Defaults to `boolean`.
     1317 * @return array|bool
     1318 */
     1319function bp_core_avatar_handle_crop( $args = '', $retval = 'boolean' ) {
    12821320
    12831321        $r = bp_parse_args(
    … …  
    13181356        }
    13191357
    1320         if ( 'array' === $return ) {
     1358        if ( 'array' === $retval ) {
    13211359                return $cropped;
    13221360        }
    … …  
    13291367 *
    13301368 * @since 2.3.0
    1331  *
    1332  * @return string|null A JSON object containing success data if the crop/capture succeeded
    1333  *                     error message otherwise.
    13341369 */
    13351370function bp_avatar_ajax_set() {
     1371
    13361372        if ( ! bp_is_post_request() ) {
    13371373                wp_send_json_error();
    … …  
    13711407
    13721408                if ( ! $cropped_webcam_avatar ) {
    1373                         wp_send_json_error( array(
    1374                                 'feedback_code' => 1
    1375                         ) );
     1409                        wp_send_json_error(
     1410                                array(
     1411                                        'feedback_code' => 1,
     1412                                )
     1413                        );
    13761414
    13771415                } else {
    13781416                        $return = array(
    1379                                 'avatar' => esc_url(
     1417                                'avatar'        => esc_url(
    13801418                                        bp_core_fetch_avatar(
    13811419                                                array(
    … …  
    14191457                $avatar_dir = 'avatars';
    14201458
    1421         // Defaults to object-avatars dir.
     1459                // Defaults to object-avatars dir.
    14221460        } else {
    14231461                $avatar_dir = sanitize_key( $avatar_data['object'] ) . '-avatars';
    … …  
    14331471                'crop_h'        => $avatar_data['crop_h'],
    14341472                'crop_x'        => $avatar_data['crop_x'],
    1435                 'crop_y'        => $avatar_data['crop_y']
     1473                'crop_y'        => $avatar_data['crop_y'],
    14361474        );
    14371475
    … …  
    14411479        if ( $cropped_avatar ) {
    14421480                $return = array(
    1443                         'avatar' => esc_url(
     1481                        'avatar'        => esc_url(
    14441482                                bp_core_fetch_avatar(
    14451483                                        array(
    … …  
    14681506                wp_send_json_success( $return );
    14691507        } else {
    1470                 wp_send_json_error( array(
    1471                         'feedback_code' => 1,
    1472                 ) );
     1508                wp_send_json_error(
     1509                        array(
     1510                                'feedback_code' => 1,
     1511                        )
     1512                );
    14731513        }
    14741514}
    … …  
    14931533                $user = get_user_by( 'id', absint( $id_or_email ) );
    14941534        } elseif ( $id_or_email instanceof WP_User ) {
    1495                 // User Object
     1535                // User Object.
    14961536                $user = $id_or_email;
    14971537        } elseif ( $id_or_email instanceof WP_Post ) {
    1498                 // Post Object
     1538                // Post Object.
    14991539                $user = get_user_by( 'id', (int) $id_or_email->post_author );
    15001540        } elseif ( $id_or_email instanceof WP_Comment ) {
    … …  
    15211561
    15221562        // Get the BuddyPress avatar URL.
    1523         if ( $bp_avatar = bp_core_fetch_avatar( $args ) ) {
     1563        $bp_avatar = bp_core_fetch_avatar( $args );
     1564        if ( $bp_avatar ) {
    15241565                return $bp_avatar;
    15251566        }
    … …  
    15351576 *
    15361577 * @param array $file The $_FILES array.
    1537  * @return bool True if no errors are found. False if there are errors.
     1578 * @return bool
    15381579 */
    15391580function bp_core_check_avatar_upload( $file ) {
    1540         if ( isset( $file['error'] ) && $file['error'] )
     1581        if ( isset( $file['error'] ) && $file['error'] ) {
    15411582                return false;
     1583        }
    15421584
    15431585        return true;
    … …  
    15531595 */
    15541596function bp_core_check_avatar_size( $file ) {
    1555         if ( $file['file']['size'] > bp_core_avatar_original_max_filesize() )
     1597        if ( $file['file']['size'] > bp_core_avatar_original_max_filesize() ) {
    15561598                return false;
     1599        }
    15571600
    15581601        return true;
    … …  
    15951638 */
    15961639function bp_core_get_allowed_avatar_mimes() {
    1597         $allowed_types  = bp_core_get_allowed_avatar_types();
     1640        $allowed_types = bp_core_get_allowed_avatar_types();
    15981641
    15991642        return bp_attachments_get_allowed_mimes( 'avatar', $allowed_types );
    … …  
    16281671
    16291672        switch ( $type ) {
    1630                 case 'upload_path' :
     1673                case 'upload_path':
    16311674                        $constant = 'BP_AVATAR_UPLOAD_PATH';
    16321675                        $key      = 'basedir';
    … …  
    16341677                        break;
    16351678
    1636                 case 'url' :
     1679                case 'url':
    16371680                        $constant = 'BP_AVATAR_URL';
    16381681                        $key      = 'baseurl';
    … …  
    16401683                        break;
    16411684
    1642                 default :
     1685                default:
    16431686                        return $retval;
    1644 
    1645                         break;
    16461687        }
    16471688
    … …  
    16591700                                $upload_dir = $bp->avatar->upload_dir;
    16601701
    1661                         // No cache, so query for it.
     1702                                // No cache, so query for it.
    16621703                        } else {
    16631704
    … …  
    17021743         * @since 1.2.0
    17031744         *
    1704          * @param string $value Absolute upload path for the WP installation.
     1745         * @param string $upload_path Absolute upload path for the WP installation.
    17051746         */
    17061747        return apply_filters( 'bp_core_avatar_upload_path', bp_core_get_upload_dir() );
    … …  
    17211762         * @since 1.2.0
    17221763         *
    1723          * @param string $value Raw base URL for the root site upload location.
     1764         * @param string $avatar_url Raw base URL for the root site upload location.
    17241765         */
    17251766        return apply_filters( 'bp_core_avatar_url', bp_core_get_upload_dir( 'url' ) );
    … …  
    17361777function bp_get_user_has_avatar( $user_id = 0 ) {
    17371778
    1738         if ( empty( $user_id ) )
     1779        if ( empty( $user_id ) ) {
    17391780                $user_id = bp_displayed_user_id();
     1781        }
    17401782
    17411783        $retval = false;
    1742         if ( bp_core_fetch_avatar( array( 'item_id' => $user_id, 'no_grav' => true, 'html' => false, 'type' => 'full' ) ) != bp_core_avatar_default( 'local' ) )
     1784        if ( bp_core_fetch_avatar(
     1785                array(
     1786                        'item_id' => $user_id,
     1787                        'no_grav' => true,
     1788                        'html'    => false,
     1789                        'type'    => 'full',
     1790                )
     1791        ) !== bp_core_avatar_default( 'local' ) ) {
    17431792                $retval = true;
     1793        }
    17441794
    17451795        /**
    … …  
    17951845         * @since 1.5.0
    17961846         *
    1797          * @param int $value Value for the 'thumb' avatar width setting.
     1847         * @param int $avatar_thumb_width Value for the 'thumb' avatar width setting.
    17981848         */
    17991849        return apply_filters( 'bp_core_avatar_thumb_width', bp_core_avatar_dimension( 'thumb', 'width' ) );
    … …  
    18141864         * @since 1.5.0
    18151865         *
    1816          * @param int $value Value for the 'thumb' avatar height setting.
     1866         * @param int $avatar_thumb_height Value for the 'thumb' avatar height setting.
    18171867         */
    18181868        return apply_filters( 'bp_core_avatar_thumb_height', bp_core_avatar_dimension( 'thumb', 'height' ) );
    … …  
    18331883         * @since 1.5.0
    18341884         *
    1835          * @param int $value Value for the 'full' avatar width setting.
     1885         * @param int $avatar_full_width Value for the 'full' avatar width setting.
    18361886         */
    18371887        return apply_filters( 'bp_core_avatar_full_width', bp_core_avatar_dimension( 'full', 'width' ) );
    … …  
    18521902         * @since 1.5.0
    18531903         *
    1854          * @param int $value Value for the 'full' avatar height setting.
     1904         * @param int $avatar_full_height Value for the 'full' avatar height setting.
    18551905         */
    18561906        return apply_filters( 'bp_core_avatar_full_height', bp_core_avatar_dimension( 'full', 'height' ) );
    … …  
    18711921         * @since 1.5.0
    18721922         *
    1873          * @param int $value Value for the max width.
     1923         * @param int $original_max_width Value for the max width.
    18741924         */
    18751925        return apply_filters( 'bp_core_avatar_original_max_width', (int) buddypress()->avatar->original_max_width );
    … …  
    18901940         * @since 1.5.0
    18911941         *
    1892          * @param int $value Value for the max filesize.
     1942         * @param int $original_max_filesize Value for the max filesize.
    18931943         */
    18941944        return apply_filters( 'bp_core_avatar_original_max_filesize', (int) buddypress()->avatar->original_max_filesize );
    … …  
    19121962                $avatar = BP_AVATAR_DEFAULT;
    19131963
    1914         // Use the local default image.
     1964                // Use the local default image.
    19151965        } elseif ( 'local' === $type ) {
    19161966                $size = '';
    … …  
    19191969                        ( isset( $params['width'] ) && $params['width'] <= 50 )
    19201970                ) {
    1921 
    19221971                        $size = '-50';
    19231972                }
    … …  
    19251974                $avatar = buddypress()->plugin_url . "bp-core/images/mystery-man{$size}.jpg";
    19261975
    1927         // Use Gravatar's mystery person as fallback.
     1976                // Use Gravatar's mystery person as fallback.
    19281977        } else {
    19291978                $size = '';
    … …  
    19822031                $avatar = BP_AVATAR_DEFAULT_THUMB;
    19832032
    1984         // Use the local default image.
     2033                // Use the local default image.
    19852034        } elseif ( 'local' === $type ) {
    19862035                $avatar = buddypress()->plugin_url . 'bp-core/images/mystery-man-50.jpg';
    19872036
    1988         // Use Gravatar's mystery person as fallback.
     2037                // Use Gravatar's mystery person as fallback.
    19892038        } else {
    19902039                $avatar = '//www.gravatar.com/avatar/00000000000000000000000000000000?d=mm&amp;s=' . bp_core_avatar_thumb_width();
    … …  
    20212070                $reset_w = bp_is_group_admin_screen( 'group-avatar' );
    20222071
    2023         // Group's avatar create screen.
     2072                // Group's avatar create screen.
    20242073        } elseif ( bp_is_group_create() ) {
    20252074                /**
    … …  
    20292078                $reset_w = 'group-avatar' === bp_action_variable( 1 );
    20302079
    2031         // User's change avatar screen.
     2080                // User's change avatar screen.
    20322081        } else {
    20332082                $reset_w = bp_is_user_change_avatar();
    … …  
    20602109                        $retval = ! bp_disable_group_avatar_uploads();
    20612110
    2062                 // Group Manage.
     2111                        // Group Manage.
    20632112                } elseif ( bp_is_group_admin_page() && bp_is_group_admin_screen( 'group-avatar' ) && 'crop-image' !== bp_get_avatar_admin_step() ) {
    20642113                        $retval = ! bp_disable_group_avatar_uploads();
    … …  
    21642213         * @since 10.0.0
    21652214         *
    2166          * @param bool $value True to disable avatar history. False otherwise.
     2215         * @param bool $avatar_history True to disable avatar history. False otherwise.
    21672216         *                    Default: `false`.
    21682217         */
    … …  
    22352284
    22362285        $avatars     = array();
    2237         $history_url = trailingslashit( bp_core_avatar_url() ) .  $avatar_dir . '/' . $item_id . '/history';
     2286        $history_url = trailingslashit( bp_core_avatar_url() ) . $avatar_dir . '/' . $item_id . '/history';
    22382287
    22392288        foreach ( $historic_avatars as $historic_avatar ) {
    … …  
    23182367                        }
    23192368
    2320                         $is_full  = preg_match( "/-bpfull/", $current_avatar->name );
    2321                         $is_thumb = preg_match( "/-bpthumb/", $current_avatar->name );
     2369                        $is_full  = preg_match( '/-bpfull/', $current_avatar->name );
     2370                        $is_thumb = preg_match( '/-bpthumb/', $current_avatar->name );
    23222371
    23232372                        if ( $is_full || $is_thumb ) {
    … …  
    23952444                } else {
    23962445                        foreach ( $avatar_types as $type_key => $avatar_path ) {
    2397                                 $filename  = wp_basename( $avatar_path );
    2398                                 $avatar_id = pathinfo( $filename, PATHINFO_FILENAME );
     2446                                $filename     = wp_basename( $avatar_path );
     2447                                $avatar_id    = pathinfo( $filename, PATHINFO_FILENAME );
    23992448                                $recycle_path = $avatar_dir_path . '/' . str_replace( $avatar_id, $recycle_timestamp . '-bp' . $type_key, $filename );
    24002449
    … …  
    24702519                        $gmdate      = gmdate( 'Y-m-d H:i:s', $avatars_history[ $latest_id ]->last_modified );
    24712520                        $date        = strtotime( get_date_from_gmt( $gmdate ) );
    2472                         $history_url = trailingslashit( bp_core_avatar_url() ) .  $avatar_dir . '/' . $item_id . '/history';
     2521                        $history_url = trailingslashit( bp_core_avatar_url() ) . $avatar_dir . '/' . $item_id . '/history';
    24732522
    24742523                        // Prepare the avatar object for JavaScript.
Note: See TracChangeset for help on using the changeset viewer.