[buddypress-trac] [BuddyPress Trac] #8356: swap the order to the two filters bp_core_avatar_folder_url and bp_core_avatar_folder_dir in bp_core_fetch_avatar

buddypress-trac noreply at wordpress.org
Sat Sep 5 08:59:49 UTC 2020


#8356: swap the order to the two filters bp_core_avatar_folder_url and
bp_core_avatar_folder_dir in bp_core_fetch_avatar
--------------------------+-----------------------------
 Reporter:  shawfactor    |      Owner:  (none)
     Type:  defect (bug)  |     Status:  new
 Priority:  normal        |  Milestone:  Awaiting Review
Component:  Core          |    Version:
 Severity:  normal        |   Keywords:
--------------------------+-----------------------------
 The order of these two filters is illogical as it makes it difficult to
 conditionally filter both of these

 The common use case is to check if a file exists in a directory and then
 if it does change the directory and the the url. But this is difficult as
 riht now the url filter is run first.

 Th patch is simple, just swap the order of the two cases of apply_filters.

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


More information about the buddypress-trac mailing list