Merge pull request #4982 from matrix-org/rav/random_fix_1

sdk: remove redundant variable

Author: richvdh

crates/matrix-sdk/src/room/mod.rs

@@ -250,16 +250,14 @@ impl Room {
     /// Only invited and left rooms can be joined via this method.
     #[doc(alias = "accept_invitation")]
     pub async fn join(&self) -> Result<()> {
-        let state = self.state();
-        if state == RoomState::Joined {
+        let prev_room_state = self.inner.state();
+        if prev_room_state == RoomState::Joined {
             return Err(Error::WrongRoomState(Box::new(WrongRoomState::new(
                 "Invited or Left",
-                state,
+                prev_room_state,
             ))));
         }
 
-        let prev_room_state = self.inner.state();
-
         let mark_as_direct = prev_room_state == RoomState::Invited
             && self.inner.is_direct().await.unwrap_or_else(|e| {
                 warn!(room_id = ?self.room_id(), "is_direct() failed: {e}");