Replace MatrixClient.isRoomEncrypted by MatrixClient.CryptoApi.isEncryptionEnabledInRoom in useIsEncrypted

[florianduros]

Checklist

• Tests written for new code (and old code if feasible).
• New or updated public/exported symbols have accurate TSDoc documentation.
• Linter and other CI checks pass.
• I have licensed the changes to Element by completing the Contributor License Agreement (CLA)

Task #26922

[t3chguy]

if encrypted is true and the room didn't change but isEncryptionEnabledInRoom threw should we really be setting to false? This seems unexpected.

We probably also need to reset to null when room changes whilst func is calling the async function which could take multiple ticks

[t3chguy]

Could probably be also done like this:

const encryptionStateEvent = useRoomState(...);
return useAsyncMemo(() => {...}, [room, encryptionStateEvent]);

[florianduros]

Could probably be also done like this:

const encryptionStateEvent = useRoomState(...);
return useAsyncMemo(() => {...}, [room, encryptionStateEvent]);

Are you suggesting to get the encryption state event with useRoomState(room, (roomState) => roomState.getStateEvents(EventType.RoomEncryption)). Where it should have only one (event)

[t3chguy]

Where it should have only one (event)

Yes but it'll be the dependency for useAsyncMemo so it reacts to state event updates