Changeset 10641

Timestamp:

03/09/2016 11:40:10 AM (9 years ago)

Author:

djpaul

Message:

Add filters and actions to support short-circuiting of standard avatar/cover image uploads/crops.

Fixes #6948

Props DJPaul, imath

Location:

trunk/src/bp-core

Files:

• bp-core-attachments.php (modified) (3 diffs)
• bp-core-avatars.php (modified) (2 diffs)

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

@@ -411 +411 @@
     ), 'attachments_get_attachment_src' );
 
+    /**
+     * Filters whether or not to handle fetching a BuddyPress image attachment.
+     *
+     * If you want to override this function, make sure you return false.
+     *
+     * @since 2.5.1
+     *
+     * @param null|string $value If null is returned, proceed with default behaviour. Otherwise, value returned verbatim.
+     * @param array $r {
+     *     @type string $object_dir The object dir (eg: members/groups). Defaults to members.
+     *     @type int    $item_id    The object id (eg: a user or a group id). Defaults to current user.
+     *     @type string $type       The type of the attachment which is also the subdir where files are saved.
+     *                              Defaults to 'cover-image'
+     *     @type string $file       The name of the file.
+     * }
+     */
+    $pre_filter = apply_filters( 'bp_attachments_pre_get_attachment', null, $r );
+    if ( $pre_filter !== null ) {
+        return $pre_filter;
+    }
+
     // Get BuddyPress Attachments Uploads Dir datas.
     $bp_attachments_uploads_dir = bp_attachments_uploads_dir_get();
@@ -478 +499 @@
 function bp_attachments_delete_file( $args = array() ) {
     $attachment_path = bp_attachments_get_attachment( 'path', $args );
+
+    /**
+     * Filters whether or not to handle deleting an existing BuddyPress attachment.
+     *
+     * If you want to override this function, make sure you return false.
+     *
+     * @since 2.5.1
+     *
+     * @param bool $value Whether or not to delete the BuddyPress attachment.
+     * @param array Array of arguments for the attachment deletion.
+     */
+    if ( ! apply_filters( 'bp_attachments_pre_delete_file', true, $args ) ) {
+        return true;
+    }
 
     if ( empty( $attachment_path ) ) {
@@ -1224 +1259 @@
     }
 
+    /**
+     * Filters whether or not to handle cover image uploading.
+     *
+     * If you want to override this function, make sure you return an array with the 'result' key set.
+     *
+     * @since 2.5.1
+     *
+     * @param array $value
+     * @param array $bp_params
+     * @param array $needs_reset Stores original value of certain globals we need to revert to later.
+     * @param array $object_data
+     */
+    $pre_filter = apply_filters( 'bp_attachments_pre_cover_image_ajax_upload', array(), $bp_params, $needs_reset, $object_data );
+    if ( isset( $pre_filter['result'] ) ) {
+        bp_attachments_json_response( $pre_filter['result'], $is_html4, $pre_filter );
+    }
+
     $cover_image_attachment = new BP_Attachment_Cover_Image();
     $uploaded = $cover_image_attachment->upload( $_FILES );

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

@@ -704 +704 @@
     extract( $args, EXTR_SKIP );
 
+    /**
+     * Filters whether or not to handle deleting an existing avatar.
+     *
+     * If you want to override this function, make sure you return false.
+     *
+     * @since 2.5.1
+     *
+     * @param bool $value Whether or not to delete the avatar.
+     * @param array $args {
+     *     Array of function parameters.
+     *
+     *     @type bool|int    $item_id    ID of the item whose avatar you're deleting.
+     *                                   Defaults to the current item of type $object.
+     *     @type string      $object     Object type of the item whose avatar you're
+     *                                   deleting. 'user', 'group', 'blog', or custom.
+     *                                   Default: 'user'.
+     *     @type bool|string $avatar_dir Subdirectory where avatar is located.
+     *                                   Default: false, which falls back on the default location
+     *                                   corresponding to the $object.
+     * }
+     */
+    if ( ! apply_filters( 'bp_core_pre_delete_existing_avatar', true, $args ) ) {
+        return true;
+    }
+
     if ( empty( $item_id ) ) {
         if ( 'user' == $object )
@@ -1068 +1093 @@
     if ( empty( $data ) || empty( $item_id ) ) {
         return false;
+    }
+
+    /**
+     * Filters whether or not to handle avatar webcam capture.
+     *
+     * If you want to override this function, make sure you return false.
+     *
+     * @since 2.5.1
+     *
+     * @param bool   $value   Whether or not to crop.
+     * @param string $data    Base64 encoded image.
+     * @param int    $item_id Item to associate.
+     */
+    if ( ! apply_filters( 'bp_avatar_pre_handle_capture', true, $data, $item_id ) ) {
+        return true;
     }