We should stop emitting channels together, it's just really complicated. Let's get rid of this function.
I'll open a spec bug about this kind of thing now too.
Here's a patch to deprecate it in master. Once merged I'll merge master into next and remove it there.
What do you think?
Great, thanks. I pushed this master, then merged master into next.
Here's the branch for next to remove the function.
Please fix the CMs too, I see gabble at least still use it
gabble and salut still use this for announcing Text MUC channels & Tube & Tubes channels together.
I have a gabble branch to remove Tubes and fix this case so emit_new_channels isn't called. This should probably be the start of the gabble 'next' branch. I've yet to do the same thing in salut.
We will need to add some kind of workaround to gabble and salut master so it doesn't call the deprecated emit_new_channels.