TwoEightNine / XVII

android messenger for vk social network
https://play.google.com/store/apps/details?id=com.twoeightnine.root.xvii
GNU General Public License v3.0
79 stars 13 forks source link

[BUG] Pinned dialogs don't work properly in case of multiple users #17

Open scorpioza opened 1 year ago

scorpioza commented 1 year ago

Описание При переключении между авторизованными пользователями закреплённые сообщения перестают быть закрепленными. При включении оффлайн отображаются диалоги предыдущего пользователя

Причина В бд. в таблице Dialogs не запоминается, для какого авторизованного пользователя отображены эти диалоги. Это приводит к тому, что если переключаться между аккаунтами, то закрепленные диалоги первого пользователя могут наследоваться у второго, а в оффлайн режиме, где сообщения берутся только из таблицы Dialogs, отображаются диалоги предыдущего пользователя.

Моё решение Не делаю pull-request, поскольку не уверена, что решение удачное. Но другого я не придумала

1) Dialogs. Добавляю поле "me"

@Parcelize
@Entity(tableName = "dialogs", primaryKeys = ["peerId","me"])
data class Dialog(
        val peerId: Int = 0,
       ......
        var me: Int = 0,

2) DialogsDao:

    @Query("SELECT * FROM dialogs WHERE :me = me ORDER BY isPinned DESC, timeStamp DESC")
    fun getDialogs(me: Int): Single<List<Dialog>>

    @Query("SELECT * FROM dialogs WHERE :peerId = peerId AND :me = me")
    fun getDialogs(me: Int, peerId: Int): Single<Dialog>

    @Query("SELECT * FROM dialogs WHERE peerId IN (:peerIds) AND :me = me")
    fun getDialogs(me: Int, peerIds: List<Int>): Single<List<Dialog>>

3) В коде в нескольких местах добавляется SessionProvider.userId. Например:

    fun loadAlias(peerId: Int) {
        appDb.dialogsDao()
                .getDialogs(SessionProvider.userId, peerId)