Rename _store_room_member_txn to _store_room_member_from_event_txn so we can create another, more sensible function of that name
Add a better _store_room_member_txn() method that takes separated fields instead of an event object; also add FIXME comment about a big bug in the logic