swap the order to the two filters bp_core_avatar_folder_url and bp_core_avatar_folder_dir in bp_core_fetch_avatar

[shawfactor]

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.

[imath]

hi @shawfactor

Sure, can you create this patch ?

[shawfactor]

New file with the filetr order swapped (it is more logical and useful that way)

[shawfactor]

Sure here it is, as I said it is very simple I am just swapping the order of the filters. If my reasons are unclear or you need further explanation on why it is more logical thay way please let me know. Is this what you mean by create the patch?

[imath]

Thanks I was more thinking about a patch format. We've just published a ​documentation page on our codex about how to generate a patch, could you have a look and try to use a patch instead of the full file ?

[devutpol]

Patch created