Inadequate details availabe on bp_attachments_pre_get_attachment

[sbrajesh]

The filter

bp_attachments_pre_get_attachment

Does not provide anyway to know which piece of data(is it url or path) is being fetched.

That renders the hook useless for most use case.

We should pass the $data too with the hook. Here is a patch attached with the same.

Please let me know if there is a better way to do it.

[sbrajesh]

Add $data as the 3rd parameter.

[imath]

Thanks for your report, let’s make it happen in 8.0.0

[imath]

In 12901:

Add a parameter to the bp_attachments_pre_get_attachment filter

Without the $data parameter it was difficult to know what was the requested data to get between the url or path. So it was pretty useless.

It's now possible to get this information as the third parameter of the filter.

Props sbrajesh

Fixes #8185