[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