[buddypress-trac] [BuddyPress Trac] #7265: uploads dir should filtered for cover images

[buddypress-trac]

#7265: uploads dir should filtered for cover images
--------------------------+-----------------------------
 Reporter:  m_uysl        |      Owner:
     Type:  defect (bug)  |     Status:  new
 Priority:  normal        |  Milestone:  Awaiting Review
Component:  Core          |    Version:
 Severity:  normal        |   Keywords:
--------------------------+-----------------------------
 I used `bp_attachments_cover_image_upload_dir` for changing default cover-
 image path and it works, I can see the file into correct folder but AJAX
 callback gives error.
 [[Image(https://cldup.com/X1ltwwrf_w.png)]]

 {{{#!php
 <?php
 // hard-coded filter
 add_filter( 'bp_attachments_cover_image_upload_dir', function (
 $upload_dir ) {
         return array(
                 'path'    => '/Applications/MAMP/htdocs/wordpress/wp-
 content/uploads/cover-image/2016/09/21/mustafa',
                 'url'     => 'http://wordpress.dev/wp-content/uploads
 /cover-image/2016/09/21/mustafa',
                 'subdir'  => '/2016/09/21/mustafa',
                 'basedir' => '/Applications/MAMP/htdocs/wordpress/wp-
 content/uploads/cover-image',
                 'baseurl' => 'http://wordpress.dev/wp-content/uploads
 /cover-image',
                 'error'   => false,
         );
 } );
 }}}


 I realized that callback fails from
 `bp_attachments_cover_image_generate_file` because
 `bp_attachments_cover_image_ajax_upload` gets uploads dir from
 `bp_attachments_uploads_dir_get()`


 So in this case, overriding the default AJAX callback with
 `bp_attachments_pre_cover_image_ajax_upload` filter is current solution.

--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/7265>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac