I don't really like this. The method is pulling most inputs from its params, but guestAccess from `this`. Possibly this whole `if.guestAccess` block could be pulled up a load? In fact, wtf even is`guestAccess` and why is it a member of `MatrixClientPeg`? 