Add try-catch when loading rooms

37795ffb · Leonardo Aramaki · f2db559a · 37795ffb
Commit 37795ffb authored Feb 08, 2018 by Leonardo Aramaki
Hide whitespace changes
Inline Side-by-side

Showing with 9 additions and 3 deletions

ChatRoomsPresenter.kt ...cket/android/chatrooms/presentation/ChatRoomsPresenter.kt +9 -3

No files found.
--- a/app/src/main/java/chat/rocket/android/chatrooms/presentation/ChatRoomsPresenter.kt
+++ b/app/src/main/java/chat/rocket/android/chatrooms/presentation/ChatRoomsPresenter.kt
@@ -38,9 +38,15 @@ class ChatRoomsPresenter @Inject constructor(private val view: ChatRoomsView,
    fun loadChatRooms() {
        launchUI(strategy) {
            view.showLoading()
-            view.updateChatRooms(loadRooms())
+            try {
-            subscribeRoomUpdates()
+                view.updateChatRooms(loadRooms())
-            view.hideLoading()
+                subscribeRoomUpdates()
+            } catch (e: RocketChatException) {
+                Timber.e(e)
+                view.showMessage(e.message!!)
+            } finally {
+                view.hideLoading()
+            }
        }
    }