Rename bbp_pre_notify_subscribers and bbp_post_notify_subscribers hooks

[netweb]

In r5413 - "Deprecate bbp_notify_subscribers() for new bbp_notify_topic_subscribers() function, to better match bbp_notify_forum_subscribers() introduced in 2.5."

We should do the same for bbp_pre_notify_subscribers and bbp_post_notify_subscribers hooks.

Renaming them bbp_pre_notify_topic_subscribers and bbp_post_notify_topic_subscribers respectively.

Better matching bbp_notify_forum_subscribers() bbp_pre_notify_forum_subscribers and bbp_post_notify_forum_subscribers hooks.

[tharsheblows]

I wouldn't change it. Just googling, it looks like there are definitely some plugins using it, eg ​https://wordpress.org/support/plugin/bbpressmoderation and ​https://github.com/rmccue/bbPress-Reply-by-Email and maybe other people too like her: ​https://bbpress.org/forums/topic/capability-needed-for-subscription-notification/

I guess on the other hand, changing it would make it obvious what it does. Although it really is pretty well commented and not unobvious at all.

The big thing is, subscriptions were just changed and some people had to change what they did using these hooks. I've just done that and if I updated without carefully checking, my users would be getting two subscription emails for every new reply (depending on how 'remove_action' fails, I suppose, I don't know).

So I'd leave it for now at least because the best case is that it looks better and makes it slightly easier in the future and the worst case is that you seriously annoy people who have just had to deal with it in the last update.

[netweb]

Agreed, maybelater, maybe never thanks :)