rework of xep 0045 plugin

[allan-simon]

I've just seen this branch, that seems to be quite an improvement compare to current xep_0045

though I'm not the author of these changes, by creating this pull request i want show that i'm interested in knowing what this branch is missing to be merged in the main branch and I'm willing to help if necessary

thanks

[legastero]

This needs testing to be sure edge cases are covered.

[ashfire908]

I've been using this branch for a month now on a project of mine. Off-hand I can name a few bugs with it.

There's a bug in the _add_joined_room method where it erases the roster for the room, clearing the client itself from the roster (ie the client is added to the roster then the roster is reset). I ended up patching it here: ashfire908/SleekXMPP@ebcc934

I've also found various typos and errors around the plugin, which I've fixed here: ashfire908/SleekXMPP@22b11a6

There appears to be a risk that the blocking on joining a room will react to the wrong room join/fail: ashfire908/SleekXMPP@fe72c14 (looking at it again I don't think that commit fully fixed that)

Mediated invites do not work - there's an issue with the stanzas that the code tries to use, IIRC. I ended up doing this manually in my own project, so I don't have a patch for you.

Everything else I've used so far has worked, but I'd still look over it for other issues.

[allan-simon]

@ashfire908 thanks for the comment it's pretty informative. Is it ok for you to create a pull request for the new_muc branch, so that it benefits from your various fixes. If you want I can take the burden and do it myself by cherry-picking your commits.

[ashfire908]

Done. I ended up rebasing my commits on the current new_muc. I was able to fix mediated invites, too. Pull request is #283. The only thing I know of left is to clean up the imports, but that's minor.

[bear]

@ashfire908 I updated the new_muc branch with the latest changes to develop but now i'm seeing test errors - can you look into them?

[ashfire908]

Sorry for the long delay before looking into this, I haven't been working on the project of mine that uses SleekXMPP until recently.

I have traced the failing tests due to these changes:

• https://github.com/fritzy/SleekXMPP/pull/279/files#diff-c12f981bbed099442fe509b80649eec3L718
• https://github.com/fritzy/SleekXMPP/pull/279/files#diff-c12f981bbed099442fe509b80649eec3L806

Reverting both of these edits allows the tests to pass. I'm not sure what purpose they had, but I had no issues with my project with those changes undone.