Merge pull request #5 from RocketChat/develop

merge

app/build.gradle

@@ -2,7 +2,9 @@ def taskRequests = getGradle().getStartParameter().getTaskRequests().toString()
 def isPlay = !(taskRequests.contains("Foss") || taskRequests.contains("foss"))
 
 apply plugin: 'com.android.application'
-if (isPlay) { apply plugin: 'io.fabric' }
+if (isPlay) {
+    apply plugin: 'io.fabric'
+}
 apply plugin: 'kotlin-android'
 apply plugin: 'kotlin-android-extensions'
 apply plugin: 'kotlin-kapt'
@@ -16,13 +18,12 @@ android {
         applicationId "chat.rocket.android"
         minSdkVersion versions.minSdk
         targetSdkVersion versions.targetSdk
-        versionCode 2057
-        versionName "3.2.0"
+        versionCode 2058
+        versionName "3.3.0"
         testInstrumentationRunner "androidx.test.runner.AndroidJUnitRunner"
         multiDexEnabled true
 
         def gitSha = 'git rev-parse --short HEAD'.execute([], project.rootDir).text.trim()
-        def buildTime = new GregorianCalendar().format("MM-dd-yyyy' 'h:mm:ss a z")
         buildConfigField "String", "GIT_SHA", "\"${gitSha}\""
 
         javaCompileOptions {
@@ -30,6 +31,17 @@ android {
                 arguments = ["room.schemaLocation": "$projectDir/schemas".toString()]
             }
         }
+
+        // For Jitsi
+        compileOptions {
+            sourceCompatibility JavaVersion.VERSION_1_8
+            targetCompatibility JavaVersion.VERSION_1_8
+        }
+
+        // For Jitsi
+        ndk {
+            abiFilters "armeabi-v7a", "x86"
+        }
     }
 
     signingConfigs {
@@ -73,7 +85,7 @@ android {
             dimension "type"
         }
 
-        // only foss
+        // only FOSS
         foss {
             dimension "type"
         }
@@ -83,6 +95,10 @@ android {
         exclude 'META-INF/core.kotlin_module'
         exclude 'META-INF/main.kotlin_module'
     }
+
+    lintOptions {
+        lintConfig file("src/main/res/xml/lint.xml")
+    }
 }
 
 dependencies {
@@ -96,7 +112,7 @@ dependencies {
     implementation project(':suggestions')
 
     implementation libraries.kotlin
-    implementation libraries.coroutines
+    implementation libraries.coroutinesCore
     implementation libraries.coroutinesAndroid
 
     implementation libraries.appCompat
@@ -123,6 +139,8 @@ dependencies {
     implementation libraries.viewmodelKtx
     implementation libraries.workmanager
 
+    implementation libraries.livedataKtx
+
     implementation libraries.rxKotlin
     implementation libraries.rxAndroid
 
@@ -133,25 +151,25 @@ dependencies {
     implementation libraries.timber
     implementation libraries.threeTenABP
 
+    kapt libraries.kotshiCompiler
+    implementation libraries.kotshiApi
+
     implementation libraries.fresco
     api libraries.frescoOkHttp
     implementation libraries.frescoAnimatedGif
     implementation libraries.frescoWebP
     implementation libraries.frescoAnimatedWebP
 
-    implementation libraries.glide
-    implementation libraries.glideTransformations
-
-    kapt libraries.kotshiCompiler
-    implementation libraries.kotshiApi
-
     implementation libraries.frescoImageViewer
 
     implementation libraries.markwon
 
     implementation libraries.aVLoadingIndicatorView
 
-    implementation libraries.livedataKtx
+    implementation libraries.glide
+    implementation libraries.glideTransformations
+
+    implementation(libraries.jitsi) { transitive = true }
 
     implementation 'com.google.code.findbugs:jsr305:3.0.2'
 
@@ -159,8 +177,8 @@ dependencies {
     playImplementation libraries.fcm
     playImplementation libraries.firebaseAnalytics
     playImplementation libraries.playServicesAuth
-    playImplementation('com.crashlytics.sdk.android:crashlytics:2.9.5@aar') { transitive = true }
-    playImplementation('com.crashlytics.sdk.android:answers:1.4.3@aar') { transitive = true }
+    playImplementation('com.crashlytics.sdk.android:crashlytics:2.9.8@aar') { transitive = true }
+    playImplementation('com.crashlytics.sdk.android:answers:1.4.6@aar') { transitive = true }
 
     testImplementation libraries.junit
     testImplementation libraries.truth
@@ -168,12 +186,6 @@ dependencies {
     androidTestImplementation libraries.espressoIntents
 }
 
-kotlin {
-    experimental {
-        coroutines "enable"
-    }
-}
-
 androidExtensions {
     experimental = true
 }
@@ -181,8 +193,8 @@ androidExtensions {
 // FIXME - build and install the sdk into the app/libs directory
 // We were having some issues with the kapt generated files from the sdk when importing as a module
 
-def sdk_location=project.properties['sdk_location'] ?: ""
-task compileSdk(type:Exec) {
+def sdk_location = project.properties['sdk_location'] ?: ""
+task compileSdk(type: Exec) {
     if (System.getProperty('os.name').toLowerCase(Locale.ROOT).contains('windows')) {
         commandLine 'cmd', '/c', 'build-sdk.sh', sdk_location
     } else {

app/src/main/AndroidManifest.xml

@@ -6,6 +6,8 @@
     <uses-permission android:name="android.permission.INTERNET" />
     <uses-permission android:name="android.permission.VIBRATE" />
     <uses-permission android:name="android.permission.WRITE_EXTERNAL_STORAGE" />
+    <uses-permission android:name="android.permission.CAMERA" />
+    <uses-permission android:name="android.permission.RECORD_AUDIO" />
 
     <application
         android:name=".app.RocketChatApplication"
@@ -73,6 +75,11 @@
             android:theme="@style/AppTheme"
             android:windowSoftInputMode="adjustResize|stateAlwaysHidden" />
 
+        <activity
+            android:name=".videoconference.ui.VideoConferenceActivity"
+            android:theme="@style/AppTheme"
+            android:windowSoftInputMode="adjustResize|stateAlwaysHidden" />
+
         <activity
             android:name=".chatroom.ui.ChatRoomActivity"
             android:theme="@style/AppTheme"

app/src/main/java/chat/rocket/android/analytics/Analytics.kt

@@ -71,4 +71,13 @@ interface Analytics {
      * @param resetPasswordSucceeded True if successful reset password, false otherwise.
      */
     fun logResetPassword(resetPasswordSucceeded: Boolean) {}
+
+    /**
+     * Logs the video conference event.
+     *
+     * @param event The [SubscriptionTypeEvent] to log.
+     * @param serverUrl The server URL to log.
+     */
+    fun logVideoConference(event: SubscriptionTypeEvent, serverUrl: String) {}
+
 }

app/src/main/java/chat/rocket/android/analytics/AnalyticsManager.kt

@@ -76,4 +76,12 @@ class AnalyticsManager @Inject constructor(
             analytics.forEach { it.logResetPassword(resetPasswordSucceeded) }
         }
     }
+
+    fun logVideoConference(event: SubscriptionTypeEvent) {
+        if (analyticsTrackingInteractor.get() && serverUrl != null) {
+            analytics.forEach { it.logVideoConference(event, serverUrl) }
+        }
+    }
+
+
 }

app/src/main/java/chat/rocket/android/app/AppLifecycleObserver.kt

@@ -3,15 +3,11 @@ package chat.rocket.android.app
 import androidx.lifecycle.Lifecycle
 import androidx.lifecycle.LifecycleObserver
 import androidx.lifecycle.OnLifecycleEvent
-import chat.rocket.android.server.domain.GetAccountInteractor
 import chat.rocket.android.server.domain.GetCurrentServerInteractor
 import chat.rocket.android.server.infraestructure.ConnectionManagerFactory
-import chat.rocket.android.server.infraestructure.RocketChatClientFactory
-import chat.rocket.common.RocketChatException
 import chat.rocket.common.model.UserStatus
-import chat.rocket.core.internal.realtime.setTemporaryStatus
-import kotlinx.coroutines.experimental.launch
-import timber.log.Timber
+import kotlinx.coroutines.GlobalScope
+import kotlinx.coroutines.launch
 import javax.inject.Inject
 
 class AppLifecycleObserver @Inject constructor(
@@ -33,7 +29,7 @@ class AppLifecycleObserver @Inject constructor(
     }
 
     private fun changeTemporaryStatus(userStatus: UserStatus) {
-        launch {
+        GlobalScope.launch {
             serverInteractor.get()?.let { currentServer ->
                 factory.create(currentServer).setTemporaryStatus(userStatus)
             }

app/src/main/java/chat/rocket/android/app/DateTimeHelper.kt

 import android.content.Context
 import chat.rocket.android.R
-import org.threeten.bp.*
+import org.threeten.bp.Instant
+import org.threeten.bp.LocalDate
+import org.threeten.bp.LocalDateTime
+import org.threeten.bp.LocalTime
+import org.threeten.bp.Period
+import org.threeten.bp.ZoneId
 import org.threeten.bp.format.DateTimeFormatter
 import org.threeten.bp.format.FormatStyle
 import org.threeten.bp.format.TextStyle
@@ -53,39 +58,39 @@ object DateTimeHelper {
         }
     }
 
-/**
- * Returns a time from a [LocalDateTime].
- *
- * @param localDateTime The [LocalDateTime].
- * @return The time from a [LocalDateTime].
- */
-fun getTime(localDateTime: LocalDateTime): String {
-    val formatter = DateTimeFormatter.ofLocalizedTime(FormatStyle.SHORT)
-    return localDateTime.toLocalTime().format(formatter).toString()
-}
+    /**
+     * Returns a time from a [LocalDateTime].
+     *
+     * @param localDateTime The [LocalDateTime].
+     * @return The time from a [LocalDateTime].
+     */
+    fun getTime(localDateTime: LocalDateTime): String {
+        val formatter = DateTimeFormatter.ofLocalizedTime(FormatStyle.SHORT)
+        return localDateTime.toLocalTime().format(formatter).toString()
+    }
 
-/**
- * Returns a date time from a [LocalDateTime].
- *
- * @param localDateTime The [LocalDateTime].
- * @return The time from a [LocalDateTime].
- */
-fun getDateTime(localDateTime: LocalDateTime): String {
-    return formatLocalDateTime(localDateTime)
-}
+    /**
+     * Returns a date time from a [LocalDateTime].
+     *
+     * @param localDateTime The [LocalDateTime].
+     * @return The time from a [LocalDateTime].
+     */
+    fun getDateTime(localDateTime: LocalDateTime): String {
+        return formatLocalDateTime(localDateTime)
+    }
 
-private fun formatLocalDateTime(localDateTime: LocalDateTime): String {
-    val formatter = DateTimeFormatter.ofLocalizedDateTime(FormatStyle.SHORT)
-    return localDateTime.format(formatter).toString()
-}
+    private fun formatLocalDateTime(localDateTime: LocalDateTime): String {
+        val formatter = DateTimeFormatter.ofLocalizedDateTime(FormatStyle.SHORT)
+        return localDateTime.format(formatter).toString()
+    }
 
-private fun formatLocalDate(localDate: LocalDate): String {
-    val formatter = DateTimeFormatter.ofLocalizedDate(FormatStyle.MEDIUM)
-    return localDate.format(formatter).toString()
-}
+    private fun formatLocalDate(localDate: LocalDate): String {
+        val formatter = DateTimeFormatter.ofLocalizedDate(FormatStyle.MEDIUM)
+        return localDate.format(formatter).toString()
+    }
 
-private fun formatLocalTime(localTime: LocalTime): String {
-    val formatter = DateTimeFormatter.ofLocalizedTime(FormatStyle.SHORT)
-    return localTime.format(formatter).toString()
-}
+    private fun formatLocalTime(localTime: LocalTime): String {
+        val formatter = DateTimeFormatter.ofLocalizedTime(FormatStyle.SHORT)
+        return localTime.format(formatter).toString()
+    }
 }
\ No newline at end of file

app/src/main/java/chat/rocket/android/app/DrawableHelper.kt

@@ -91,7 +91,7 @@ object DrawableHelper {
     }
 
     /**
-     * Compounds a Drawable (to appear to the left of the text) into a TextView.
+     * Compounds a Drawable (to appear on the left side of a text) into a TextView.
      *
      * @param textView The TextView.
      * @param drawable The Drawable.
@@ -100,6 +100,16 @@ object DrawableHelper {
     fun compoundDrawable(textView: TextView, drawable: Drawable) =
         textView.setCompoundDrawablesWithIntrinsicBounds(drawable, null, null, null)
 
+    /**
+     * Compounds a Drawable (to appear on the right side of a text) into a TextView.
+     *
+     * @param textView The TextView.
+     * @param drawable The Drawable.
+     * @see compoundDrawable
+     */
+    fun compoundRightDrawable(textView: TextView, drawable: Drawable) =
+        textView.setCompoundDrawablesWithIntrinsicBounds(null, null, drawable, null)
+
     /**
      * Returns the user status drawable.
      *

app/src/main/java/chat/rocket/android/app/RocketChatApplication.kt

@@ -37,7 +37,8 @@ import dagger.android.DispatchingAndroidInjector
 import dagger.android.HasActivityInjector
 import dagger.android.HasBroadcastReceiverInjector
 import dagger.android.HasServiceInjector
-import kotlinx.coroutines.experimental.launch
+import kotlinx.coroutines.GlobalScope
+import kotlinx.coroutines.launch
 import timber.log.Timber
 import java.lang.ref.WeakReference
 import javax.inject.Inject
@@ -174,7 +175,7 @@ class RocketChatApplication : Application(), HasActivityInjector, HasServiceInje
         EmojiRepository.init(this)
         val currentServer = getCurrentServerInteractor.get()
         currentServer?.let { server ->
-            launch {
+            GlobalScope.launch {
                 val client = factory.create(server)
                 EmojiRepository.setCurrentServerUrl(server)
                 val customEmojiList = mutableListOf<Emoji>()

app/src/main/java/chat/rocket/android/authentication/di/AuthenticationModule.kt

@@ -7,7 +7,7 @@ import chat.rocket.android.core.lifecycle.CancelStrategy
 import chat.rocket.android.dagger.scope.PerActivity
 import dagger.Module
 import dagger.Provides
-import kotlinx.coroutines.experimental.Job
+import kotlinx.coroutines.Job
 
 @Module
 class AuthenticationModule {

app/src/main/java/chat/rocket/android/authentication/infraestructure/SharedPreferencesMultiServerTokenRepository.kt

@@ -8,19 +8,16 @@ import chat.rocket.android.server.domain.MultiServerTokenRepository
 import com.squareup.moshi.Moshi
 
 @PerActivity
-class SharedPreferencesMultiServerTokenRepository(private val repository: LocalRepository,
-                                                  private val moshi: Moshi
+class SharedPreferencesMultiServerTokenRepository(
+    private val repository: LocalRepository,
+    private val moshi: Moshi
 ) : MultiServerTokenRepository {
 
     override fun get(server: String): TokenModel? {
         val token = repository.get("$TOKEN_KEY$server")
         val adapter = moshi.adapter<TokenModel>(TokenModel::class.java)
 
-        token?.let {
-            return adapter.fromJson(token)
-        }
-
-        return null
+        return token?.let { adapter.fromJson(it) }
     }
 
     override fun save(server: String, token: TokenModel) {

app/src/main/java/chat/rocket/android/authentication/login/ui/LoginFragment.kt

@@ -39,11 +39,9 @@ internal const val REQUEST_CODE_FOR_SIGN_IN_REQUIRED = 1
 internal const val REQUEST_CODE_FOR_MULTIPLE_ACCOUNTS_RESOLUTION = 2
 internal const val REQUEST_CODE_FOR_SAVE_RESOLUTION = 3
 
-fun newInstance(serverName: String): Fragment {
-    return LoginFragment().apply {
-        arguments = Bundle(1).apply {
-            putString(SERVER_NAME, serverName)
-        }
+fun newInstance(serverName: String): Fragment = LoginFragment().apply {
+    arguments = Bundle(1).apply {
+        putString(SERVER_NAME, serverName)
     }
 }
 
@@ -59,9 +57,8 @@ class LoginFragment : Fragment(), LoginView {
         super.onCreate(savedInstanceState)
         AndroidSupportInjection.inject(this)
 
-        val bundle = arguments
-        if (bundle != null) {
-            serverName = bundle.getString(SERVER_NAME)
+        arguments?.run {
+            serverName = getString(SERVER_NAME)
         }
     }
 
@@ -93,7 +90,7 @@ class LoginFragment : Fragment(), LoginView {
                             text_username_or_email.setText(credential.first)
                             text_password.setText(credential.second)
                         }
-                    REQUEST_CODE_FOR_SAVE_RESOLUTION -> showMessage(getString(R.string.message_credentials_saved_successfully))
+                    REQUEST_CODE_FOR_SAVE_RESOLUTION -> showMessage(getString(R.string.msg_credentials_saved_successfully))
                 }
             }
         }
@@ -150,7 +147,7 @@ class LoginFragment : Fragment(), LoginView {
     override fun showGenericErrorMessage() = showMessage(R.string.msg_generic_error)
 
     private fun setupOnClickListener() =
-        ui { _ ->
+        ui {
             button_log_in.setOnClickListener {
                 presenter.authenticateWithUserAndPassword(
                     text_username_or_email.textContent,
@@ -160,7 +157,7 @@ class LoginFragment : Fragment(), LoginView {
         }
 
     override fun showForgotPasswordView() {
-        ui { _ ->
+        ui {
             button_forgot_your_password.isVisible = true
             button_forgot_your_password.setOnClickListener { presenter.forgotPassword() }
 

app/src/main/java/chat/rocket/android/authentication/loginoptions/presentation/LoginOptionsPresenter.kt

@@ -31,8 +31,7 @@ import chat.rocket.core.internal.rest.loginWithCas
 import chat.rocket.core.internal.rest.loginWithOauth
 import chat.rocket.core.internal.rest.loginWithSaml
 import chat.rocket.core.internal.rest.me
-import kotlinx.coroutines.experimental.delay
-import java.util.concurrent.TimeUnit
+import kotlinx.coroutines.delay
 import javax.inject.Inject
 
 private const val TYPE_LOGIN_OAUTH = 1
@@ -109,11 +108,11 @@ class LoginOptionsPresenter @Inject constructor(
                     when (loginType) {
                         TYPE_LOGIN_OAUTH -> client.loginWithOauth(credentialToken, credentialSecret)
                         TYPE_LOGIN_CAS -> {
-                            delay(3, TimeUnit.SECONDS)
+                            delay(3000)
                             client.loginWithCas(credentialToken)
                         }
                         TYPE_LOGIN_SAML -> {
-                            delay(3, TimeUnit.SECONDS)
+                            delay(3000)
                             client.loginWithSaml(credentialToken)
                         }
                         TYPE_LOGIN_DEEP_LINK -> {

app/src/main/java/chat/rocket/android/authentication/loginoptions/ui/LoginOptionsFragment.kt

@@ -88,36 +88,34 @@ fun newInstance(
     isLoginFormEnabled: Boolean,
     isNewAccountCreationEnabled: Boolean,
     deepLinkInfo: LoginDeepLinkInfo? = null
-): Fragment {
-    return LoginOptionsFragment().apply {
-        arguments = Bundle(23).apply {
-            putString(SERVER_NAME, serverName)
-            putString(STATE, state)
-            putString(FACEBOOK_OAUTH_URL, facebookOauthUrl)
-            putString(GITHUB_OAUTH_URL, githubOauthUrl)
-            putString(GOOGLE_OAUTH_URL, googleOauthUrl)
-            putString(LINKEDIN_OAUTH_URL, linkedinOauthUrl)
-            putString(GITLAB_OAUTH_URL, gitlabOauthUrl)
-            putString(WORDPRESS_OAUTH_URL, wordpressOauthUrl)
-            putString(CAS_LOGIN_URL, casLoginUrl)
-            putString(CAS_TOKEN, casToken)
-            putString(CAS_SERVICE_NAME, casServiceName)
-            putInt(CAS_SERVICE_NAME_TEXT_COLOR, casServiceNameTextColor)
-            putInt(CAS_SERVICE_BUTTON_COLOR, casServiceButtonColor)
-            putString(CUSTOM_OAUTH_URL, customOauthUrl)
-            putString(CUSTOM_OAUTH_SERVICE_NAME, customOauthServiceName)
-            putInt(CUSTOM_OAUTH_SERVICE_NAME_TEXT_COLOR, customOauthServiceNameTextColor)
-            putInt(CUSTOM_OAUTH_SERVICE_BUTTON_COLOR, customOauthServiceButtonColor)
-            putString(SAML_URL, samlUrl)
-            putString(SAML_TOKEN, samlToken)
-            putString(SAML_SERVICE_NAME, samlServiceName)
-            putInt(SAML_SERVICE_NAME_TEXT_COLOR, samlServiceNameTextColor)
-            putInt(SAML_SERVICE_BUTTON_COLOR, samlServiceButtonColor)
-            putInt(TOTAL_SOCIAL_ACCOUNTS, totalSocialAccountsEnabled)
-            putBoolean(IS_LOGIN_FORM_ENABLED, isLoginFormEnabled)
-            putBoolean(IS_NEW_ACCOUNT_CREATION_ENABLED, isNewAccountCreationEnabled)
-            putParcelable(DEEP_LINK_INFO, deepLinkInfo)
-        }
+): Fragment = LoginOptionsFragment().apply {
+    arguments = Bundle(23).apply {
+        putString(SERVER_NAME, serverName)
+        putString(STATE, state)
+        putString(FACEBOOK_OAUTH_URL, facebookOauthUrl)
+        putString(GITHUB_OAUTH_URL, githubOauthUrl)
+        putString(GOOGLE_OAUTH_URL, googleOauthUrl)
+        putString(LINKEDIN_OAUTH_URL, linkedinOauthUrl)
+        putString(GITLAB_OAUTH_URL, gitlabOauthUrl)
+        putString(WORDPRESS_OAUTH_URL, wordpressOauthUrl)
+        putString(CAS_LOGIN_URL, casLoginUrl)
+        putString(CAS_TOKEN, casToken)
+        putString(CAS_SERVICE_NAME, casServiceName)
+        putInt(CAS_SERVICE_NAME_TEXT_COLOR, casServiceNameTextColor)
+        putInt(CAS_SERVICE_BUTTON_COLOR, casServiceButtonColor)
+        putString(CUSTOM_OAUTH_URL, customOauthUrl)
+        putString(CUSTOM_OAUTH_SERVICE_NAME, customOauthServiceName)
+        putInt(CUSTOM_OAUTH_SERVICE_NAME_TEXT_COLOR, customOauthServiceNameTextColor)
+        putInt(CUSTOM_OAUTH_SERVICE_BUTTON_COLOR, customOauthServiceButtonColor)
+        putString(SAML_URL, samlUrl)
+        putString(SAML_TOKEN, samlToken)
+        putString(SAML_SERVICE_NAME, samlServiceName)
+        putInt(SAML_SERVICE_NAME_TEXT_COLOR, samlServiceNameTextColor)
+        putInt(SAML_SERVICE_BUTTON_COLOR, samlServiceButtonColor)
+        putInt(TOTAL_SOCIAL_ACCOUNTS, totalSocialAccountsEnabled)
+        putBoolean(IS_LOGIN_FORM_ENABLED, isLoginFormEnabled)
+        putBoolean(IS_NEW_ACCOUNT_CREATION_ENABLED, isNewAccountCreationEnabled)
+        putParcelable(DEEP_LINK_INFO, deepLinkInfo)
     }
 }
 
@@ -157,34 +155,33 @@ class LoginOptionsFragment : Fragment(), LoginOptionsView {
         super.onCreate(savedInstanceState)
         AndroidSupportInjection.inject(this)
 
-        val bundle = arguments
-        if (bundle != null) {
-            serverName = bundle.getString(SERVER_NAME)
-            state = bundle.getString(STATE)
-            facebookOauthUrl = bundle.getString(FACEBOOK_OAUTH_URL)
-            githubOauthUrl = bundle.getString(GITHUB_OAUTH_URL)
-            googleOauthUrl = bundle.getString(GOOGLE_OAUTH_URL)
-            linkedinOauthUrl = bundle.getString(LINKEDIN_OAUTH_URL)
-            gitlabOauthUrl = bundle.getString(GITLAB_OAUTH_URL)
-            wordpressOauthUrl = bundle.getString(WORDPRESS_OAUTH_URL)
-            casLoginUrl = bundle.getString(CAS_LOGIN_URL)
-            casToken = bundle.getString(CAS_TOKEN)
-            casServiceName = bundle.getString(CAS_SERVICE_NAME)
-            casServiceNameTextColor = bundle.getInt(CAS_SERVICE_NAME_TEXT_COLOR)
-            casServiceButtonColor = bundle.getInt(CAS_SERVICE_BUTTON_COLOR)
-            customOauthUrl = bundle.getString(CUSTOM_OAUTH_URL)
-            customOauthServiceName = bundle.getString(CUSTOM_OAUTH_SERVICE_NAME)
-            customOauthServiceTextColor = bundle.getInt(CUSTOM_OAUTH_SERVICE_NAME_TEXT_COLOR)
-            customOauthServiceButtonColor = bundle.getInt(CUSTOM_OAUTH_SERVICE_BUTTON_COLOR)
-            samlUrl = bundle.getString(SAML_URL)
-            samlToken = bundle.getString(SAML_TOKEN)
-            samlServiceName = bundle.getString(SAML_SERVICE_NAME)
-            samlServiceTextColor = bundle.getInt(SAML_SERVICE_NAME_TEXT_COLOR)
-            samlServiceButtonColor = bundle.getInt(SAML_SERVICE_BUTTON_COLOR)
-            totalSocialAccountsEnabled = bundle.getInt(TOTAL_SOCIAL_ACCOUNTS)
-            isLoginFormEnabled = bundle.getBoolean(IS_LOGIN_FORM_ENABLED)
-            isNewAccountCreationEnabled = bundle.getBoolean(IS_NEW_ACCOUNT_CREATION_ENABLED)
-            deepLinkInfo = bundle.getParcelable(DEEP_LINK_INFO)
+        arguments?.run {
+            serverName = getString(SERVER_NAME)
+            state = getString(STATE)
+            facebookOauthUrl = getString(FACEBOOK_OAUTH_URL)
+            githubOauthUrl = getString(GITHUB_OAUTH_URL)
+            googleOauthUrl = getString(GOOGLE_OAUTH_URL)
+            linkedinOauthUrl = getString(LINKEDIN_OAUTH_URL)
+            gitlabOauthUrl = getString(GITLAB_OAUTH_URL)
+            wordpressOauthUrl = getString(WORDPRESS_OAUTH_URL)
+            casLoginUrl = getString(CAS_LOGIN_URL)
+            casToken = getString(CAS_TOKEN)
+            casServiceName = getString(CAS_SERVICE_NAME)
+            casServiceNameTextColor = getInt(CAS_SERVICE_NAME_TEXT_COLOR)
+            casServiceButtonColor = getInt(CAS_SERVICE_BUTTON_COLOR)
+            customOauthUrl = getString(CUSTOM_OAUTH_URL)
+            customOauthServiceName = getString(CUSTOM_OAUTH_SERVICE_NAME)
+            customOauthServiceTextColor = getInt(CUSTOM_OAUTH_SERVICE_NAME_TEXT_COLOR)
+            customOauthServiceButtonColor = getInt(CUSTOM_OAUTH_SERVICE_BUTTON_COLOR)
+            samlUrl = getString(SAML_URL)
+            samlToken = getString(SAML_TOKEN)
+            samlServiceName = getString(SAML_SERVICE_NAME)
+            samlServiceTextColor = getInt(SAML_SERVICE_NAME_TEXT_COLOR)
+            samlServiceButtonColor = getInt(SAML_SERVICE_BUTTON_COLOR)
+            totalSocialAccountsEnabled = getInt(TOTAL_SOCIAL_ACCOUNTS)
+            isLoginFormEnabled = getBoolean(IS_LOGIN_FORM_ENABLED)
+            isNewAccountCreationEnabled = getBoolean(IS_NEW_ACCOUNT_CREATION_ENABLED)
+            deepLinkInfo = getParcelable(DEEP_LINK_INFO)
         }
     }
 
@@ -388,7 +385,7 @@ class LoginOptionsFragment : Fragment(), LoginOptionsView {
     }
 
     override fun setupExpandAccountsView() {
-        ui { _ ->
+        ui {
             expand_more_accounts_container.isVisible = true
             var isAccountsCollapsed = true
             button_expand_collapse_accounts.setOnClickListener {
@@ -406,14 +403,14 @@ class LoginOptionsFragment : Fragment(), LoginOptionsView {
     }
 
     override fun showLoginWithEmailButton() {
-        ui { _ ->
+        ui {
             button_login_with_email.setOnClickListener { presenter.toLoginWithEmail() }
             button_login_with_email.isVisible = true
         }
     }
 
     override fun showCreateNewAccountButton() {
-        ui { _ ->
+        ui {
             button_create_an_account.setOnClickListener { presenter.toCreateAccount() }
             button_create_an_account.isVisible = true
         }

app/src/main/java/chat/rocket/android/authentication/onboarding/presentation/OnBoardingPresenter.kt

@@ -10,8 +10,8 @@ import chat.rocket.android.server.domain.SaveConnectingServerInteractor
 import chat.rocket.android.server.infraestructure.RocketChatClientFactory
 import chat.rocket.android.server.presentation.CheckServerPresenter
 import chat.rocket.android.util.extension.launchUI
-import kotlinx.coroutines.experimental.DefaultDispatcher
-import kotlinx.coroutines.experimental.withContext
+import kotlinx.coroutines.Dispatchers
+import kotlinx.coroutines.withContext
 import javax.inject.Inject
 
 class OnBoardingPresenter @Inject constructor(
@@ -19,7 +19,7 @@ class OnBoardingPresenter @Inject constructor(
     private val strategy: CancelStrategy,
     private val navigator: AuthenticationNavigator,
     private val serverInteractor: SaveConnectingServerInteractor,
-    private val refreshSettingsInteractor: RefreshSettingsInteractor,
+    refreshSettingsInteractor: RefreshSettingsInteractor,
     private val getAccountsInteractor: GetAccountsInteractor,
     val settingsInteractor: GetSettingsInteractor,
     val factory: RocketChatClientFactory
@@ -80,7 +80,7 @@ class OnBoardingPresenter @Inject constructor(
             }
             view.showLoading()
             try {
-                withContext(DefaultDispatcher) {
+                withContext(Dispatchers.Default) {
                     setupConnectionInfo(serverUrl)
 
                     // preparing next fragment before showing it

app/src/main/java/chat/rocket/android/authentication/registerusername/ui/RegisterUsernameFragment.kt

@@ -31,12 +31,10 @@ import javax.inject.Inject
 private const val BUNDLE_USER_ID = "user_id"
 private const val BUNDLE_AUTH_TOKEN = "auth_token"
 
-fun newInstance(userId: String, authToken: String): Fragment {
-    return RegisterUsernameFragment().apply {
-        arguments = Bundle(2).apply {
-            putString(BUNDLE_USER_ID, userId)
-            putString(BUNDLE_AUTH_TOKEN, authToken)
-        }
+fun newInstance(userId: String, authToken: String): Fragment = RegisterUsernameFragment().apply {
+    arguments = Bundle(2).apply {
+        putString(BUNDLE_USER_ID, userId)
+        putString(BUNDLE_AUTH_TOKEN, authToken)
     }
 }
 
@@ -53,13 +51,10 @@ class RegisterUsernameFragment : Fragment(), RegisterUsernameView {
         super.onCreate(savedInstanceState)
         AndroidSupportInjection.inject(this)
 
-        val bundle = arguments
-        if (bundle != null) {
-            userId = bundle.getString(BUNDLE_USER_ID)
-            authToken = bundle.getString(BUNDLE_AUTH_TOKEN)
-        } else {
-            requireNotNull(bundle) { "no arguments supplied when the fragment was instantiated" }
-        }
+        arguments?.run {
+            userId = getString(BUNDLE_USER_ID, "")
+            authToken = getString(BUNDLE_AUTH_TOKEN, "")
+        } ?: requireNotNull(arguments) { "no arguments supplied when the fragment was instantiated" }
     }
 
     override fun onCreateView(

app/src/main/java/chat/rocket/android/authentication/server/presentation/ServerPresenter.kt

@@ -12,8 +12,8 @@ import chat.rocket.android.server.infraestructure.RocketChatClientFactory
 import chat.rocket.android.server.presentation.CheckServerPresenter
 import chat.rocket.android.util.extension.launchUI
 import chat.rocket.android.util.extensions.isValidUrl
-import kotlinx.coroutines.experimental.DefaultDispatcher
-import kotlinx.coroutines.experimental.withContext
+import kotlinx.coroutines.Dispatchers
+import kotlinx.coroutines.withContext
 import javax.inject.Inject
 
 class ServerPresenter @Inject constructor(
@@ -98,7 +98,7 @@ class ServerPresenter @Inject constructor(
                 }
                 view.showLoading()
                 try {
-                    withContext(DefaultDispatcher) {
+                    withContext(Dispatchers.Default) {
                         // preparing next fragment before showing it
                         refreshServerAccounts()
                         checkEnabledAccounts(serverUrl)

app/src/main/java/chat/rocket/android/authentication/signup/ui/SignupFragment.kt

@@ -63,7 +63,7 @@ class SignupFragment : Fragment(), SignupView {
         if (resultCode == Activity.RESULT_OK) {
             if (data != null) {
                 if (requestCode == SAVE_CREDENTIALS) {
-                    showMessage(getString(R.string.message_credentials_saved_successfully))
+                    showMessage(getString(R.string.msg_credentials_saved_successfully))
                 }
             }
         }
@@ -75,7 +75,7 @@ class SignupFragment : Fragment(), SignupView {
     }
 
     private fun setupOnClickListener() =
-        ui { _ ->
+        ui {
             button_register.setOnClickListener {
                 presenter.signup(
                     text_username.textContent,

app/src/main/java/chat/rocket/android/authentication/twofactor/ui/TwoFAFragment.kt

@@ -25,12 +25,10 @@ import io.reactivex.disposables.Disposable
 import kotlinx.android.synthetic.main.fragment_authentication_two_fa.*
 import javax.inject.Inject
 
-fun newInstance(username: String, password: String): Fragment {
-    return TwoFAFragment().apply {
-        arguments = Bundle(2).apply {
-            putString(BUNDLE_USERNAME, username)
-            putString(BUNDLE_PASSWORD, password)
-        }
+fun newInstance(username: String, password: String): Fragment = TwoFAFragment().apply {
+    arguments = Bundle(2).apply {
+        putString(BUNDLE_USERNAME, username)
+        putString(BUNDLE_PASSWORD, password)
     }
 }
 
@@ -50,13 +48,10 @@ class TwoFAFragment : Fragment(), TwoFAView {
         super.onCreate(savedInstanceState)
         AndroidSupportInjection.inject(this)
 
-        val bundle = arguments
-        if (bundle != null) {
-            username = bundle.getString(BUNDLE_USERNAME)
-            password = bundle.getString(BUNDLE_PASSWORD)
-        } else {
-            requireNotNull(bundle) { "no arguments supplied when the fragment was instantiated" }
-        }
+        arguments?.run {
+            username = getString(BUNDLE_USERNAME, "")
+            password = getString(BUNDLE_PASSWORD, "")
+        } ?: requireNotNull(arguments) { "no arguments supplied when the fragment was instantiated" }
     }
 
     override fun onCreateView(

app/src/main/java/chat/rocket/android/chatdetails/adapter/OptionViewHolder.kt

 package chat.rocket.android.chatdetails.adapter
 
-import android.content.Context
+import DrawableHelper
 import android.view.View
 import android.widget.ImageView
 import android.widget.TextView

app/src/main/java/chat/rocket/android/chatdetails/presentation/ChatDetailsPresenter.kt

@@ -7,8 +7,10 @@ import chat.rocket.android.server.domain.GetCurrentServerInteractor
 import chat.rocket.android.server.infraestructure.ConnectionManagerFactory
 import chat.rocket.android.util.extension.launchUI
 import chat.rocket.android.util.retryIO
+import chat.rocket.common.RocketChatException
 import chat.rocket.common.model.roomTypeOf
 import chat.rocket.common.util.ifNull
+import chat.rocket.core.internal.rest.favorite
 import chat.rocket.core.internal.rest.getInfo
 import chat.rocket.core.model.Room
 import timber.log.Timber
@@ -25,6 +27,31 @@ class ChatDetailsPresenter @Inject constructor(
     private val manager = factory.create(currentServer)
     private val client = manager.client
 
+    fun toggleFavoriteChatRoom(roomId: String, isFavorite: Boolean) {
+        launchUI(strategy) {
+            try {
+                // Note: If it is favorite then the user wants to remove the favorite - and vice versa.
+                retryIO("favorite($roomId, ${!isFavorite})") {
+                    client.favorite(roomId, !isFavorite)
+                }
+                view.showFavoriteIcon(!isFavorite)
+            } catch (e: RocketChatException) {
+                Timber.e(
+                    e,
+                    "Error while trying to favorite or removing the favorite of a chat room."
+                )
+                e.message?.let {
+                    view.showMessage(it)
+                }.ifNull {
+                    view.showGenericErrorMessage()
+                }
+            }
+        }
+    }
+
+    fun toVideoConference(roomId: String, chatRoomType: String) =
+        navigator.toVideoConference(roomId, chatRoomType)
+
     fun getDetails(chatRoomId: String, chatRoomType: String) {
         launchUI(strategy) {
             try {
@@ -32,7 +59,7 @@ class ChatDetailsPresenter @Inject constructor(
                     client.getInfo(chatRoomId, null, roomTypeOf(chatRoomType))
                 }
                 view.displayDetails(roomToChatDetails(room))
-            } catch(exception: Exception) {
+            } catch (exception: Exception) {
                 Timber.e(exception)
                 exception.message?.let {
                     view.showMessage(it)

app/src/main/java/chat/rocket/android/chatdetails/presentation/ChatDetailsView.kt

@@ -5,5 +5,16 @@ import chat.rocket.android.core.behaviours.LoadingView
 import chat.rocket.android.core.behaviours.MessageView
 
 interface ChatDetailsView: MessageView {
+
+    /**
+     * Shows the corresponding favorite icon for a favorite or non-favorite chat room.
+     *
+     * @param isFavorite True if a chat room is favorite, false otherwise.
+     */
+    fun showFavoriteIcon(isFavorite: Boolean)
+
+    /**
+     * Shows the details of a chat room.
+     */
     fun displayDetails(room: ChatDetails)
 }
\ No newline at end of file

app/src/main/java/chat/rocket/android/chatdetails/ui/ChatDetailsAdapter.kt

@@ -11,9 +11,15 @@ import chat.rocket.android.util.extensions.inflate
 class ChatDetailsAdapter: RecyclerView.Adapter<OptionViewHolder>() {
     private val options: MutableList<Option> = ArrayList()
 
-    override fun onCreateViewHolder(parent: ViewGroup, viewType: Int): OptionViewHolder = OptionViewHolder(parent.inflate(R.layout.item_detail_option))
+    override fun onCreateViewHolder(
+        parent: ViewGroup,
+        viewType: Int
+    ): OptionViewHolder = OptionViewHolder(parent.inflate(R.layout.item_detail_option))
 
-    override fun onBindViewHolder(holder: OptionViewHolder, position: Int) = holder.bindViews(OptionItemHolder(options[position]))
+    override fun onBindViewHolder(
+        holder: OptionViewHolder,
+        position: Int
+    ) = holder.bindViews(OptionItemHolder(options[position]))
 
     override fun getItemCount(): Int = options.size
 

app/src/main/java/chat/rocket/android/chatdetails/ui/ChatDetailsFragment.kt

@@ -3,6 +3,8 @@ package chat.rocket.android.chatdetails.ui
 import DrawableHelper
 import android.os.Bundle
 import android.view.LayoutInflater
+import android.view.Menu
+import android.view.MenuItem
 import android.view.View
 import android.view.ViewGroup
 import androidx.fragment.app.Fragment
@@ -17,6 +19,8 @@ import chat.rocket.android.chatdetails.presentation.ChatDetailsView
 import chat.rocket.android.chatdetails.viewmodel.ChatDetailsViewModel
 import chat.rocket.android.chatdetails.viewmodel.ChatDetailsViewModelFactory
 import chat.rocket.android.chatroom.ui.ChatRoomActivity
+import chat.rocket.android.server.domain.CurrentServerRepository
+import chat.rocket.android.server.domain.GetSettingsInteractor
 import chat.rocket.android.util.extensions.inflate
 import chat.rocket.android.util.extensions.showToast
 import chat.rocket.android.util.extensions.ui
@@ -31,23 +35,28 @@ fun newInstance(
     chatRoomId: String,
     chatRoomType: String,
     isSubscribed: Boolean,
+    isFavorite: Boolean,
     disableMenu: Boolean
 ): ChatDetailsFragment {
     return ChatDetailsFragment().apply {
-        arguments = Bundle(4).apply {
+        arguments = Bundle(5).apply {
             putString(BUNDLE_CHAT_ROOM_ID, chatRoomId)
             putString(BUNDLE_CHAT_ROOM_TYPE, chatRoomType)
             putBoolean(BUNDLE_IS_SUBSCRIBED, isSubscribed)
+            putBoolean(BUNDLE_IS_FAVORITE, isFavorite)
             putBoolean(BUNDLE_DISABLE_MENU, disableMenu)
         }
-    }
+   }
 }
 
 internal const val TAG_CHAT_DETAILS_FRAGMENT = "ChatDetailsFragment"
+internal const val MENU_ACTION_FAVORITE_REMOVE_FAVORITE = 1
+internal const val MENU_ACTION_VIDEO_CALL = 2
 
 private const val BUNDLE_CHAT_ROOM_ID = "BUNDLE_CHAT_ROOM_ID"
 private const val BUNDLE_CHAT_ROOM_TYPE = "BUNDLE_CHAT_ROOM_TYPE"
 private const val BUNDLE_IS_SUBSCRIBED = "BUNDLE_IS_SUBSCRIBED"
+private const val BUNDLE_IS_FAVORITE = "BUNDLE_IS_FAVORITE"
 private const val BUNDLE_DISABLE_MENU = "BUNDLE_DISABLE_MENU"
 
 class ChatDetailsFragment : Fragment(), ChatDetailsView {
@@ -55,26 +64,32 @@ class ChatDetailsFragment : Fragment(), ChatDetailsView {
     lateinit var presenter: ChatDetailsPresenter
     @Inject
     lateinit var factory: ChatDetailsViewModelFactory
+    @Inject
+    lateinit var serverUrl: CurrentServerRepository
+    @Inject
+    lateinit var settings: GetSettingsInteractor
     private var adapter: ChatDetailsAdapter? = null
     private lateinit var viewModel: ChatDetailsViewModel
 
-    private var chatRoomId: String? = null
-    private var chatRoomType: String? = null
+    internal lateinit var chatRoomId: String
+    internal lateinit var chatRoomType: String
     private var isSubscribed: Boolean = true
+    internal var isFavorite: Boolean = false
     private var disableMenu: Boolean = false
 
     override fun onCreate(savedInstanceState: Bundle?) {
         super.onCreate(savedInstanceState)
         AndroidSupportInjection.inject(this)
-        val bundle = arguments
-        if (bundle != null) {
-            chatRoomId = bundle.getString(BUNDLE_CHAT_ROOM_ID)
-            chatRoomType = bundle.getString(BUNDLE_CHAT_ROOM_TYPE)
-            isSubscribed = bundle.getBoolean(BUNDLE_IS_SUBSCRIBED)
-            disableMenu = bundle.getBoolean(BUNDLE_DISABLE_MENU)
-        } else {
-            requireNotNull(bundle) { "no arguments supplied when the fragment was instantiated" }
-        }
+
+        arguments?.run {
+            chatRoomId = getString(BUNDLE_CHAT_ROOM_ID)
+            chatRoomType = getString(BUNDLE_CHAT_ROOM_TYPE)
+            isSubscribed = getBoolean(BUNDLE_IS_SUBSCRIBED)
+            isFavorite = getBoolean(BUNDLE_IS_FAVORITE)
+            disableMenu = getBoolean(BUNDLE_DISABLE_MENU)
+        } ?: requireNotNull(arguments) { "no arguments supplied when the fragment was instantiated" }
+
+        setHasOptionsMenu(true)
     }
 
     override fun onCreateView(
@@ -91,11 +106,27 @@ class ChatDetailsFragment : Fragment(), ChatDetailsView {
         getDetails()
     }
 
+    override fun onPrepareOptionsMenu(menu: Menu) {
+        menu.clear()
+        setupMenu(menu)
+        super.onPrepareOptionsMenu(menu)
+    }
+
+    override fun onOptionsItemSelected(item: MenuItem): Boolean {
+        setOnMenuItemClickListener(item)
+        return true
+    }
+
+    override fun showFavoriteIcon(isFavorite: Boolean) {
+        this.isFavorite = isFavorite
+        activity?.invalidateOptionsMenu()
+    }
+
     override fun displayDetails(room: ChatDetails) {
         ui {
             val text = room.name
             name.text = text
-            bindImage(chatRoomType!!)
+            bindImage(chatRoomType)
             content_topic.text =
                     if (room.topic.isNullOrEmpty()) getString(R.string.msg_no_topic) else room.topic
             content_announcement.text =
@@ -207,8 +238,8 @@ class ChatDetailsFragment : Fragment(), ChatDetailsView {
 
     private fun setupToolbar() {
         with((activity as ChatRoomActivity)) {
-            hideToolbarChatRoomIcon()
-            showToolbarTitle(getString(R.string.title_channel_details))
+            hideExpandMoreForToolbar()
+            setupToolbarTitle(getString(R.string.title_channel_details))
         }
     }
 }
\ No newline at end of file

app/src/main/java/chat/rocket/android/chatdetails/ui/Menu.kt

+package chat.rocket.android.chatdetails.ui
+
+import android.view.Menu
+import android.view.MenuItem
+import chat.rocket.android.R
+import chat.rocket.android.server.domain.isJitsiEnabled
+import chat.rocket.android.server.domain.isJitsiEnabledForChannels
+import chat.rocket.common.model.RoomType
+import chat.rocket.common.model.roomTypeOf
+
+internal fun ChatDetailsFragment.setupMenu(menu: Menu) {
+    serverUrl.get()?.let {
+        with(settings.get(it)) {
+            if (isJitsiEnabled()) {
+                if (roomTypeOf(chatRoomType) !is RoomType.DirectMessage && !isJitsiEnabledForChannels()) {
+                    return
+                }
+                menu.add(
+                    Menu.NONE,
+                    MENU_ACTION_VIDEO_CALL,
+                    Menu.NONE,
+                    R.string.msg_video_call
+                ).setIcon(R.drawable.ic_video_24dp).setShowAsAction(MenuItem.SHOW_AS_ACTION_IF_ROOM)
+            }
+        }
+    }
+
+    if (isFavorite) {
+        menu.add(
+            Menu.NONE,
+            MENU_ACTION_FAVORITE_REMOVE_FAVORITE,
+            Menu.NONE,
+            R.string.action_remove_favorite
+        ).setIcon(R.drawable.ic_star_yellow_24dp).setShowAsAction(MenuItem.SHOW_AS_ACTION_IF_ROOM)
+    } else {
+        menu.add(
+            Menu.NONE,
+            MENU_ACTION_FAVORITE_REMOVE_FAVORITE,
+            Menu.NONE,
+            R.string.action_favorite
+        ).setIcon(R.drawable.ic_star_border_white_24dp)
+            .setShowAsAction(MenuItem.SHOW_AS_ACTION_IF_ROOM)
+    }
+}
+
+internal fun ChatDetailsFragment.setOnMenuItemClickListener(item: MenuItem) {
+    if (item.itemId == MENU_ACTION_FAVORITE_REMOVE_FAVORITE) {
+        presenter.toggleFavoriteChatRoom(chatRoomId, isFavorite)
+    } else if (item.itemId == MENU_ACTION_VIDEO_CALL) {
+        presenter.toVideoConference(chatRoomId, chatRoomType)
+    }
+}

app/src/main/java/chat/rocket/android/chatdetails/viewmodel/ChatDetailsViewModelFactory.kt

@@ -5,9 +5,10 @@ import androidx.lifecycle.ViewModelProvider
 import chat.rocket.android.db.ChatRoomDao
 import javax.inject.Inject
 
-class ChatDetailsViewModelFactory @Inject constructor(private val chatRoomDao: ChatRoomDao) : ViewModelProvider.NewInstanceFactory() {
+class ChatDetailsViewModelFactory @Inject constructor(
+    private val chatRoomDao: ChatRoomDao
+) : ViewModelProvider.NewInstanceFactory() {
 
     @Suppress("UNCHECKED_CAST")
-    override fun <T : ViewModel?> create(modelClass: Class<T>) =
-            ChatDetailsViewModel(chatRoomDao) as T
+    override fun <T : ViewModel?> create(modelClass: Class<T>) = ChatDetailsViewModel(chatRoomDao) as T
 }
\ No newline at end of file

app/src/main/java/chat/rocket/android/chatinformation/di/MessageInfoFragmentModule.kt

@@ -7,7 +7,7 @@ import chat.rocket.android.core.lifecycle.CancelStrategy
 import chat.rocket.android.dagger.scope.PerFragment
 import dagger.Module
 import dagger.Provides
-import kotlinx.coroutines.experimental.Job
+import kotlinx.coroutines.Job
 
 @Module
 class MessageInfoFragmentModule {

app/src/main/java/chat/rocket/android/chatinformation/ui/MessageInfoActivity.kt

@@ -24,7 +24,6 @@ fun Context.messageInformationIntent(messageId: String): Intent {
 private const val INTENT_MESSAGE_ID = "message_id"
 
 class MessageInfoActivity : AppCompatActivity(), HasSupportFragmentInjector {
-
     @Inject
     lateinit var fragmentDispatchingAndroidInjector: DispatchingAndroidInjector<Fragment>
 
@@ -45,7 +44,7 @@ class MessageInfoActivity : AppCompatActivity(), HasSupportFragmentInjector {
     }
 
     private fun setupToolbar() {
-        text_room_name.textContent = getString(R.string.message_information_title)
+        text_toolbar_title.textContent = getString(R.string.message_information_title)
         setSupportActionBar(toolbar)
         supportActionBar?.setDisplayShowTitleEnabled(false)
         toolbar.setNavigationIcon(R.drawable.ic_arrow_back_white_24dp)

app/src/main/java/chat/rocket/android/chatroom/adapter/ActionsListAdapter.kt

@@ -12,7 +12,10 @@ import com.facebook.drawee.backends.pipeline.Fresco
 import kotlinx.android.synthetic.main.item_action_button.view.*
 import timber.log.Timber
 
-class ActionsListAdapter(actions: List<Action>, var actionAttachmentOnClickListener: ActionAttachmentOnClickListener) : RecyclerView.Adapter<ActionsListAdapter.ViewHolder>() {
+class ActionsListAdapter(
+    actions: List<Action>,
+    var actionAttachmentOnClickListener: ActionAttachmentOnClickListener
+) : RecyclerView.Adapter<ActionsListAdapter.ViewHolder>() {
 
     var actions: List<Action> = actions
 
@@ -62,9 +65,7 @@ class ActionsListAdapter(actions: List<Action>, var actionAttachmentOnClickListe
         return ViewHolder(view)
     }
 
-    override fun getItemCount(): Int {
-        return actions.size
-    }
+    override fun getItemCount(): Int = actions.size
 
     override fun onBindViewHolder(holder: ViewHolder, position: Int) {
         val action = actions[position]

app/src/main/java/chat/rocket/android/chatroom/adapter/ChatRoomAdapter.kt

@@ -44,8 +44,14 @@ class ChatRoomAdapter(
                 MessageViewHolder(
                     view,
                     actionsListener,
-                    reactionListener
-                ) { userId -> navigator?.toUserDetails(userId) }
+                    reactionListener,
+                    { userId -> navigator?.toUserDetails(userId) },
+                    {
+                        if (roomId != null && roomType != null) {
+                            navigator?.toVideoConference(roomId, roomType)
+                        }
+                    }
+                )
             }
             BaseUiModel.ViewType.URL_PREVIEW -> {
                 val view = parent.inflate(R.layout.message_url_preview)
@@ -76,13 +82,9 @@ class ChatRoomAdapter(
         }
     }
 
-    override fun getItemViewType(position: Int): Int {
-        return dataSet[position].viewType
-    }
+    override fun getItemViewType(position: Int): Int = dataSet[position].viewType
 
-    override fun getItemCount(): Int {
-        return dataSet.size
-    }
+    override fun getItemCount(): Int = dataSet.size
 
     override fun onBindViewHolder(holder: BaseViewHolder<*>, position: Int) {
         if (holder !is MessageViewHolder) {
@@ -105,8 +107,9 @@ class ChatRoomAdapter(
         when (holder) {
             is MessageViewHolder ->
                 holder.bind(dataSet[position] as MessageUiModel)
-            is UrlPreviewViewHolder ->
+            is UrlPreviewViewHolder -> {
                 holder.bind(dataSet[position] as UrlPreviewUiModel)
+            }
             is MessageReplyViewHolder ->
                 holder.bind(dataSet[position] as MessageReplyUiModel)
             is AttachmentViewHolder ->
@@ -174,12 +177,12 @@ class ChatRoomAdapter(
         Timber.d("index: $index")
         if (index > -1) {
             dataSet[index] = message
-            dataSet.forEachIndexed { index, viewModel ->
+            dataSet.forEachIndexed { ind, viewModel ->
                 if (viewModel.messageId == message.messageId) {
                     if (viewModel.nextDownStreamMessage == null) {
                         viewModel.reactions = message.reactions
                     }
-                    notifyItemChanged(index)
+                    notifyItemChanged(ind)
                 }
             }
             // Delete message only if current is a system message update, i.e.: Message Removed
@@ -258,10 +261,10 @@ class ChatRoomAdapter(
                         actionSelectListener?.editMessage(roomId, id, message.message)
                     }
                     R.id.action_message_star -> {
-                        actionSelectListener?.toogleStar(id, !item.isChecked)
+                        actionSelectListener?.toggleStar(id, !item.isChecked)
                     }
                     R.id.action_message_unpin -> {
-                        actionSelectListener?.tooglePin(id, !item.isChecked)
+                        actionSelectListener?.togglePin(id, !item.isChecked)
                     }
                     R.id.action_message_delete -> {
                         actionSelectListener?.deleteMessage(roomId, id)
@@ -298,9 +301,9 @@ class ChatRoomAdapter(
 
         fun editMessage(roomId: String, messageId: String, text: String)
 
-        fun toogleStar(id: String, star: Boolean)
+        fun toggleStar(id: String, star: Boolean)
 
-        fun tooglePin(id: String, pin: Boolean)
+        fun togglePin(id: String, pin: Boolean)
 
         fun deleteMessage(roomId: String, id: String)
 

app/src/main/java/chat/rocket/android/chatroom/adapter/MessageReactionsAdapter.kt

@@ -13,8 +13,8 @@ import chat.rocket.android.emoji.Emoji
 import chat.rocket.android.emoji.EmojiKeyboardListener
 import chat.rocket.android.emoji.EmojiPickerPopup
 import chat.rocket.android.emoji.EmojiReactionListener
-import chat.rocket.android.emoji.internal.GlideApp
 import chat.rocket.android.infrastructure.LocalRepository
+import com.bumptech.glide.Glide
 import kotlinx.android.synthetic.main.item_reaction.view.*
 import java.util.concurrent.CopyOnWriteArrayList
 import javax.inject.Inject
@@ -103,9 +103,9 @@ class MessageReactionsAdapter : RecyclerView.Adapter<RecyclerView.ViewHolder>()
                     // The view at index 1 corresponds to the one to display custom emojis which are images.
                     view_flipper_reaction.displayedChild = 1
                     val glideRequest = if (reaction.url!!.endsWith("gif", true)) {
-                        GlideApp.with(context).asGif()
+                        Glide.with(context).asGif()
                     } else {
-                        GlideApp.with(context).asBitmap()
+                        Glide.with(context).asBitmap()
                     }
 
                     glideRequest.load(reaction.url).into(image_emoji)

app/src/main/java/chat/rocket/android/chatroom/adapter/MessageViewHolder.kt

@@ -10,6 +10,7 @@ import androidx.core.view.isVisible
 import chat.rocket.android.R
 import chat.rocket.android.chatroom.uimodel.MessageUiModel
 import chat.rocket.android.emoji.EmojiReactionListener
+import chat.rocket.core.model.MessageType
 import chat.rocket.core.model.isSystemMessage
 import com.bumptech.glide.load.resource.gif.GifDrawable
 import kotlinx.android.synthetic.main.avatar.view.*
@@ -19,7 +20,8 @@ class MessageViewHolder(
     itemView: View,
     listener: ActionsListener,
     reactionListener: EmojiReactionListener? = null,
-    private val avatarListener: (String) -> Unit
+    private val avatarListener: (String) -> Unit,
+    private val joinVideoCallListener: (View) -> Unit
 ) : BaseViewHolder<MessageUiModel>(itemView, listener, reactionListener), Drawable.Callback {
 
     init {
@@ -51,6 +53,9 @@ class MessageViewHolder(
 
             text_content.text_content.text = data.content
 
+            button_join_video_call.isVisible = data.message.type is MessageType.JitsiCallStarted
+            button_join_video_call.setOnClickListener { joinVideoCallListener(it) }
+
             image_avatar.setImageURI(data.avatar)
             text_content.setTextColor(if (data.isTemporary) Color.GRAY else Color.BLACK)
 

app/src/main/java/chat/rocket/android/chatroom/adapter/RoomSuggestionsAdapter.kt

@@ -24,8 +24,8 @@ class RoomSuggestionsAdapter : SuggestionsAdapter<RoomSuggestionsViewHolder>("#"
         override fun bind(item: SuggestionModel, itemClickListener: SuggestionsAdapter.ItemClickListener?) {
             item as ChatRoomSuggestionUiModel
             with(itemView) {
-                val fullname = itemView.findViewById<TextView>(R.id.text_fullname)
-                val name = itemView.findViewById<TextView>(R.id.text_name)
+                val fullname = findViewById<TextView>(R.id.text_fullname)
+                val name = findViewById<TextView>(R.id.text_name)
                 name.text = item.name
                 fullname.text = item.fullName
                 setOnClickListener {

app/src/main/java/chat/rocket/android/chatroom/adapter/UrlPreviewViewHolder.kt

@@ -8,15 +8,14 @@ import chat.rocket.android.util.extensions.content
 import chat.rocket.android.util.extensions.openTabbedUrl
 import kotlinx.android.synthetic.main.message_url_preview.view.*
 
-class UrlPreviewViewHolder(itemView: View,
-                           listener: ActionsListener,
-                           reactionListener: EmojiReactionListener? = null)
-    : BaseViewHolder<UrlPreviewUiModel>(itemView, listener, reactionListener) {
+class UrlPreviewViewHolder(
+    itemView: View,
+    listener: ActionsListener,
+    reactionListener: EmojiReactionListener? = null
+) : BaseViewHolder<UrlPreviewUiModel>(itemView, listener, reactionListener) {
 
     init {
-        with(itemView) {
-            setupActionMenu(url_preview_layout)
-        }
+        setupActionMenu(itemView.url_preview_layout)
     }
 
     override fun bindViews(data: UrlPreviewUiModel) {

app/src/main/java/chat/rocket/android/chatroom/di/ChatRoomModule.kt

@@ -7,7 +7,7 @@ import chat.rocket.android.core.lifecycle.CancelStrategy
 import chat.rocket.android.dagger.scope.PerActivity
 import dagger.Module
 import dagger.Provides
-import kotlinx.coroutines.experimental.Job
+import kotlinx.coroutines.Job
 
 @Module
 class ChatRoomModule {

app/src/main/java/chat/rocket/android/chatroom/presentation/ChatRoomNavigator.kt

 package chat.rocket.android.chatroom.presentation
 
+import android.os.Build
+import android.widget.Toast
 import chat.rocket.android.R
 import chat.rocket.android.chatdetails.ui.TAG_CHAT_DETAILS_FRAGMENT
 import chat.rocket.android.chatinformation.ui.messageInformationIntent
@@ -13,6 +15,7 @@ import chat.rocket.android.pinnedmessages.ui.TAG_PINNED_MESSAGES_FRAGMENT
 import chat.rocket.android.server.ui.changeServerIntent
 import chat.rocket.android.userdetails.ui.TAG_USER_DETAILS_FRAGMENT
 import chat.rocket.android.util.extensions.addFragmentBackStack
+import chat.rocket.android.videoconference.ui.videoConferenceIntent
 
 class ChatRoomNavigator(internal val activity: ChatRoomActivity) {
 
@@ -22,6 +25,19 @@ class ChatRoomNavigator(internal val activity: ChatRoomActivity) {
         }
     }
 
+    fun toVideoConference(chatRoomId: String, chatRoomType: String) {
+        // TODO: Jitsi isn't working with Android M- version. We need to remove the condition bellow after it's solved. (https://github.com/jitsi/jitsi-meet/pull/3967)/
+        if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.M) {
+            activity.startActivity(activity.videoConferenceIntent(chatRoomId, chatRoomType))
+        } else {
+            Toast.makeText(
+                activity,
+                "Sorry, unable to open the video conference due to device configuration",
+                Toast.LENGTH_SHORT
+            ).show()
+        }
+    }
+
     fun toChatRoom(
         chatRoomId: String,
         chatRoomName: String,
@@ -51,6 +67,7 @@ class ChatRoomNavigator(internal val activity: ChatRoomActivity) {
         chatRoomId: String,
         chatRoomType: String,
         isChatRoomSubscribed: Boolean,
+        isChatRoomFavorite: Boolean,
         isMenuDisabled: Boolean
     ) {
         activity.addFragmentBackStack(TAG_CHAT_DETAILS_FRAGMENT, R.id.fragment_container) {
@@ -58,6 +75,7 @@ class ChatRoomNavigator(internal val activity: ChatRoomActivity) {
                 chatRoomId,
                 chatRoomType,
                 isChatRoomSubscribed,
+                isChatRoomFavorite,
                 isMenuDisabled
             )
         }

app/src/main/java/chat/rocket/android/chatroom/presentation/ChatRoomPresenter.kt

@@ -53,7 +53,6 @@ import chat.rocket.core.internal.realtime.unsubscribe
 import chat.rocket.core.internal.rest.chatRoomRoles
 import chat.rocket.core.internal.rest.commands
 import chat.rocket.core.internal.rest.deleteMessage
-import chat.rocket.core.internal.rest.favorite
 import chat.rocket.core.internal.rest.getMembers
 import chat.rocket.core.internal.rest.history
 import chat.rocket.core.internal.rest.joinChat
@@ -76,12 +75,11 @@ import chat.rocket.core.model.ChatRoomRole
 import chat.rocket.core.model.Command
 import chat.rocket.core.model.Message
 import chat.rocket.core.model.Room
-import kotlinx.coroutines.experimental.CommonPool
-import kotlinx.coroutines.experimental.DefaultDispatcher
-import kotlinx.coroutines.experimental.android.UI
-import kotlinx.coroutines.experimental.channels.Channel
-import kotlinx.coroutines.experimental.launch
-import kotlinx.coroutines.experimental.withContext
+import kotlinx.coroutines.Dispatchers
+import kotlinx.coroutines.GlobalScope
+import kotlinx.coroutines.channels.Channel
+import kotlinx.coroutines.launch
+import kotlinx.coroutines.withContext
 import org.threeten.bp.Instant
 import timber.log.Timber
 import java.util.*
@@ -134,7 +132,7 @@ class ChatRoomPresenter @Inject constructor(
         draftKey = "${currentServer}_${LocalRepository.DRAFT_KEY}$roomId"
         chatRoomId = roomId
         chatRoomType = roomType
-        launch(CommonPool + strategy.jobs) {
+        GlobalScope.launch(Dispatchers.IO + strategy.jobs) {
             try {
                 chatRoles = if (roomTypeOf(roomType) !is RoomType.DirectMessage) {
                     client.chatRoomRoles(roomType = roomTypeOf(roomType), roomName = roomName)
@@ -178,12 +176,12 @@ class ChatRoomPresenter @Inject constructor(
     }
 
     private suspend fun subscribeRoomChanges() {
-        withContext(CommonPool + strategy.jobs) {
+        withContext(Dispatchers.IO + strategy.jobs) {
             chatRoomId?.let {
                 manager.addRoomChannel(it, roomChangesChannel)
                 for (room in roomChangesChannel) {
-                    dbManager.getRoom(room.id)?.let {
-                        view.onRoomUpdated(roomMapper.map(chatRoom = it, showLastMessage = true))
+                    dbManager.getRoom(room.id)?.let { chatRoom ->
+                        view.onRoomUpdated(roomMapper.map(chatRoom = chatRoom, showLastMessage = true))
                     }
                 }
             }
@@ -405,7 +403,7 @@ class ChatRoomPresenter @Inject constructor(
         launchUI(strategy) {
             view.showLoading()
             try {
-                withContext(DefaultDispatcher) {
+                withContext(Dispatchers.Default) {
                     val fileName = uriInteractor.getFileName(uri) ?: uri.toString()
                     if (fileName.isEmpty()) {
                         view.showInvalidFileMessage()
@@ -443,7 +441,7 @@ class ChatRoomPresenter @Inject constructor(
         launchUI(strategy) {
             view.showLoading()
             try {
-                withContext(DefaultDispatcher) {
+                withContext(Dispatchers.Default) {
                     val fileName = uriInteractor.getFileName(uri) ?: uri.toString()
                     val fileSize = uriInteractor.getFileSize(uri)
                     val maxFileSizeAllowed = settings.uploadMaxFileSize()
@@ -484,7 +482,7 @@ class ChatRoomPresenter @Inject constructor(
         launchUI(strategy) {
             view.showLoading()
             try {
-                withContext(DefaultDispatcher) {
+                withContext(Dispatchers.Default) {
                     val fileName = UUID.randomUUID().toString() + ".png"
                     val fileSize = byteArray.size
                     val mimeType = "image/png"
@@ -522,7 +520,7 @@ class ChatRoomPresenter @Inject constructor(
     }
 
     fun sendTyping() {
-        launch(CommonPool + strategy.jobs) {
+        GlobalScope.launch(Dispatchers.IO + strategy.jobs) {
             if (chatRoomId != null && currentLoggedUsername != null) {
                 client.setTypingStatus(chatRoomId.toString(), currentLoggedUsername, true)
             }
@@ -530,7 +528,7 @@ class ChatRoomPresenter @Inject constructor(
     }
 
     fun sendNotTyping() {
-        launch(CommonPool + strategy.jobs) {
+        GlobalScope.launch(Dispatchers.IO + strategy.jobs) {
             if (chatRoomId != null && currentLoggedUsername != null) {
                 client.setTypingStatus(chatRoomId.toString(), currentLoggedUsername, false)
             }
@@ -552,11 +550,11 @@ class ChatRoomPresenter @Inject constructor(
         Timber.d("Subscribing to Status changes")
         lastState = manager.state
         manager.addStatusChannel(stateChannel)
-        launch(CommonPool + strategy.jobs) {
+        GlobalScope.launch(Dispatchers.IO + strategy.jobs) {
             for (state in stateChannel) {
                 Timber.d("Got new state: $state - last: $lastState")
                 if (state != lastState) {
-                    launch(UI) {
+                    launch(Dispatchers.Main) {
                         view.showConnectionState(state)
                     }
 
@@ -573,7 +571,7 @@ class ChatRoomPresenter @Inject constructor(
     private fun subscribeMessages(roomId: String) {
         manager.subscribeRoomMessages(roomId, messagesChannel)
 
-        launch(CommonPool + strategy.jobs) {
+        GlobalScope.launch(Dispatchers.IO + strategy.jobs) {
             for (message in messagesChannel) {
                 Timber.d("New message for room ${message.roomId}")
                 updateMessage(message)
@@ -582,7 +580,7 @@ class ChatRoomPresenter @Inject constructor(
     }
 
     private fun loadMissingMessages() {
-        launch(parent = strategy.jobs) {
+        GlobalScope.launch(strategy.jobs) {
             chatRoomId?.let { chatRoomId ->
                 val roomType = roomTypeOf(chatRoomType)
                 val lastSyncDate = messagesRepository.getLastSyncDate(chatRoomId)
@@ -891,32 +889,14 @@ class ChatRoomPresenter @Inject constructor(
         }
     }
 
-    fun toggleFavoriteChatRoom(roomId: String, isFavorite: Boolean) {
-        launchUI(strategy) {
-            try {
-                // Note that if it is favorite then the user wants to unfavorite - and vice versa.
-                retryIO("favorite($roomId, $isFavorite)") {
-                    client.favorite(roomId, !isFavorite)
-                }
-                view.showFavoriteIcon(!isFavorite)
-            } catch (e: RocketChatException) {
-                Timber.e(e, "Error while trying to favorite/unfavorite chat room.")
-                e.message?.let {
-                    view.showMessage(it)
-                }.ifNull {
-                    view.showGenericErrorMessage()
-                }
-            }
-        }
-    }
-
     fun toChatDetails(
         chatRoomId: String,
         chatRoomType: String,
         isSubscribed: Boolean,
+        isFavorite: Boolean,
         isMenuDisabled: Boolean
     ) {
-        navigator.toChatDetails(chatRoomId, chatRoomType, isSubscribed, isMenuDisabled)
+        navigator.toChatDetails(chatRoomId, chatRoomType, isSubscribed, isFavorite, isMenuDisabled)
     }
 
     fun loadChatRoomsSuggestions() {
@@ -944,7 +924,7 @@ class ChatRoomPresenter @Inject constructor(
     }
 
     // TODO: move this to new interactor or FetchChatRoomsInteractor?
-    private suspend fun getChatRoomAsync(roomId: String): ChatRoom? = withContext(CommonPool) {
+    private suspend fun getChatRoomAsync(roomId: String): ChatRoom? = withContext(Dispatchers.IO) {
         retryDB("getRoom($roomId)") {
             dbManager.chatRoomDao().getSync(roomId)?.let {
                 with(it.chatRoom) {
@@ -982,7 +962,7 @@ class ChatRoomPresenter @Inject constructor(
     }
 
     // TODO: move this to new interactor or FetchChatRoomsInteractor?
-    private suspend fun getChatRoomsAsync(name: String? = null): List<ChatRoom> = withContext(CommonPool) {
+    private suspend fun getChatRoomsAsync(name: String? = null): List<ChatRoom> = withContext(Dispatchers.IO) {
         retryDB("getAllSync()") {
             dbManager.chatRoomDao().getAllSync().filter {
                 if (name == null) {
@@ -1074,7 +1054,7 @@ class ChatRoomPresenter @Inject constructor(
         launchUI(strategy) {
             try {
                 messagesRepository.getById(messageId)?.let { message ->
-                    getChatRoomAsync(message.roomId)?.let { chatRoom ->
+                    getChatRoomAsync(message.roomId)?.let {
                         val models = mapper.map(message)
                         models.firstOrNull()?.permalink?.let {
                             view.copyToClipboard(it)
@@ -1212,7 +1192,7 @@ class ChatRoomPresenter @Inject constructor(
     }
 
     private fun subscribeTypingStatus() {
-        launch(CommonPool + strategy.jobs) {
+        GlobalScope.launch(Dispatchers.IO + strategy.jobs) {
             client.subscribeTypingStatus(chatRoomId.toString()) { _, id ->
                 typingStatusSubscriptionId = id
             }

app/src/main/java/chat/rocket/android/chatroom/presentation/ChatRoomView.kt

@@ -13,13 +13,6 @@ import chat.rocket.core.model.ChatRoom
 
 interface ChatRoomView : LoadingView, MessageView {
 
-    /**
-     * Shows the Favorite/Unfavorite chat room icon.
-     *
-     * @param isFavorite Shows the favorite icon if true, otherwise shows the unfavorite icon.
-     */
-    fun showFavoriteIcon(isFavorite: Boolean)
-
     /**
      * Shows the chat room messages.
      *

app/src/main/java/chat/rocket/android/chatroom/service/MessageService.kt

@@ -10,8 +10,9 @@ import chat.rocket.android.server.infraestructure.DatabaseMessagesRepository
 import chat.rocket.core.internal.rest.sendMessage
 import chat.rocket.core.model.Message
 import dagger.android.AndroidInjection
-import kotlinx.coroutines.experimental.CommonPool
-import kotlinx.coroutines.experimental.launch
+import kotlinx.coroutines.Dispatchers
+import kotlinx.coroutines.GlobalScope
+import kotlinx.coroutines.launch
 import timber.log.Timber
 import javax.inject.Inject
 
@@ -33,7 +34,7 @@ class MessageService : JobService() {
     }
 
     override fun onStartJob(params: JobParameters?): Boolean {
-        launch(CommonPool) {
+        GlobalScope.launch(Dispatchers.IO) {
             getAccountsInteractor.get().forEach { account ->
                 retrySendingMessages(params, account.serverUrl)
             }
@@ -44,7 +45,8 @@ class MessageService : JobService() {
 
     private suspend fun retrySendingMessages(params: JobParameters?, serverUrl: String) {
         val dbManager = dbFactory.create(serverUrl)
-        val messageRepository = DatabaseMessagesRepository(dbManager, DatabaseMessageMapper(dbManager))
+        val messageRepository =
+            DatabaseMessagesRepository(dbManager, DatabaseMessageMapper(dbManager))
         val temporaryMessages = messageRepository.getAllUnsent()
             .sortedWith(compareBy(Message::timestamp))
         if (temporaryMessages.isNotEmpty()) {

app/src/main/java/chat/rocket/android/chatroom/ui/ChatRoomActivity.kt

@@ -4,16 +4,15 @@ import DrawableHelper
 import android.content.Context
 import android.content.Intent
 import android.os.Bundle
-import androidx.fragment.app.Fragment
+import android.view.View
 import androidx.appcompat.app.AppCompatActivity
+import androidx.fragment.app.Fragment
 import chat.rocket.android.R
 import chat.rocket.android.chatroom.presentation.ChatRoomNavigator
 import chat.rocket.android.server.domain.GetCurrentServerInteractor
 import chat.rocket.android.server.infraestructure.ConnectionManagerFactory
 import chat.rocket.android.util.extensions.addFragment
 import chat.rocket.android.util.extensions.textContent
-import chat.rocket.common.model.RoomType
-import chat.rocket.common.model.roomTypeOf
 import dagger.android.AndroidInjection
 import dagger.android.AndroidInjector
 import dagger.android.DispatchingAndroidInjector
@@ -31,18 +30,16 @@ fun Context.chatRoomIntent(
     isCreator: Boolean = false,
     isFavorite: Boolean = false,
     chatRoomMessage: String? = null
-): Intent {
-    return Intent(this, ChatRoomActivity::class.java).apply {
-        putExtra(INTENT_CHAT_ROOM_ID, chatRoomId)
-        putExtra(INTENT_CHAT_ROOM_NAME, chatRoomName)
-        putExtra(INTENT_CHAT_ROOM_TYPE, chatRoomType)
-        putExtra(INTENT_CHAT_ROOM_IS_READ_ONLY, isReadOnly)
-        putExtra(INTENT_CHAT_ROOM_LAST_SEEN, chatRoomLastSeen)
-        putExtra(INTENT_CHAT_IS_SUBSCRIBED, isSubscribed)
-        putExtra(INTENT_CHAT_ROOM_IS_CREATOR, isCreator)
-        putExtra(INTENT_CHAT_ROOM_IS_FAVORITE, isFavorite)
-        putExtra(INTENT_CHAT_ROOM_MESSAGE, chatRoomMessage)
-    }
+): Intent = Intent(this, ChatRoomActivity::class.java).apply {
+    putExtra(INTENT_CHAT_ROOM_ID, chatRoomId)
+    putExtra(INTENT_CHAT_ROOM_NAME, chatRoomName)
+    putExtra(INTENT_CHAT_ROOM_TYPE, chatRoomType)
+    putExtra(INTENT_CHAT_ROOM_IS_READ_ONLY, isReadOnly)
+    putExtra(INTENT_CHAT_ROOM_LAST_SEEN, chatRoomLastSeen)
+    putExtra(INTENT_CHAT_IS_SUBSCRIBED, isSubscribed)
+    putExtra(INTENT_CHAT_ROOM_IS_CREATOR, isCreator)
+    putExtra(INTENT_CHAT_ROOM_IS_FAVORITE, isFavorite)
+    putExtra(INTENT_CHAT_ROOM_MESSAGE, chatRoomMessage)
 }
 
 private const val INTENT_CHAT_ROOM_ID = "chat_room_id"
@@ -58,7 +55,6 @@ private const val INTENT_CHAT_ROOM_MESSAGE = "chat_room_message"
 class ChatRoomActivity : AppCompatActivity(), HasSupportFragmentInjector {
     @Inject
     lateinit var fragmentDispatchingAndroidInjector: DispatchingAndroidInjector<Fragment>
-
     // TODO - workaround for now... We will move to a single activity
     @Inject
     lateinit var serverInteractor: GetCurrentServerInteractor
@@ -81,42 +77,44 @@ class ChatRoomActivity : AppCompatActivity(), HasSupportFragmentInjector {
             return
         }
 
-        val chatRoomId = intent.getStringExtra(INTENT_CHAT_ROOM_ID)
-        requireNotNull(chatRoomId) { "no chat_room_id provided in Intent extras" }
+        with(intent) {
+            val chatRoomId = getStringExtra(INTENT_CHAT_ROOM_ID)
+            requireNotNull(chatRoomId) { "no chat_room_id provided in Intent extras" }
 
-        val chatRoomName = intent.getStringExtra(INTENT_CHAT_ROOM_NAME)
-        requireNotNull(chatRoomName) { "no chat_room_name provided in Intent extras" }
+            val chatRoomName = getStringExtra(INTENT_CHAT_ROOM_NAME)
+            requireNotNull(chatRoomName) { "no chat_room_name provided in Intent extras" }
 
-        val chatRoomType = intent.getStringExtra(INTENT_CHAT_ROOM_TYPE)
-        requireNotNull(chatRoomType) { "no chat_room_type provided in Intent extras" }
+            val chatRoomType = getStringExtra(INTENT_CHAT_ROOM_TYPE)
+            requireNotNull(chatRoomType) { "no chat_room_type provided in Intent extras" }
 
-        val isReadOnly = intent.getBooleanExtra(INTENT_CHAT_ROOM_IS_READ_ONLY, true)
+            val isReadOnly = getBooleanExtra(INTENT_CHAT_ROOM_IS_READ_ONLY, true)
 
-        val isCreator = intent.getBooleanExtra(INTENT_CHAT_ROOM_IS_CREATOR, false)
+            val isCreator = getBooleanExtra(INTENT_CHAT_ROOM_IS_CREATOR, false)
 
-        val isFavorite = intent.getBooleanExtra(INTENT_CHAT_ROOM_IS_FAVORITE, false)
+            val isFavorite = getBooleanExtra(INTENT_CHAT_ROOM_IS_FAVORITE, false)
 
-        val chatRoomLastSeen = intent.getLongExtra(INTENT_CHAT_ROOM_LAST_SEEN, -1)
+            val chatRoomLastSeen = getLongExtra(INTENT_CHAT_ROOM_LAST_SEEN, -1)
 
-        val isSubscribed = intent.getBooleanExtra(INTENT_CHAT_IS_SUBSCRIBED, true)
+            val isSubscribed = getBooleanExtra(INTENT_CHAT_IS_SUBSCRIBED, true)
 
-        val chatRoomMessage = intent.getStringExtra(INTENT_CHAT_ROOM_MESSAGE)
+            val chatRoomMessage = getStringExtra(INTENT_CHAT_ROOM_MESSAGE)
 
-        setupToolbar()
+            setupToolbar()
 
-        if (supportFragmentManager.findFragmentByTag(TAG_CHAT_ROOM_FRAGMENT) == null) {
-            addFragment(TAG_CHAT_ROOM_FRAGMENT, R.id.fragment_container) {
-                newInstance(
-                    chatRoomId,
-                    chatRoomName,
-                    chatRoomType,
-                    isReadOnly,
-                    chatRoomLastSeen,
-                    isSubscribed,
-                    isCreator,
-                    isFavorite,
-                    chatRoomMessage
-                )
+            if (supportFragmentManager.findFragmentByTag(TAG_CHAT_ROOM_FRAGMENT) == null) {
+                addFragment(TAG_CHAT_ROOM_FRAGMENT, R.id.fragment_container) {
+                    newInstance(
+                            chatRoomId,
+                            chatRoomName,
+                            chatRoomType,
+                            isReadOnly,
+                            chatRoomLastSeen,
+                            isSubscribed,
+                            isCreator,
+                            isFavorite,
+                            chatRoomMessage
+                    )
+                }
             }
         }
     }
@@ -136,31 +134,21 @@ class ChatRoomActivity : AppCompatActivity(), HasSupportFragmentInjector {
         toolbar.setNavigationOnClickListener { finishActivity() }
     }
 
-    fun showToolbarTitle(title: String) {
-        text_room_name.textContent = title
+    fun setupToolbarTitle(title: String) {
+        text_toolbar_title.textContent = title
     }
 
-    fun showToolbarChatRoomIcon(chatRoomType: String) {
-        val drawable = when (roomTypeOf(chatRoomType)) {
-            is RoomType.Channel -> {
-                DrawableHelper.getDrawableFromId(R.drawable.ic_hashtag_black_12dp, this)
-            }
-            is RoomType.PrivateGroup -> {
-                DrawableHelper.getDrawableFromId(R.drawable.ic_lock_black_12_dp, this)
-            }
-            else -> null
-        }
-
-        drawable?.let {
-            val wrappedDrawable = DrawableHelper.wrapDrawable(it)
-            val mutableDrawable = wrappedDrawable.mutate()
-            DrawableHelper.tintDrawable(mutableDrawable, this, R.color.colorWhite)
-            DrawableHelper.compoundDrawable(text_room_name, mutableDrawable)
-        }
+    fun setupExpandMoreForToolbar(listener: (View) -> Unit) {
+        DrawableHelper.compoundRightDrawable(
+            text_toolbar_title,
+            DrawableHelper.getDrawableFromId(R.drawable.ic_chatroom_toolbar_expand_more_20dp, this)
+        )
+        text_toolbar_title.setOnClickListener { listener(it) }
     }
 
-    fun hideToolbarChatRoomIcon() {
-        text_room_name.setCompoundDrawablesWithIntrinsicBounds(0, 0, 0, 0)
+    fun hideExpandMoreForToolbar() {
+        text_toolbar_title.setCompoundDrawablesWithIntrinsicBounds(0, 0, 0, 0)
+        text_toolbar_title.setOnClickListener(null)
     }
 
     private fun finishActivity() {

app/src/main/java/chat/rocket/android/chatroom/ui/Dialog.kt

@@ -4,10 +4,11 @@ import android.graphics.Bitmap
 import android.graphics.drawable.Drawable
 import android.net.Uri
 import androidx.core.view.isVisible
-import chat.rocket.android.emoji.internal.GlideApp
 import chat.rocket.android.util.extensions.getFileName
 import chat.rocket.android.util.extensions.getMimeType
 import chat.rocket.common.util.ifNull
+import com.bumptech.glide.Glide
+import com.bumptech.glide.request.RequestOptions
 import com.bumptech.glide.request.target.SimpleTarget
 import com.bumptech.glide.request.transition.Transition
 
@@ -25,18 +26,18 @@ fun ChatRoomFragment.showFileAttachmentDialog(uri: Uri) {
             when {
                 mimeType.startsWith("image") -> {
                     if (mimeType.contains("gif")) {
-                        GlideApp
+                        Glide
                             .with(context)
                             .asGif()
                             .load(uri)
-                            .fitCenter()
+                            .apply(RequestOptions().fitCenter())
                             .into(imagePreview)
                     } else {
-                        GlideApp
+                        Glide
                             .with(context)
                             .asBitmap()
                             .load(uri)
-                            .fitCenter()
+                            .apply(RequestOptions().fitCenter())
                             .into(object : SimpleTarget<Bitmap>() {
                                 override fun onResourceReady(
                                     resource: Bitmap,

app/src/main/java/chat/rocket/android/chatroom/ui/Menu.kt

@@ -11,23 +11,6 @@ import chat.rocket.android.util.extension.onQueryTextListener
 
 internal fun ChatRoomFragment.setupMenu(menu: Menu) {
     setupSearchMessageMenuItem(menu, requireContext())
-    setupFavoriteMenuItem(menu)
-    setupDetailsMenuItem(menu)
-}
-
-internal fun ChatRoomFragment.setOnMenuItemClickListener(item: MenuItem) {
-    when (item.itemId) {
-        MENU_ACTION_FAVORITE_UNFAVOURITE_CHAT -> presenter.toggleFavoriteChatRoom(
-            chatRoomId,
-            isFavorite
-        )
-        MENU_ACTION_SHOW_DETAILS -> presenter.toChatDetails(
-            chatRoomId,
-            chatRoomType,
-            isSubscribed,
-            disableMenu
-        )
-    }
 }
 
 private fun ChatRoomFragment.setupSearchMessageMenuItem(menu: Menu, context: Context) {
@@ -37,27 +20,23 @@ private fun ChatRoomFragment.setupSearchMessageMenuItem(menu: Menu, context: Con
         Menu.NONE,
         R.string.title_search_message
     ).setActionView(SearchView(context))
-        .setIcon(R.drawable.ic_search_white_24dp)
+        .setIcon(R.drawable.ic_chatroom_toolbar_magnifier_20dp)
         .setShowAsActionFlags(
             MenuItem.SHOW_AS_ACTION_IF_ROOM or MenuItem.SHOW_AS_ACTION_COLLAPSE_ACTION_VIEW
         )
         .setOnActionExpandListener(object : MenuItem.OnActionExpandListener {
             override fun onMenuItemActionExpand(item: MenuItem?): Boolean {
                 dismissEmojiKeyboard()
-                removeFavoriteMenuItem(menu)
-                removeDetailMenuItem(menu)
                 return true
             }
 
             override fun onMenuItemActionCollapse(item: MenuItem?): Boolean {
                 dismissEmojiKeyboard()
-                setupFavoriteMenuItem(menu)
-                setupDetailsMenuItem(menu)
                 return true
             }
         })
 
-    (searchItem?.actionView as? SearchView)?.let {
+    (searchItem.actionView as? SearchView)?.let {
         // TODO: Check why we need to stylize the search text programmatically instead of by defining it in the styles.xml (ChatRoom.SearchView)
         stylizeSearchView(it, context)
         setupSearchViewTextListener(it)
@@ -77,6 +56,7 @@ private fun stylizeSearchView(searchView: SearchView, context: Context) {
 
 private fun ChatRoomFragment.setupSearchViewTextListener(searchView: SearchView) {
     searchView.onQueryTextListener {
+        // TODO: We use isSearchTermQueried to avoid querying when the search view is expanded but the user doesn't start typing. Check for a native solution.
         if (it.isEmpty() && isSearchTermQueried) {
             presenter.loadMessages(chatRoomId, chatRoomType, clearDataSet = true)
         } else if (it.isNotEmpty()) {
@@ -84,42 +64,4 @@ private fun ChatRoomFragment.setupSearchViewTextListener(searchView: SearchView)
             isSearchTermQueried = true
         }
     }
-}
-
-private fun ChatRoomFragment.setupFavoriteMenuItem(menu: Menu) {
-    if (isFavorite) {
-        menu.add(
-            Menu.NONE,
-            MENU_ACTION_FAVORITE_UNFAVOURITE_CHAT,
-            Menu.NONE,
-            R.string.title_unfavorite_chat
-        ).setIcon(R.drawable.ic_star_yellow_24dp)
-            .setShowAsAction(MenuItem.SHOW_AS_ACTION_IF_ROOM)
-    } else {
-        menu.add(
-            Menu.NONE,
-            MENU_ACTION_FAVORITE_UNFAVOURITE_CHAT,
-            Menu.NONE,
-            R.string.title_favorite_chat
-        ).setIcon(R.drawable.ic_star_border_white_24dp)
-            .setShowAsAction(MenuItem.SHOW_AS_ACTION_IF_ROOM)
-    }
-}
-
-private fun ChatRoomFragment.setupDetailsMenuItem(menu: Menu) {
-    menu.add(
-        Menu.NONE,
-        MENU_ACTION_SHOW_DETAILS,
-        Menu.NONE,
-        R.string.title_channel_details
-    ).setIcon(R.drawable.ic_info_outline_white_24dp)
-        .setShowAsAction(MenuItem.SHOW_AS_ACTION_IF_ROOM)
-}
-
-private fun removeFavoriteMenuItem(menu: Menu) {
-    menu.removeItem(MENU_ACTION_FAVORITE_UNFAVOURITE_CHAT)
-}
-
-private fun removeDetailMenuItem(menu: Menu) {
-    menu.removeItem(MENU_ACTION_SHOW_DETAILS)
 }
\ No newline at end of file

app/src/main/java/chat/rocket/android/chatroom/uimodel/BaseUiModel.kt

@@ -30,5 +30,5 @@ interface BaseUiModel<out T> {
 
 internal fun Int.toViewType(): BaseUiModel.ViewType {
     return BaseUiModel.ViewType.values().firstOrNull { it.viewType == this }
-            ?: throw InvalidParameterException("Invalid viewType: $this for BaseUiModel.ViewType")
+        ?: throw InvalidParameterException("Invalid viewType: $this for BaseUiModel.ViewType")
 }
\ No newline at end of file

app/src/main/java/chat/rocket/android/chatroom/uimodel/MessageUiModel.kt

@@ -26,7 +26,6 @@ data class MessageUiModel(
 ) : BaseMessageUiModel<Message> {
     override val viewType: Int
         get() = BaseUiModel.ViewType.MESSAGE.viewType
-
     override val layoutId: Int
         get() = R.layout.item_message
 }
\ No newline at end of file

app/src/main/java/chat/rocket/android/chatroom/uimodel/UiModelMapper.kt

@@ -44,8 +44,8 @@ import chat.rocket.core.model.attachment.Attachment
 import chat.rocket.core.model.attachment.Field
 import chat.rocket.core.model.isSystemMessage
 import chat.rocket.core.model.url.Url
-import kotlinx.coroutines.experimental.CommonPool
-import kotlinx.coroutines.experimental.withContext
+import kotlinx.coroutines.Dispatchers
+import kotlinx.coroutines.withContext
 import okhttp3.HttpUrl
 import java.security.InvalidParameterException
 import java.util.*
@@ -79,7 +79,7 @@ class UiModelMapper @Inject constructor(
         message: Message,
         roomUiModel: RoomUiModel = RoomUiModel(roles = emptyList(), isBroadcast = true)
     ): List<BaseUiModel<*>> =
-        withContext(CommonPool) {
+        withContext(Dispatchers.IO) {
             return@withContext translate(message, roomUiModel)
         }
 
@@ -88,7 +88,7 @@ class UiModelMapper @Inject constructor(
         roomUiModel: RoomUiModel = RoomUiModel(roles = emptyList(), isBroadcast = true),
         asNotReversed: Boolean = false
     ): List<BaseUiModel<*>> =
-        withContext(CommonPool) {
+        withContext(Dispatchers.IO) {
             val list = ArrayList<BaseUiModel<*>>(messages.size)
 
             messages.forEach {
@@ -102,7 +102,7 @@ class UiModelMapper @Inject constructor(
 
     suspend fun map(
         readReceipts: List<ReadReceipt>
-    ): List<ReadReceiptViewModel> = withContext(CommonPool) {
+    ): List<ReadReceiptViewModel> = withContext(Dispatchers.IO) {
         val list = arrayListOf<ReadReceiptViewModel>()
 
         readReceipts.forEach {
@@ -121,7 +121,7 @@ class UiModelMapper @Inject constructor(
         message: Message,
         roomUiModel: RoomUiModel
     ): List<BaseUiModel<*>> =
-        withContext(CommonPool) {
+        withContext(Dispatchers.IO) {
             val list = ArrayList<BaseUiModel<*>>()
 
             getChatRoomAsync(message.roomId)?.let { chatRoom ->
@@ -167,7 +167,7 @@ class UiModelMapper @Inject constructor(
         }
 
     // TODO: move this to new interactor or FetchChatRoomsInteractor?
-    private suspend fun getChatRoomAsync(roomId: String): ChatRoom? = withContext(CommonPool) {
+    private suspend fun getChatRoomAsync(roomId: String): ChatRoom? = withContext(Dispatchers.IO) {
         return@withContext dbManager.getRoom(id = roomId)?.let {
             with(it.chatRoom) {
                 ChatRoom(
@@ -212,7 +212,7 @@ class UiModelMapper @Inject constructor(
         message: Message,
         roomUiModel: RoomUiModel
     ): List<BaseUiModel<*>> =
-        withContext(CommonPool) {
+        withContext(Dispatchers.IO) {
             val list = ArrayList<BaseUiModel<*>>()
 
             getChatRoomAsync(message.roomId)?.let { chatRoom ->
@@ -416,21 +416,19 @@ class UiModelMapper @Inject constructor(
         return fullUrl
     }
 
-    private fun attachmentText(text: String?, attachment: Attachment?, context: Context): String? {
-        return if (attachment != null) {
+    private fun attachmentText(text: String?, attachment: Attachment?, context: Context): String? = attachment?.run {
+        with(context) {
             when {
-                attachment.imageUrl.isNotNullNorEmpty() -> context.getString(R.string.msg_preview_photo)
-                attachment.videoUrl.isNotNullNorEmpty() -> context.getString(R.string.msg_preview_video)
-                attachment.audioUrl.isNotNullNorEmpty() -> context.getString(R.string.msg_preview_audio)
-                attachment.titleLink.isNotNullNorEmpty() &&
-                        attachment.type?.contentEquals("file") == true ->
-                    context.getString(R.string.msg_preview_file)
+                imageUrl.isNotNullNorEmpty() -> getString(R.string.msg_preview_photo)
+                videoUrl.isNotNullNorEmpty() -> getString(R.string.msg_preview_video)
+                audioUrl.isNotNullNorEmpty() -> getString(R.string.msg_preview_audio)
+                titleLink.isNotNullNorEmpty() &&
+                        type?.contentEquals("file") == true ->
+                    getString(R.string.msg_preview_file)
                 else -> text
             }
-        } else {
-            text
         }
-    }
+    } ?: text
 
     private fun attachmentDescription(attachment: Attachment): String? {
         return attachment.description
@@ -439,7 +437,7 @@ class UiModelMapper @Inject constructor(
     private suspend fun mapMessage(
         message: Message,
         chatRoom: ChatRoom
-    ): MessageUiModel = withContext(CommonPool) {
+    ): MessageUiModel = withContext(Dispatchers.IO) {
         val sender = getSenderName(message)
         val time = getTime(message.timestamp)
         val avatar = getUserAvatar(message)
@@ -464,11 +462,9 @@ class UiModelMapper @Inject constructor(
             subscriptionId = chatRoom.subscriptionId)
     }
 
-    private fun mapMessagePreview(message: Message): Message {
-        return when (message.isSystemMessage()) {
-            false -> stripMessageQuotes(message)
-            true -> message.copy(message = getSystemMessage(message).toString())
-        }
+    private fun mapMessagePreview(message: Message): Message = when (message.isSystemMessage()) {
+        false -> stripMessageQuotes(message)
+        true -> message.copy(message = getSystemMessage(message).toString())
     }
 
     private fun getReactions(message: Message): List<ReactionUiModel> {
@@ -535,38 +531,36 @@ class UiModelMapper @Inject constructor(
 
     private fun getTime(timestamp: Long) = DateTimeHelper.getTime(DateTimeHelper.getLocalDateTime(timestamp))
 
-    private fun getContent(message: Message): CharSequence {
-        return when (message.isSystemMessage()) {
-            true -> getSystemMessage(message)
-            false -> parser.render(message, currentUsername)
-        }
+    private fun getContent(message: Message): CharSequence = when (message.isSystemMessage()) {
+        true -> getSystemMessage(message)
+        false -> parser.render(message, currentUsername)
     }
 
     private fun getSystemMessage(message: Message): CharSequence {
-        val content = when (message.type) {
-            //TODO: Add implementation for Welcome type.
-            is MessageType.MessageRemoved -> context.getString(R.string.message_removed)
-            is MessageType.UserJoined -> context.getString(R.string.message_user_joined_channel)
-            is MessageType.UserLeft -> context.getString(R.string.message_user_left)
-            is MessageType.UserAdded -> context.getString(R.string.message_user_added_by, message.message, message.sender?.username)
-            is MessageType.RoomNameChanged -> context.getString(R.string.message_room_name_changed, message.message, message.sender?.username)
-            is MessageType.UserRemoved -> context.getString(R.string.message_user_removed_by, message.message, message.sender?.username)
-            is MessageType.MessagePinned -> context.getString(R.string.message_pinned)
-            is MessageType.UserMuted -> context.getString(R.string.message_muted, message.message, message.sender?.username)
-            is MessageType.UserUnMuted -> context.getString(R.string.message_unmuted, message.message, message.sender?.username)
-            is MessageType.SubscriptionRoleAdded -> context.getString(R.string.message_role_add, message.message, message.role, message.sender?.username)
-            is MessageType.SubscriptionRoleRemoved -> context.getString(R.string.message_role_removed, message.message, message.role, message.sender?.username)
-            is MessageType.RoomChangedPrivacy -> context.getString(R.string.message_room_changed_privacy, message.message, message.sender?.username)
-            else -> {
-                throw InvalidParameterException("Invalid message type: ${message.type}")
+        val content = with(context) {
+            when (message.type) {
+                //TODO: Add implementation for Welcome type.
+                is MessageType.MessageRemoved -> getString(R.string.message_removed)
+                is MessageType.UserJoined -> getString(R.string.message_user_joined_channel)
+                is MessageType.UserLeft -> getString(R.string.message_user_left)
+                is MessageType.UserAdded -> getString(R.string.message_user_added_by, message.message, message.sender?.username)
+                is MessageType.RoomNameChanged -> getString(R.string.message_room_name_changed, message.message, message.sender?.username)
+                is MessageType.UserRemoved -> getString(R.string.message_user_removed_by, message.message, message.sender?.username)
+                is MessageType.MessagePinned -> getString(R.string.message_pinned)
+                is MessageType.UserMuted -> getString(R.string.message_muted, message.message, message.sender?.username)
+                is MessageType.UserUnMuted -> getString(R.string.message_unmuted, message.message, message.sender?.username)
+                is MessageType.SubscriptionRoleAdded -> getString(R.string.message_role_add, message.message, message.role, message.sender?.username)
+                is MessageType.SubscriptionRoleRemoved -> getString(R.string.message_role_removed, message.message, message.role, message.sender?.username)
+                is MessageType.RoomChangedPrivacy -> getString(R.string.message_room_changed_privacy, message.message, message.sender?.username)
+                is MessageType.JitsiCallStarted -> context.getString(
+                    R.string.message_video_call_started, message.sender?.username
+                )
+                else -> throw InvalidParameterException("Invalid message type: ${message.type}")
             }
         }
         val spannableMsg = SpannableStringBuilder(content)
-        spannableMsg.setSpan(StyleSpan(Typeface.ITALIC), 0, spannableMsg.length,
-            0)
-        spannableMsg.setSpan(ForegroundColorSpan(Color.GRAY), 0, spannableMsg.length,
-            0)
-
+        spannableMsg.setSpan(StyleSpan(Typeface.ITALIC), 0, spannableMsg.length, 0)
+        spannableMsg.setSpan(ForegroundColorSpan(Color.GRAY), 0, spannableMsg.length, 0)
         return spannableMsg
     }
 }
\ No newline at end of file

app/src/main/java/chat/rocket/android/chatrooms/adapter/RoomUiModelMapper.kt

@@ -77,77 +77,71 @@ class RoomUiModelMapper(
         return list
     }
 
-    private fun mapUser(user: User): RoomUiModel {
-        return with(user) {
-            val name = mapName(user.username!!, user.name)
-            val status = user.status
-            val avatar = serverUrl.avatarUrl(user.username!!)
-            val username = user.username!!
-
-            RoomUiModel(
+    private fun mapUser(user: User): RoomUiModel = with(user) {
+        val name = mapName(user.username!!, user.name)
+        val status = user.status
+        val avatar = serverUrl.avatarUrl(user.username!!)
+        val username = user.username!!
+
+        RoomUiModel(
                 id = user.id,
                 name = name,
                 type = roomTypeOf(RoomType.DIRECT_MESSAGE),
                 avatar = avatar,
                 status = status,
                 username = username
-            )
-        }
+        )
     }
 
-    private fun mapRoom(room: Room, showLastMessage: Boolean = true): RoomUiModel {
-        return with(room) {
-            RoomUiModel(
+    private fun mapRoom(room: Room, showLastMessage: Boolean = true): RoomUiModel = with(room) {
+        RoomUiModel(
                 id = id,
                 name = name!!,
                 type = type,
                 avatar = serverUrl.avatarUrl(name!!, isGroupOrChannel = true),
                 lastMessage = if (showLastMessage) {
                     mapLastMessage(
-                        lastMessage?.sender?.id, lastMessage?.sender?.username,
-                        lastMessage?.sender?.name, lastMessage?.message,
-                        isDirectMessage = type is RoomType.DirectMessage
+                            lastMessage?.sender?.id, lastMessage?.sender?.username,
+                            lastMessage?.sender?.name, lastMessage?.message,
+                            isDirectMessage = type is RoomType.DirectMessage
                     )
                 } else {
                     null
                 },
                 muted = muted.orEmpty(),
                 writable = isChannelWritable(muted)
-            )
-        }
+        )
     }
 
-    fun map(chatRoom: ChatRoom, showLastMessage: Boolean = true): RoomUiModel {
-        return with(chatRoom.chatRoom) {
-            val isUnread = alert || unread > 0
-            val type = roomTypeOf(type)
-            val status = chatRoom.status?.let { userStatusOf(it) }
-            val roomName = mapName(name, fullname)
-            val favorite = favorite
-            val timestamp = mapDate(lastMessageTimestamp ?: updatedAt)
-            val avatar = if (type is RoomType.DirectMessage) {
-                serverUrl.avatarUrl(name)
-            } else {
-                serverUrl.avatarUrl(name, isGroupOrChannel = true)
-            }
-            val unread = mapUnread(unread)
-            val lastMessage = if (showLastMessage) {
-                mapLastMessage(
+    fun map(chatRoom: ChatRoom, showLastMessage: Boolean = true): RoomUiModel = with(chatRoom.chatRoom) {
+        val isUnread = alert || unread > 0
+        val type = roomTypeOf(type)
+        val status = chatRoom.status?.let { userStatusOf(it) }
+        val roomName = mapName(name, fullname)
+        val favorite = favorite
+        val timestamp = mapDate(lastMessageTimestamp ?: updatedAt)
+        val avatar = if (type is RoomType.DirectMessage) {
+            serverUrl.avatarUrl(name)
+        } else {
+            serverUrl.avatarUrl(name, isGroupOrChannel = true)
+        }
+        val unread = mapUnread(unread)
+        val lastMessage = if (showLastMessage) {
+            mapLastMessage(
                     lastMessageUserId,
                     chatRoom.lastMessageUserName,
                     chatRoom.lastMessageUserFullName,
                     lastMessageText,
                     type is RoomType.DirectMessage
-                )
-            } else {
-                null
-            }
-            val hasMentions = mapMentions(userMentions, groupMentions)
-            val open = open
-            val lastMessageMarkdown =
-                lastMessage?.let { Markwon.markdown(context, it.toString()).toString() }
+            )
+        } else {
+            null
+        }
+        val hasMentions = mapMentions(userMentions, groupMentions)
+        val open = open
+        val lastMessageMarkdown = lastMessage?.let { Markwon.markdown(context, it.toString()).toString() }
 
-            RoomUiModel(
+        RoomUiModel(
                 id = id,
                 name = roomName,
                 type = type,
@@ -163,8 +157,7 @@ class RoomUiModelMapper(
                 username = if (type is RoomType.DirectMessage) name else null,
                 muted = muted.orEmpty(),
                 writable = isChannelWritable(muted)
-            )
-        }
+        )
     }
 
     private fun isChannelWritable(muted: List<String>?): Boolean {
@@ -172,15 +165,13 @@ class RoomUiModelMapper(
         return canWriteToReadOnlyChannels || !muted.orEmpty().contains(currentUser?.username)
     }
 
-    private fun roomType(type: String): String {
-        val resources = context.resources
-
-        return when (type) {
-            RoomType.CHANNEL -> resources.getString(R.string.header_channel)
-            RoomType.PRIVATE_GROUP -> resources.getString(R.string.header_private_groups)
-            RoomType.DIRECT_MESSAGE -> resources.getString(R.string.header_direct_messages)
-            RoomType.LIVECHAT -> resources.getString(R.string.header_live_chats)
-            else -> resources.getString(R.string.header_unknown)
+    private fun roomType(type: String): String = with(context.resources) {
+        when (type) {
+            RoomType.CHANNEL -> getString(R.string.header_channel)
+            RoomType.PRIVATE_GROUP -> getString(R.string.header_private_groups)
+            RoomType.DIRECT_MESSAGE -> getString(R.string.header_direct_messages)
+            RoomType.LIVECHAT -> getString(R.string.header_live_chats)
+            else -> getString(R.string.header_unknown)
         }
     }
 
@@ -213,13 +204,11 @@ class RoomUiModelMapper(
         }
     }
 
-    private fun mapUnread(unread: Long): String? {
-        return when (unread) {
-            0L -> null
-            in 1..99 -> unread.toString()
-            else -> context.getString(R.string.msg_more_than_ninety_nine_unread_messages)
+    private fun mapUnread(unread: Long): String? = when (unread) {
+        0L -> null
+        in 1..99 -> unread.toString()
+        else -> context.getString(R.string.msg_more_than_ninety_nine_unread_messages)
 
-        }
     }
 
     private fun mapMentions(userMentions: Long?, groupMentions: Long?): Boolean {

app/src/main/java/chat/rocket/android/chatrooms/adapter/RoomViewHolder.kt

@@ -8,6 +8,7 @@ import androidx.core.view.isInvisible
 import androidx.core.view.isVisible
 import chat.rocket.android.R
 import chat.rocket.android.chatrooms.adapter.model.RoomUiModel
+import chat.rocket.android.util.extension.setTextViewAppearance
 import chat.rocket.common.model.RoomType
 import chat.rocket.common.model.UserStatus
 import kotlinx.android.synthetic.main.item_chat.view.*
@@ -16,12 +17,12 @@ import kotlinx.android.synthetic.main.unread_messages_badge.view.*
 class RoomViewHolder(itemView: View, private val listener: (RoomUiModel) -> Unit) :
     ViewHolder<RoomItemHolder>(itemView) {
     private val resources: Resources = itemView.resources
-    private val channelIcon: Drawable = resources.getDrawable(R.drawable.ic_hashtag_12dp)
-    private val groupIcon: Drawable = resources.getDrawable(R.drawable.ic_lock_12_dp)
-    private val onlineIcon: Drawable = resources.getDrawable(R.drawable.ic_status_online_12dp)
-    private val awayIcon: Drawable = resources.getDrawable(R.drawable.ic_status_away_12dp)
-    private val busyIcon: Drawable = resources.getDrawable(R.drawable.ic_status_busy_12dp)
-    private val offlineIcon: Drawable = resources.getDrawable(R.drawable.ic_status_invisible_12dp)
+    private val channelIcon: Drawable = resources.getDrawable(R.drawable.ic_hashtag_12dp, null)
+    private val groupIcon: Drawable = resources.getDrawable(R.drawable.ic_lock_12_dp, null)
+    private val onlineIcon: Drawable = resources.getDrawable(R.drawable.ic_status_online_12dp, null)
+    private val awayIcon: Drawable = resources.getDrawable(R.drawable.ic_status_away_12dp, null)
+    private val busyIcon: Drawable = resources.getDrawable(R.drawable.ic_status_busy_12dp, null)
+    private val offlineIcon: Drawable = resources.getDrawable(R.drawable.ic_status_invisible_12dp, null)
 
     override fun bindViews(data: RoomItemHolder) {
         val room = data.data
@@ -53,14 +54,14 @@ class RoomViewHolder(itemView: View, private val listener: (RoomUiModel) -> Unit
                 if (room.unread == null) text_total_unread_messages.text = "!"
                 if (room.unread != null) text_total_unread_messages.text = room.unread
                 if (room.mentions) text_total_unread_messages.text = "@${room.unread}"
-                text_chat_name.setTextAppearance(context, R.style.ChatList_ChatName_Unread_TextView)
-                text_timestamp.setTextAppearance(context, R.style.ChatList_Timestamp_Unread_TextView)
-                text_last_message.setTextAppearance(context, R.style.ChatList_LastMessage_Unread_TextView)
+                text_chat_name.setTextViewAppearance(context, R.style.ChatList_ChatName_Unread_TextView)
+                text_timestamp.setTextViewAppearance(context, R.style.ChatList_Timestamp_Unread_TextView)
+                text_last_message.setTextViewAppearance(context, R.style.ChatList_LastMessage_Unread_TextView)
                 text_total_unread_messages.isVisible = true
             } else {
-                text_chat_name.setTextAppearance(context, R.style.ChatList_ChatName_TextView)
-                text_timestamp.setTextAppearance(context, R.style.ChatList_Timestamp_TextView)
-                text_last_message.setTextAppearance(context, R.style.ChatList_LastMessage_TextView)
+                text_chat_name.setTextViewAppearance(context, R.style.ChatList_ChatName_TextView)
+                text_timestamp.setTextViewAppearance(context, R.style.ChatList_Timestamp_TextView)
+                text_last_message.setTextViewAppearance(context, R.style.ChatList_LastMessage_TextView)
                 text_total_unread_messages.isInvisible = true
             }
 
@@ -68,20 +69,16 @@ class RoomViewHolder(itemView: View, private val listener: (RoomUiModel) -> Unit
         }
     }
 
-    private fun getRoomDrawable(type: RoomType): Drawable? {
-        return when (type) {
-            is RoomType.Channel -> channelIcon
-            is RoomType.PrivateGroup -> groupIcon
-            else -> null
-        }
+    private fun getRoomDrawable(type: RoomType): Drawable? = when (type) {
+        is RoomType.Channel -> channelIcon
+        is RoomType.PrivateGroup -> groupIcon
+        else -> null
     }
 
-    private fun getStatusDrawable(status: UserStatus): Drawable {
-        return when (status) {
-            is UserStatus.Online -> onlineIcon
-            is UserStatus.Away -> awayIcon
-            is UserStatus.Busy -> busyIcon
-            else -> offlineIcon
-        }
+    private fun getStatusDrawable(status: UserStatus): Drawable = when (status) {
+        is UserStatus.Online -> onlineIcon
+        is UserStatus.Away -> awayIcon
+        is UserStatus.Busy -> busyIcon
+        else -> offlineIcon
     }
 }
\ No newline at end of file

app/src/main/java/chat/rocket/android/chatrooms/adapter/RoomsAdapter.kt

@@ -19,22 +19,20 @@ class RoomsAdapter(private val listener: (RoomUiModel) -> Unit) :
             notifyDataSetChanged()
         }
 
-    override fun onCreateViewHolder(parent: ViewGroup, viewType: Int): ViewHolder<*> {
-        return when (viewType) {
-            VIEW_TYPE_ROOM -> {
-                val view = parent.inflate(R.layout.item_chat)
-                RoomViewHolder(view, listener)
-            }
-            VIEW_TYPE_HEADER -> {
-                val view = parent.inflate(R.layout.item_chatroom_header)
-                HeaderViewHolder(view)
-            }
-            VIEW_TYPE_LOADING -> {
-                val view = parent.inflate(R.layout.item_loading)
-                LoadingViewHolder(view)
-            }
-            else -> throw IllegalStateException("View type must be either Room, Header or Loading")
+    override fun onCreateViewHolder(parent: ViewGroup, viewType: Int): ViewHolder<*> = when (viewType) {
+        VIEW_TYPE_ROOM -> {
+            val view = parent.inflate(R.layout.item_chat)
+            RoomViewHolder(view, listener)
+        }
+        VIEW_TYPE_HEADER -> {
+            val view = parent.inflate(R.layout.item_chatroom_header)
+            HeaderViewHolder(view)
+        }
+        VIEW_TYPE_LOADING -> {
+            val view = parent.inflate(R.layout.item_loading)
+            LoadingViewHolder(view)
         }
+        else -> throw IllegalStateException("View type must be either Room, Header or Loading")
     }
 
     override fun getItemCount() = values.size
@@ -49,13 +47,11 @@ class RoomsAdapter(private val listener: (RoomUiModel) -> Unit) :
         }
     }
 
-    override fun getItemViewType(position: Int): Int {
-        return when (values[position]) {
-            is RoomItemHolder -> VIEW_TYPE_ROOM
-            is HeaderItemHolder -> VIEW_TYPE_HEADER
-            is LoadingItemHolder -> VIEW_TYPE_LOADING
-            else -> throw IllegalStateException("View type must be either Room, Header or Loading")
-        }
+    override fun getItemViewType(position: Int): Int = when (values[position]) {
+        is RoomItemHolder -> VIEW_TYPE_ROOM
+        is HeaderItemHolder -> VIEW_TYPE_HEADER
+        is LoadingItemHolder -> VIEW_TYPE_LOADING
+        else -> throw IllegalStateException("View type must be either Room, Header or Loading")
     }
 
     override fun onBindViewHolder(holder: ViewHolder<*>, position: Int) {

app/src/main/java/chat/rocket/android/chatrooms/presentation/ChatRoomsPresenter.kt

@@ -23,11 +23,12 @@ import chat.rocket.common.model.roomTypeOf
 import chat.rocket.core.internal.realtime.createDirectMessage
 import chat.rocket.core.internal.rest.me
 import chat.rocket.core.internal.rest.show
-import kotlinx.coroutines.experimental.withTimeout
+import kotlinx.coroutines.withTimeout
 import timber.log.Timber
 import javax.inject.Inject
 import javax.inject.Named
-import kotlin.coroutines.experimental.suspendCoroutine
+import kotlin.coroutines.resume
+import kotlin.coroutines.suspendCoroutine
 
 class ChatRoomsPresenter @Inject constructor(
     private val view: ChatRoomsView,

app/src/main/java/chat/rocket/android/chatrooms/ui/ChatRoomsFragment.kt

@@ -36,7 +36,6 @@ import chat.rocket.android.helper.SharedPreferenceHelper
 import chat.rocket.android.util.extension.onQueryTextListener
 import chat.rocket.android.util.extensions.fadeIn
 import chat.rocket.android.util.extensions.fadeOut
-import chat.rocket.android.util.extensions.ifNotNullNorEmpty
 import chat.rocket.android.util.extensions.ifNotNullNotEmpty
 import chat.rocket.android.util.extensions.inflate
 import chat.rocket.android.util.extensions.showToast
@@ -69,11 +68,9 @@ class ChatRoomsFragment : Fragment(), ChatRoomsView {
     private var progressDialog: ProgressDialog? = null
 
     companion object {
-        fun newInstance(chatRoomId: String? = null): ChatRoomsFragment {
-            return ChatRoomsFragment().apply {
-                arguments = Bundle(1).apply {
-                    putString(BUNDLE_CHAT_ROOM_ID, chatRoomId)
-                }
+        fun newInstance(chatRoomId: String? = null): ChatRoomsFragment = ChatRoomsFragment().apply {
+            arguments = Bundle(1).apply {
+                putString(BUNDLE_CHAT_ROOM_ID, chatRoomId)
             }
         }
     }
@@ -82,9 +79,8 @@ class ChatRoomsFragment : Fragment(), ChatRoomsView {
         super.onCreate(savedInstanceState)
         AndroidSupportInjection.inject(this)
         setHasOptionsMenu(true)
-        val bundle = arguments
-        if (bundle != null) {
-            chatRoomId = bundle.getString(BUNDLE_CHAT_ROOM_ID)
+        arguments?.run {
+            chatRoomId = getString(BUNDLE_CHAT_ROOM_ID)
             chatRoomId.ifNotNullNotEmpty { roomId ->
                 presenter.loadChatRoom(roomId)
                 chatRoomId = null
@@ -129,12 +125,14 @@ class ChatRoomsFragment : Fragment(), ChatRoomsView {
                 )
             )
             recycler_view.itemAnimator = DefaultItemAnimator()
-            recycler_view.adapter = adapter
 
             viewModel.getChatRooms().observe(viewLifecycleOwner, Observer { rooms ->
                 rooms?.let {
                     Timber.d("Got items: $it")
                     adapter.values = it
+                    if (recycler_view.adapter != adapter) {
+                        recycler_view.adapter = adapter
+                    }
                     if (rooms.isNotEmpty()) {
                         text_no_data_to_display.isVisible = false
                     }
@@ -320,21 +318,20 @@ class ChatRoomsFragment : Fragment(), ChatRoomsView {
         ui {
             text_connection_status.fadeIn()
             handler.removeCallbacks(dismissStatus)
-            when (state) {
+            text_connection_status.text = when (state) {
                 is State.Connected -> {
-                    text_connection_status.text = getString(R.string.status_connected)
                     handler.postDelayed(dismissStatus, 2000)
+                    getString(R.string.status_connected)
+                }
+                is State.Disconnected -> getString(R.string.status_disconnected)
+                is State.Connecting -> getString(R.string.status_connecting)
+                is State.Authenticating -> getString(R.string.status_authenticating)
+                is State.Disconnecting -> getString(R.string.status_disconnecting)
+                is State.Waiting -> getString(R.string.status_waiting, state.seconds)
+                else -> {
+                    handler.postDelayed(dismissStatus, 500)
+                    ""
                 }
-                is State.Disconnected -> text_connection_status.text =
-                        getString(R.string.status_disconnected)
-                is State.Connecting -> text_connection_status.text =
-                        getString(R.string.status_connecting)
-                is State.Authenticating -> text_connection_status.text =
-                        getString(R.string.status_authenticating)
-                is State.Disconnecting -> text_connection_status.text =
-                        getString(R.string.status_disconnecting)
-                is State.Waiting -> text_connection_status.text =
-                        getString(R.string.status_waiting, state.seconds)
             }
         }
     }

app/src/main/java/chat/rocket/android/chatrooms/viewmodel/ChatRoomsViewModel.kt

@@ -20,16 +20,16 @@ import chat.rocket.core.model.SpotlightResult
 import com.shopify.livedataktx.distinct
 import com.shopify.livedataktx.map
 import com.shopify.livedataktx.nonNull
-import kotlinx.coroutines.experimental.android.UI
-import kotlinx.coroutines.experimental.delay
-import kotlinx.coroutines.experimental.isActive
-import kotlinx.coroutines.experimental.launch
-import kotlinx.coroutines.experimental.newSingleThreadContext
-import kotlinx.coroutines.experimental.withContext
+import kotlinx.coroutines.Dispatchers
+import kotlinx.coroutines.GlobalScope
+import kotlinx.coroutines.delay
+import kotlinx.coroutines.isActive
+import kotlinx.coroutines.launch
+import kotlinx.coroutines.newSingleThreadContext
+import kotlinx.coroutines.withContext
 import timber.log.Timber
 import java.lang.IllegalArgumentException
-import kotlin.coroutines.experimental.coroutineContext
-
+import kotlin.coroutines.coroutineContext
 
 class ChatRoomsViewModel(
     private val connectionManager: ConnectionManager,
@@ -107,7 +107,7 @@ class ChatRoomsViewModel(
     }
 
     private fun fetchRooms() {
-        launch {
+        GlobalScope.launch {
             setLoadingState(LoadingState.Loading(repository.count()))
             try {
                 interactor.refreshChatRooms()
@@ -125,7 +125,7 @@ class ChatRoomsViewModel(
     }
 
     private suspend fun setLoadingState(state: LoadingState) {
-        withContext(UI) {
+        withContext(Dispatchers.Main) {
             loadingState.value = state
         }
     }

app/src/main/java/chat/rocket/android/createchannel/ui/CreateChannelFragment.kt

@@ -45,9 +45,7 @@ class CreateChannelFragment : Fragment(), CreateChannelView, ActionMode.Callback
     lateinit var analyticsManager: AnalyticsManager
     private var actionMode: ActionMode? = null
     private val adapter: MembersAdapter = MembersAdapter {
-        if (it.username != null) {
-            processSelectedMember(it.username)
-        }
+        it.username?.run { processSelectedMember(this) }
     }
     private val compositeDisposable = CompositeDisposable()
     private var channelType: String = RoomType.CHANNEL
@@ -294,8 +292,8 @@ class CreateChannelFragment : Fragment(), CreateChannelView, ActionMode.Callback
 
     private fun addChip(chipText: String) {
         val chip = Chip(context)
-        chip.chipText = chipText
-        chip.isCloseIconEnabled = true
+        chip.text = chipText
+        chip.isCloseIconVisible = true
         chip.setChipBackgroundColorResource(R.color.icon_grey)
         setupChipOnCloseIconClickListener(chip)
         chip_group_member.addView(chip)
@@ -304,7 +302,7 @@ class CreateChannelFragment : Fragment(), CreateChannelView, ActionMode.Callback
     private fun setupChipOnCloseIconClickListener(chip: Chip) {
         chip.setOnCloseIconClickListener {
             removeChip(it)
-            removeMember((it as Chip).chipText.toString())
+            removeMember((it as Chip).text.toString())
             // whenever we remove a chip we should process the chip group visibility.
             processChipGroupVisibility()
         }

app/src/main/java/chat/rocket/android/dagger/module/ActivityBuilder.kt

@@ -37,6 +37,8 @@ import chat.rocket.android.settings.di.SettingsFragmentProvider
 import chat.rocket.android.settings.password.di.PasswordFragmentProvider
 import chat.rocket.android.settings.password.ui.PasswordActivity
 import chat.rocket.android.userdetails.di.UserDetailsFragmentProvider
+import chat.rocket.android.videoconference.di.VideoConferenceModule
+import chat.rocket.android.videoconference.ui.VideoConferenceActivity
 import chat.rocket.android.webview.adminpanel.di.AdminPanelWebViewFragmentProvider
 import dagger.Module
 import dagger.android.ContributesAndroidInjector
@@ -103,4 +105,8 @@ abstract class ActivityBuilder {
     @PerActivity
     @ContributesAndroidInjector(modules = [DrawModule::class])
     abstract fun bindDrawingActivity(): DrawingActivity
+
+    @PerActivity
+    @ContributesAndroidInjector(modules = [VideoConferenceModule::class])
+    abstract fun bindVideoConferenceActivity(): VideoConferenceActivity
 }

app/src/main/java/chat/rocket/android/dagger/module/AppModule.kt

@@ -63,7 +63,6 @@ import chat.rocket.common.internal.FallbackSealedClassJsonAdapter
 import chat.rocket.common.internal.ISO8601Date
 import chat.rocket.common.model.TimestampAdapter
 import chat.rocket.common.util.CalendarISO8601Converter
-import chat.rocket.common.util.Logger
 import chat.rocket.common.util.NoOpLogger
 import chat.rocket.common.util.PlatformLogger
 import chat.rocket.core.internal.AttachmentAdapterFactory

app/src/main/java/chat/rocket/android/dagger/module/LocalModule.kt

@@ -13,7 +13,6 @@ import chat.rocket.common.internal.FallbackSealedClassJsonAdapter
 import chat.rocket.common.internal.ISO8601Date
 import chat.rocket.common.model.TimestampAdapter
 import chat.rocket.common.util.CalendarISO8601Converter
-import chat.rocket.common.util.Logger
 import chat.rocket.common.util.NoOpLogger
 import chat.rocket.common.util.PlatformLogger
 import chat.rocket.core.internal.AttachmentAdapterFactory

app/src/main/java/chat/rocket/android/db/model/Attachments.kt

@@ -115,12 +115,12 @@ fun Attachment.asEntity(msgId: String, context: Context): List<BaseMessageEntity
 
     val text = mapAttachmentText(text, attachments?.firstOrNull(), context)
 
-    val entity = AttachmentEntity(
+    list.add(AttachmentEntity(
             _id = attachmentId,
             messageId = msgId,
             title = title,
             type = type,
-            description =  description,
+            description = description,
             text = text,
             titleLink = titleLink,
             titleLinkDownload = titleLinkDownload.orFalse(),
@@ -144,16 +144,14 @@ fun Attachment.asEntity(msgId: String, context: Context): List<BaseMessageEntity
             buttonAlignment = buttonAlignment,
             hasActions = actions?.isNotEmpty() == true,
             hasFields = fields?.isNotEmpty() == true
-    )
-    list.add(entity)
+    ))
 
     fields?.forEach { field ->
-        val entity = AttachmentFieldEntity(
+        list.add(AttachmentFieldEntity(
                 attachmentId = attachmentId,
                 title = field.title,
                 value = field.value
-        )
-        list.add(entity)
+        ))
     }
 
     actions?.forEach { action ->
@@ -175,18 +173,14 @@ fun Attachment.asEntity(msgId: String, context: Context): List<BaseMessageEntity
     return list
 }
 
-fun mapAttachmentText(text: String?, attachment: Attachment?, context: Context): String? {
-    return if (attachment != null) {
-        when {
-            attachment.imageUrl.isNotNullNorEmpty() -> context.getString(R.string.msg_preview_photo)
-            attachment.videoUrl.isNotNullNorEmpty() -> context.getString(R.string.msg_preview_video)
-            attachment.audioUrl.isNotNullNorEmpty() -> context.getString(R.string.msg_preview_audio)
-            attachment.titleLink.isNotNullNorEmpty() &&
-                    attachment.type?.contentEquals("file") == true ->
-                context.getString(R.string.msg_preview_file)
-            else -> text
-        }
-    } else {
-        text
+fun mapAttachmentText(text: String?, attachment: Attachment?, context: Context): String?  = attachment?.run {
+    when {
+        imageUrl.isNotNullNorEmpty() -> context.getString(R.string.msg_preview_photo)
+        videoUrl.isNotNullNorEmpty() -> context.getString(R.string.msg_preview_video)
+        audioUrl.isNotNullNorEmpty() -> context.getString(R.string.msg_preview_audio)
+        titleLink.isNotNullNorEmpty() &&
+                type?.contentEquals("file") == true ->
+            context.getString(R.string.msg_preview_file)
+        else -> text
     }
-}
+} ?: text
\ No newline at end of file

app/src/main/java/chat/rocket/android/favoritemessages/presentation/FavoriteMessagesPresenter.kt

@@ -36,8 +36,7 @@ class FavoriteMessagesPresenter @Inject constructor(
             try {
                 view.showLoading()
                 dbManager.getRoom(roomId)?.let {
-                    val favoriteMessages =
-                            client.getFavoriteMessages(roomId, roomTypeOf(it.chatRoom.type), offset)
+                    val favoriteMessages = client.getFavoriteMessages(roomId, roomTypeOf(it.chatRoom.type), offset)
                     val messageList = mapper.map(favoriteMessages.result, asNotReversed = true)
                     view.showFavoriteMessages(messageList)
                     offset += 1 * 30

app/src/main/java/chat/rocket/android/favoritemessages/ui/FavoriteMessagesFragment.kt

@@ -25,11 +25,9 @@ import dagger.android.support.AndroidSupportInjection
 import kotlinx.android.synthetic.main.fragment_favorite_messages.*
 import javax.inject.Inject
 
-fun newInstance(chatRoomId: String): Fragment {
-    return FavoriteMessagesFragment().apply {
-        arguments = Bundle(1).apply {
-            putString(INTENT_CHAT_ROOM_ID, chatRoomId)
-        }
+fun newInstance(chatRoomId: String): Fragment = FavoriteMessagesFragment().apply {
+    arguments = Bundle(1).apply {
+        putString(INTENT_CHAT_ROOM_ID, chatRoomId)
     }
 }
 
@@ -48,12 +46,9 @@ class FavoriteMessagesFragment : Fragment(), FavoriteMessagesView {
         super.onCreate(savedInstanceState)
         AndroidSupportInjection.inject(this)
 
-        val bundle = arguments
-        if (bundle != null) {
-            chatRoomId = bundle.getString(INTENT_CHAT_ROOM_ID)
-        } else {
-            requireNotNull(bundle) { "no arguments supplied when the fragment was instantiated" }
-        }
+        arguments?.run {
+            chatRoomId = getString(INTENT_CHAT_ROOM_ID, "")
+        } ?: requireNotNull(arguments) { "no arguments supplied when the fragment was instantiated" }
     }
 
     override fun onCreateView(
@@ -115,6 +110,6 @@ class FavoriteMessagesFragment : Fragment(), FavoriteMessagesView {
     }
 
     private fun setupToolbar() {
-        (activity as ChatRoomActivity).showToolbarTitle(getString(R.string.title_favorite_messages))
+        (activity as ChatRoomActivity).setupToolbarTitle(getString(R.string.title_favorite_messages))
     }
 }
\ No newline at end of file

app/src/main/java/chat/rocket/android/files/ui/FilesFragment.kt

@@ -30,11 +30,9 @@ import dagger.android.support.AndroidSupportInjection
 import kotlinx.android.synthetic.main.fragment_files.*
 import javax.inject.Inject
 
-fun newInstance(chatRoomId: String): Fragment {
-    return FilesFragment().apply {
-        arguments = Bundle(1).apply {
-            putString(BUNDLE_CHAT_ROOM_ID, chatRoomId)
-        }
+fun newInstance(chatRoomId: String): Fragment = FilesFragment().apply {
+    arguments = Bundle(1).apply {
+        putString(BUNDLE_CHAT_ROOM_ID, chatRoomId)
     }
 }
 
@@ -55,12 +53,9 @@ class FilesFragment : Fragment(), FilesView {
         super.onCreate(savedInstanceState)
         AndroidSupportInjection.inject(this)
 
-        val bundle = arguments
-        if (bundle != null) {
-            chatRoomId = bundle.getString(BUNDLE_CHAT_ROOM_ID)
-        } else {
-            requireNotNull(bundle) { "no arguments supplied when the fragment was instantiated" }
-        }
+        arguments?.run {
+            chatRoomId = getString(BUNDLE_CHAT_ROOM_ID, "")
+        } ?: requireNotNull(arguments) { "no arguments supplied when the fragment was instantiated" }
     }
 
     override fun onCreateView(
@@ -152,7 +147,7 @@ class FilesFragment : Fragment(), FilesView {
     }
 
     private fun setupToolbar(totalFiles: Long) {
-        (activity as ChatRoomActivity).showToolbarTitle(
+        (activity as ChatRoomActivity).setupToolbarTitle(
             (getString(
                 R.string.title_files_total,
                 totalFiles

app/src/main/java/chat/rocket/android/helper/AndroidPermissionsHelper.kt

@@ -11,8 +11,10 @@ object AndroidPermissionsHelper {
     const val WRITE_EXTERNAL_STORAGE_CODE = 1
 
     fun checkPermission(context: Context, permission: String): Boolean {
-        return ContextCompat.checkSelfPermission(context, permission) ==
-            PackageManager.PERMISSION_GRANTED
+        return ContextCompat.checkSelfPermission(
+            context,
+            permission
+        ) == PackageManager.PERMISSION_GRANTED
     }
 
     fun requestPermission(context: Activity, permission: String, requestCode: Int) {

app/src/main/java/chat/rocket/android/helper/ImageHelper.kt

@@ -53,8 +53,8 @@ object ImageHelper {
         )
         val toolbar = Toolbar(context).also {
             it.inflateMenu(R.menu.image_actions)
-            it.setOnMenuItemClickListener {
-                return@setOnMenuItemClickListener when (it.itemId) {
+            it.setOnMenuItemClickListener { view ->
+                return@setOnMenuItemClickListener when (view.itemId) {
                     R.id.action_save_image -> saveImage(context)
                     else -> true
                 }
@@ -62,20 +62,24 @@ object ImageHelper {
 
             val titleSize = context.resources
                 .getDimensionPixelSize(R.dimen.viewer_toolbar_title)
-            val titleTextView = TextView(context).also {
-                it.text = imageName
-                it.setTextColor(Color.WHITE)
-                it.setTextSize(TypedValue.COMPLEX_UNIT_PX, titleSize.toFloat())
-                it.ellipsize = TextUtils.TruncateAt.END
-                it.setSingleLine()
-                it.typeface = Typeface.DEFAULT_BOLD
-                it.setPadding(pad)
+            val titleTextView = TextView(context).also { tv ->
+                with(tv) {
+                    text = imageName
+                    setTextColor(Color.WHITE)
+                    setTextSize(TypedValue.COMPLEX_UNIT_PX, titleSize.toFloat())
+                    ellipsize = TextUtils.TruncateAt.END
+                    setSingleLine()
+                    typeface = Typeface.DEFAULT_BOLD
+                    setPadding(pad)
+                }
             }
 
-            val backArrowView = ImageView(context).also {
-                it.setImageResource(R.drawable.ic_arrow_back_white_24dp)
-                it.setOnClickListener { imageViewer?.onDismiss() }
-                it.setPadding(0, pad, pad, pad)
+            val backArrowView = ImageView(context).also { imgView ->
+                with(imgView) {
+                    setImageResource(R.drawable.ic_arrow_back_white_24dp)
+                    setOnClickListener { imageViewer?.onDismiss() }
+                    setPadding(0, pad, pad, pad)
+                }
             }
 
             val layoutParams = AppBarLayout.LayoutParams(
@@ -88,14 +92,16 @@ object ImageHelper {
         }
 
         val appBarLayout = AppBarLayout(context).also {
-            it.layoutParams = lparams
-            it.setBackgroundColor(Color.BLACK)
-            it.addView(
-                toolbar, AppBarLayout.LayoutParams(
-                    AppBarLayout.LayoutParams.MATCH_PARENT,
-                    ViewGroup.LayoutParams.WRAP_CONTENT
+            with(it) {
+                layoutParams = lparams
+                setBackgroundColor(Color.BLACK)
+                addView(
+                        toolbar, AppBarLayout.LayoutParams(
+                        AppBarLayout.LayoutParams.MATCH_PARENT,
+                        ViewGroup.LayoutParams.WRAP_CONTENT
                 )
-            )
+                )
+            }
         }
 
         val builder = ImageViewer.createPipelineDraweeControllerBuilder()

app/src/main/java/chat/rocket/android/helper/JitsiHelper.kt

+package chat.rocket.android.helper
+
+object JitsiHelper {
+
+    /**
+     * Returns the for the Jitsi video conferencing URL.
+     *
+     * @param isSecureProtocol True if using SSL, false otherwise - from the public settings.
+     * @param domain The Jitsi domain - from public settings.
+     * @param prefix The Jitsi prefix - from public settings.
+     * @param uniqueIdentifier The server unique identifier - from public settings.
+     * @param chatRoomId The ChatRoom ID where the video conferencing was started.
+     */
+    fun getJitsiUrl(
+        isSecureProtocol: Boolean,
+        domain: String?,
+        prefix: String?,
+        uniqueIdentifier: String?,
+        chatRoomId: String?
+    ): String =
+        getJitsiProtocol(isSecureProtocol) +
+                domain +
+                "/" +
+                prefix +
+                uniqueIdentifier +
+                chatRoomId
+
+    private fun getJitsiProtocol(isSecureProtocol: Boolean) =
+        if (isSecureProtocol) "https://" else "http://"
+}
\ No newline at end of file

app/src/main/java/chat/rocket/android/main/di/MainModule.kt

@@ -8,7 +8,7 @@ import chat.rocket.android.main.presentation.MainView
 import chat.rocket.android.main.ui.MainActivity
 import dagger.Module
 import dagger.Provides
-import kotlinx.coroutines.experimental.Job
+import kotlinx.coroutines.Job
 
 @Module
 class MainModule {

app/src/main/java/chat/rocket/android/main/presentation/MainPresenter.kt

@@ -37,7 +37,7 @@ import chat.rocket.core.RocketChatClient
 import chat.rocket.core.internal.rest.getCustomEmojis
 import chat.rocket.core.internal.rest.me
 import chat.rocket.core.model.Myself
-import kotlinx.coroutines.experimental.channels.Channel
+import kotlinx.coroutines.channels.Channel
 import timber.log.Timber
 import javax.inject.Inject
 

app/src/main/java/chat/rocket/android/members/adapter/MembersAdapter.kt

@@ -10,8 +10,9 @@ import chat.rocket.android.util.extensions.inflate
 import kotlinx.android.synthetic.main.avatar.view.*
 import kotlinx.android.synthetic.main.item_member.view.*
 
-class MembersAdapter(private val listener: (MemberUiModel) -> Unit) :
-    RecyclerView.Adapter<MembersAdapter.ViewHolder>() {
+class MembersAdapter(
+    private val listener: (MemberUiModel) -> Unit
+) : RecyclerView.Adapter<MembersAdapter.ViewHolder>() {
     private var dataSet: List<MemberUiModel> = ArrayList()
 
     override fun onCreateViewHolder(parent: ViewGroup, viewType: Int): MembersAdapter.ViewHolder =
@@ -43,7 +44,8 @@ class MembersAdapter(private val listener: (MemberUiModel) -> Unit) :
         fun bind(memberUiModel: MemberUiModel, listener: (MemberUiModel) -> Unit) = with(itemView) {
             image_avatar.setImageURI(memberUiModel.avatarUri)
             text_member.content = memberUiModel.displayName
-            text_member.setCompoundDrawablesRelativeWithIntrinsicBounds(DrawableHelper.getUserStatusDrawable(memberUiModel.status, context), null, null, null)
+            text_member.setCompoundDrawablesRelativeWithIntrinsicBounds(
+                    DrawableHelper.getUserStatusDrawable(memberUiModel.status, context), null, null, null)
             setOnClickListener { listener(memberUiModel) }
         }
     }

app/src/main/java/chat/rocket/android/members/presentation/MembersPresenter.kt

@@ -3,6 +3,7 @@ package chat.rocket.android.members.presentation
 import chat.rocket.android.chatroom.presentation.ChatRoomNavigator
 import chat.rocket.android.core.lifecycle.CancelStrategy
 import chat.rocket.android.db.DatabaseManager
+import chat.rocket.android.helper.UserHelper
 import chat.rocket.android.members.uimodel.MemberUiModel
 import chat.rocket.android.members.uimodel.MemberUiModelMapper
 import chat.rocket.android.server.infraestructure.RocketChatClientFactory
@@ -23,7 +24,8 @@ class MembersPresenter @Inject constructor(
     @Named("currentServer") private val currentServer: String,
     private val strategy: CancelStrategy,
     private val mapper: MemberUiModelMapper,
-    val factory: RocketChatClientFactory
+    val factory: RocketChatClientFactory,
+    private val userHelper: UserHelper
 ) {
     private val client: RocketChatClient = factory.create(currentServer)
     private var offset: Long = 0
@@ -39,7 +41,7 @@ class MembersPresenter @Inject constructor(
                 view.showLoading()
                 dbManager.getRoom(roomId)?.let {
                     val members =
-                            client.getMembers(roomId, roomTypeOf(it.chatRoom.type), offset, 60)
+                        client.getMembers(roomId, roomTypeOf(it.chatRoom.type), offset, 60)
                     val memberUiModels = mapper.mapToUiModelList(members.result)
                     view.showMembers(memberUiModels, members.total)
                     offset += 1 * 60L
@@ -59,6 +61,10 @@ class MembersPresenter @Inject constructor(
     }
 
     fun toMemberDetails(memberUiModel: MemberUiModel) {
-        navigator.toMemberDetails(memberUiModel.userId)
+        with(memberUiModel) {
+            if (userId != userHelper.user()?.id) {
+                navigator.toMemberDetails(userId)
+            }
+        }
     }
 }

app/src/main/java/chat/rocket/android/members/ui/MembersFragment.kt

@@ -27,11 +27,9 @@ import kotlinx.android.synthetic.main.app_bar_chat_room.*
 import kotlinx.android.synthetic.main.fragment_members.*
 import javax.inject.Inject
 
-fun newInstance(chatRoomId: String): Fragment {
-    return MembersFragment().apply {
-        arguments = Bundle(1).apply {
-            putString(BUNDLE_CHAT_ROOM_ID, chatRoomId)
-        }
+fun newInstance(chatRoomId: String): Fragment = MembersFragment().apply {
+    arguments = Bundle(1).apply {
+        putString(BUNDLE_CHAT_ROOM_ID, chatRoomId)
     }
 }
 
@@ -52,12 +50,9 @@ class MembersFragment : Fragment(), MembersView {
         super.onCreate(savedInstanceState)
         AndroidSupportInjection.inject(this)
 
-        val bundle = arguments
-        if (bundle != null) {
-            chatRoomId = bundle.getString(BUNDLE_CHAT_ROOM_ID)
-        } else {
-            requireNotNull(bundle) { "no arguments supplied when the fragment was instantiated" }
-        }
+        arguments?.run {
+            chatRoomId = getString(BUNDLE_CHAT_ROOM_ID, "")
+        } ?: requireNotNull(arguments) { "no arguments supplied when the fragment was instantiated" }
     }
 
     override fun onCreateView(
@@ -80,7 +75,7 @@ class MembersFragment : Fragment(), MembersView {
             setupToolbar(total)
             if (adapter.itemCount == 0) {
                 adapter.prependData(dataSet)
-                if (dataSet.size >= 59) { // TODO Check why the API retorns the specified count -1
+                if (dataSet.size >= 59) { // TODO Check why the API returns the specified count -1
                     recycler_view.addOnScrollListener(object :
                         EndlessRecyclerViewScrollListener(linearLayoutManager) {
                         override fun onLoadMore(
@@ -136,9 +131,9 @@ class MembersFragment : Fragment(), MembersView {
     private fun setupToolbar(totalMembers: Long? = null) {
         with((activity as ChatRoomActivity)) {
             if (totalMembers != null) {
-                showToolbarTitle((getString(R.string.title_counted_members, totalMembers)))
+                setupToolbarTitle((getString(R.string.title_counted_members, totalMembers)))
             } else {
-                showToolbarTitle((getString(R.string.title_members)))
+                setupToolbarTitle((getString(R.string.title_members)))
             }
             this.clearLightStatusBar()
             toolbar.isVisible = true

app/src/main/java/chat/rocket/android/mentions/ui/MentionsFragment.kt

@@ -25,11 +25,9 @@ import dagger.android.support.AndroidSupportInjection
 import kotlinx.android.synthetic.main.fragment_mentions.*
 import javax.inject.Inject
 
-fun newInstance(chatRoomId: String): Fragment {
-    return MentionsFragment().apply {
-        arguments = Bundle(1).apply {
-            putString(BUNDLE_CHAT_ROOM_ID, chatRoomId)
-        }
+fun newInstance(chatRoomId: String): Fragment = MentionsFragment().apply {
+    arguments = Bundle(1).apply {
+        putString(BUNDLE_CHAT_ROOM_ID, chatRoomId)
     }
 }
 
@@ -48,12 +46,9 @@ class MentionsFragment : Fragment(), MentionsView {
         super.onCreate(savedInstanceState)
         AndroidSupportInjection.inject(this)
 
-        val bundle = arguments
-        if (bundle != null) {
-            chatRoomId = bundle.getString(BUNDLE_CHAT_ROOM_ID)
-        } else {
-            requireNotNull(bundle) { "no arguments supplied when the fragment was instantiated" }
-        }
+        arguments?.run {
+            chatRoomId = getString(BUNDLE_CHAT_ROOM_ID, "")
+        } ?: requireNotNull(arguments) { "no arguments supplied when the fragment was instantiated" }
     }
 
     override fun onCreateView(
@@ -121,6 +116,6 @@ class MentionsFragment : Fragment(), MentionsView {
     }
 
     private fun setupToolbar() {
-        (activity as ChatRoomActivity).showToolbarTitle((getString(R.string.msg_mentions)))
+        (activity as ChatRoomActivity).setupToolbarTitle((getString(R.string.msg_mentions)))
     }
 }
\ No newline at end of file

app/src/main/java/chat/rocket/android/pinnedmessages/ui/PinnedMessagesFragment.kt

@@ -25,11 +25,9 @@ import dagger.android.support.AndroidSupportInjection
 import kotlinx.android.synthetic.main.fragment_pinned_messages.*
 import javax.inject.Inject
 
-fun newInstance(chatRoomId: String): Fragment {
-    return PinnedMessagesFragment().apply {
-        arguments = Bundle(1).apply {
-            putString(BUNDLE_CHAT_ROOM_ID, chatRoomId)
-        }
+fun newInstance(chatRoomId: String): Fragment = PinnedMessagesFragment().apply {
+    arguments = Bundle(1).apply {
+        putString(BUNDLE_CHAT_ROOM_ID, chatRoomId)
     }
 }
 
@@ -48,12 +46,9 @@ class PinnedMessagesFragment : Fragment(), PinnedMessagesView {
         super.onCreate(savedInstanceState)
         AndroidSupportInjection.inject(this)
 
-        val bundle = arguments
-        if (bundle != null) {
-            chatRoomId = bundle.getString(BUNDLE_CHAT_ROOM_ID)
-        } else {
-            requireNotNull(bundle) { "no arguments supplied when the fragment was instantiated" }
-        }
+        arguments?.run {
+            chatRoomId = getString(BUNDLE_CHAT_ROOM_ID, "")
+        } ?: requireNotNull(arguments) { "no arguments supplied when the fragment was instantiated" }
     }
 
     override fun onCreateView(
@@ -121,6 +116,6 @@ class PinnedMessagesFragment : Fragment(), PinnedMessagesView {
     }
 
     private fun setupToolbar() {
-        (activity as ChatRoomActivity).showToolbarTitle((getString(R.string.title_pinned_messages)))
+        (activity as ChatRoomActivity).setupToolbarTitle((getString(R.string.title_pinned_messages)))
     }
 }
\ No newline at end of file

app/src/main/java/chat/rocket/android/profile/presentation/ProfilePresenter.kt

@@ -27,8 +27,8 @@ import chat.rocket.core.internal.rest.deleteOwnAccount
 import chat.rocket.core.internal.rest.resetAvatar
 import chat.rocket.core.internal.rest.setAvatar
 import chat.rocket.core.internal.rest.updateProfile
-import kotlinx.coroutines.experimental.DefaultDispatcher
-import kotlinx.coroutines.experimental.withContext
+import kotlinx.coroutines.Dispatchers
+import kotlinx.coroutines.withContext
 import java.util.*
 import javax.inject.Inject
 
@@ -180,7 +180,7 @@ class ProfilePresenter @Inject constructor(
         launchUI(strategy) {
             view.showLoading()
             try {
-                withContext(DefaultDispatcher) {
+                withContext(Dispatchers.Default) {
                     // REMARK: Backend API is only working with a lowercase hash.
                     // https://github.com/RocketChat/Rocket.Chat/issues/12573
                     retryIO { client.deleteOwnAccount(password.gethash().toHex().toLowerCase()) }

app/src/main/java/chat/rocket/android/profile/ui/ProfileFragment.kt

@@ -94,11 +94,13 @@ class ProfileFragment : Fragment(), ProfileView, ActionMode.Callback {
     }
 
     override fun onActivityResult(requestCode: Int, resultCode: Int, resultData: Intent?) {
-        if (resultData != null && resultCode == Activity.RESULT_OK) {
-            if (requestCode == REQUEST_CODE_FOR_PERFORM_SAF) {
-                presenter.updateAvatar(resultData.data)
-            } else if (requestCode == REQUEST_CODE_FOR_PERFORM_CAMERA) {
-                presenter.preparePhotoAndUpdateAvatar(resultData.extras["data"] as Bitmap)
+        resultData?.run {
+            if (resultCode == Activity.RESULT_OK) {
+                if (requestCode == REQUEST_CODE_FOR_PERFORM_SAF) {
+                    data?.let { presenter.updateAvatar(it) }
+                } else if (requestCode == REQUEST_CODE_FOR_PERFORM_CAMERA) {
+                    extras?.get("data")?.let { presenter.preparePhotoAndUpdateAvatar(it as Bitmap) }
+                }
             }
         }
     }
@@ -203,8 +205,7 @@ class ProfileFragment : Fragment(), ProfileView, ActionMode.Callback {
     }
 
     private fun setupToolbar() {
-        (activity as AppCompatActivity?)?.supportActionBar?.title =
-                getString(R.string.title_profile)
+        (activity as AppCompatActivity?)?.supportActionBar?.title = getString(R.string.title_profile)
     }
 
     private fun setupListeners() {

app/src/main/java/chat/rocket/android/push/DirectReplyReceiver.kt

@@ -11,8 +11,8 @@ import chat.rocket.android.server.infraestructure.ConnectionManagerFactory
 import chat.rocket.common.RocketChatException
 import chat.rocket.core.internal.rest.sendMessage
 import dagger.android.AndroidInjection
-import kotlinx.coroutines.experimental.android.UI
-import kotlinx.coroutines.experimental.launch
+import kotlinx.coroutines.MainScope
+import kotlinx.coroutines.launch
 import timber.log.Timber
 import java.util.*
 import javax.inject.Inject
@@ -36,7 +36,7 @@ class DirectReplyReceiver : BroadcastReceiver() {
         if (ACTION_REPLY == intent.action) {
             val message = intent.getParcelableExtra<PushMessage>(EXTRA_PUSH_MESSAGE)
             message?.let {
-                launch(UI) {
+                MainScope().launch {
                     val notificationId = it.notificationId.toInt()
                     val hostname = it.info.host
                     try {

app/src/main/java/chat/rocket/android/push/PushManager.kt

@@ -12,13 +12,14 @@ import android.os.Build
 import android.os.Bundle
 import android.os.Parcel
 import android.os.Parcelable
+import android.text.Html
+import android.text.Spanned
 import androidx.annotation.RequiresApi
 import androidx.core.app.NotificationCompat
 import androidx.core.app.NotificationManagerCompat
 import androidx.core.app.RemoteInput
-import android.text.Html
-import android.text.Spanned
 import androidx.core.content.ContextCompat
+import androidx.core.text.HtmlCompat.FROM_HTML_MODE_LEGACY
 import chat.rocket.android.R
 import chat.rocket.android.main.ui.MainActivity
 import chat.rocket.android.server.domain.GetAccountInteractor
@@ -29,7 +30,7 @@ import chat.rocket.common.model.RoomType
 import chat.rocket.common.model.roomTypeOf
 import com.squareup.moshi.Json
 import com.squareup.moshi.Moshi
-import kotlinx.coroutines.experimental.runBlocking
+import kotlinx.coroutines.runBlocking
 import se.ansman.kotshi.JsonSerializable
 import se.ansman.kotshi.KotshiConstructor
 import timber.log.Timber
@@ -303,7 +304,11 @@ class PushManager @Inject constructor(
 
     // CharSequence extensions
     private fun CharSequence.fromHtml(): Spanned {
-        return Html.fromHtml(this as String)
+        return if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.N) {
+            Html.fromHtml(this as String, FROM_HTML_MODE_LEGACY, null, null)
+        } else {
+            Html.fromHtml(this as String)
+        }
     }
 
     // NotificationCompat.Builder extensions
@@ -383,12 +388,12 @@ data class PushMessage(
 ) : Parcelable {
 
     constructor(parcel: Parcel) : this(
+        parcel.readString().orEmpty(),
+        parcel.readString().orEmpty(),
+        parcel.readParcelable(PushMessage::class.java.classLoader) ?: PushInfo.EMPTY,
         parcel.readString(),
         parcel.readString(),
-        parcel.readParcelable(PushMessage::class.java.classLoader),
-        parcel.readString(),
-        parcel.readString(),
-        parcel.readString(),
+        parcel.readString().orEmpty(),
         parcel.readString(),
         parcel.readString())
 
@@ -433,9 +438,9 @@ data class PushInfo @KotshiConstructor constructor(
     }
 
     constructor(parcel: Parcel) : this(
-        parcel.readString(),
-        parcel.readString(),
-        roomTypeOf(parcel.readString()),
+        parcel.readString().orEmpty(),
+        parcel.readString().orEmpty(),
+        roomTypeOf(parcel.readString().orEmpty()),
         parcel.readString(),
         parcel.readParcelable(PushInfo::class.java.classLoader))
 
@@ -481,7 +486,7 @@ data class PushSender @KotshiConstructor constructor(
     val name: String?
 ) : Parcelable {
     constructor(parcel: Parcel) : this(
-        parcel.readString(),
+        parcel.readString().orEmpty(),
         parcel.readString(),
         parcel.readString())
 

app/src/main/java/chat/rocket/android/server/di/ChangeServerModule.kt

@@ -3,13 +3,12 @@ package chat.rocket.android.server.di
 import androidx.lifecycle.LifecycleOwner
 import chat.rocket.android.core.lifecycle.CancelStrategy
 import chat.rocket.android.dagger.scope.PerActivity
-import chat.rocket.android.dagger.scope.PerFragment
 import chat.rocket.android.server.presentation.ChangeServerNavigator
 import chat.rocket.android.server.presentation.ChangeServerView
 import chat.rocket.android.server.ui.ChangeServerActivity
 import dagger.Module
 import dagger.Provides
-import kotlinx.coroutines.experimental.Job
+import kotlinx.coroutines.Job
 
 @Module
 class ChangeServerModule {

app/src/main/java/chat/rocket/android/server/domain/ChatRoomsInteractor.kt

 package chat.rocket.android.server.domain
 
 import chat.rocket.core.model.ChatRoom
-import kotlinx.coroutines.experimental.CommonPool
-import kotlinx.coroutines.experimental.withContext
+import kotlinx.coroutines.Dispatchers
+import kotlinx.coroutines.withContext
 import javax.inject.Inject
 
 class ChatRoomsInteractor @Inject constructor(private val repository: ChatRoomsRepository) {
@@ -23,15 +23,16 @@ class ChatRoomsInteractor @Inject constructor(private val repository: ChatRoomsR
      * @param name The name of chat room to look for or a chat room that contains this name.
      * @return A list of ChatRoom objects with the given name.
      */
-    suspend fun getAllByName(url: String, name: String): List<ChatRoom> = withContext(CommonPool) {
-        val allChatRooms = repository.get(url)
-        if (name.isEmpty()) {
-            return@withContext allChatRooms
+    suspend fun getAllByName(url: String, name: String): List<ChatRoom> =
+        withContext(Dispatchers.IO) {
+            val allChatRooms = repository.get(url)
+            if (name.isEmpty()) {
+                return@withContext allChatRooms
+            }
+            return@withContext allChatRooms.filter {
+                it.name.contains(name, true)
+            }
         }
-        return@withContext allChatRooms.filter {
-            it.name.contains(name, true)
-        }
-    }
 
     /**
      * Get a specific [ChatRoom] by its id.
@@ -40,11 +41,12 @@ class ChatRoomsInteractor @Inject constructor(private val repository: ChatRoomsR
      * @param roomId The id of the room to get.
      * @return The [ChatRoom] object or null if we couldn't find any.
      */
-    suspend fun getById(serverUrl: String, roomId: String): ChatRoom? = withContext(CommonPool) {
-        return@withContext repository.get(serverUrl).find {
-            it.id == roomId
+    suspend fun getById(serverUrl: String, roomId: String): ChatRoom? =
+        withContext(Dispatchers.IO) {
+            return@withContext repository.get(serverUrl).find {
+                it.id == roomId
+            }
         }
-    }
 
     /**
      * Get a specific [ChatRoom] by its name.

app/src/main/java/chat/rocket/android/server/domain/RefreshPermissionsInteractor.kt

@@ -3,8 +3,9 @@ package chat.rocket.android.server.domain
 import chat.rocket.android.server.infraestructure.RocketChatClientFactory
 import chat.rocket.android.util.retryIO
 import chat.rocket.core.internal.rest.permissions
-import kotlinx.coroutines.experimental.CommonPool
-import kotlinx.coroutines.experimental.launch
+import kotlinx.coroutines.Dispatchers
+import kotlinx.coroutines.GlobalScope
+import kotlinx.coroutines.launch
 import timber.log.Timber
 import javax.inject.Inject
 
@@ -17,7 +18,7 @@ class RefreshPermissionsInteractor @Inject constructor(
 ) {
 
     fun refreshAsync(server: String) {
-        launch(CommonPool) {
+        GlobalScope.launch(Dispatchers.IO) {
             try {
                 factory.create(server).let { client ->
                     val permissions = retryIO(

app/src/main/java/chat/rocket/android/server/domain/RefreshSettingsInteractor.kt

@@ -3,9 +3,10 @@ package chat.rocket.android.server.domain
 import chat.rocket.android.server.infraestructure.RocketChatClientFactory
 import chat.rocket.android.util.retryIO
 import chat.rocket.core.internal.rest.settings
-import kotlinx.coroutines.experimental.CommonPool
-import kotlinx.coroutines.experimental.launch
-import kotlinx.coroutines.experimental.withContext
+import kotlinx.coroutines.Dispatchers
+import kotlinx.coroutines.GlobalScope
+import kotlinx.coroutines.launch
+import kotlinx.coroutines.withContext
 import timber.log.Timber
 import javax.inject.Inject
 
@@ -18,10 +19,11 @@ class RefreshSettingsInteractor @Inject constructor(
 ) {
 
     private var settingsFilter = arrayOf(
+        UNIQUE_IDENTIFIER,
+
         LDAP_ENABLE,
         CAS_ENABLE,
         CAS_LOGIN_URL,
-
         ACCOUNT_REGISTRATION,
         ACCOUNT_LOGIN_FORM,
         ACCOUNT_PASSWORD_RESET,
@@ -37,6 +39,12 @@ class RefreshSettingsInteractor @Inject constructor(
         ACCOUNT_WORDPRESS,
         ACCOUNT_WORDPRESS_URL,
 
+        JITSI_ENABLED,
+        JISTI_ENABLE_CHANNELS,
+        JITSI_SSL,
+        JITSI_DOMAIN,
+        JITSI_URL_ROOM_PREFIX,
+
         SITE_URL,
         SITE_NAME,
         FAVICON_512,
@@ -65,7 +73,7 @@ class RefreshSettingsInteractor @Inject constructor(
     )
 
     suspend fun refresh(server: String) {
-        withContext(CommonPool) {
+        withContext(Dispatchers.IO) {
             factory.create(server).let { client ->
                 val settings = retryIO(
                     description = "settings",
@@ -81,7 +89,7 @@ class RefreshSettingsInteractor @Inject constructor(
     }
 
     fun refreshAsync(server: String) {
-        launch(CommonPool) {
+        GlobalScope.launch(Dispatchers.IO) {
             try {
                 refresh(server)
             } catch (ex: Exception) {

app/src/main/java/chat/rocket/android/server/domain/SettingsRepository.kt

@@ -5,7 +5,9 @@ import chat.rocket.core.model.Value
 
 typealias PublicSettings = Map<String, Value<Any>>
 
-// Authentication methods.
+const val UNIQUE_IDENTIFIER = "uniqueID"
+
+// Authentication methods
 const val LDAP_ENABLE = "LDAP_Enable"
 const val CAS_ENABLE = "CAS_enabled"
 const val CAS_LOGIN_URL = "CAS_login_url"
@@ -24,6 +26,13 @@ const val ACCOUNT_GITLAB_URL = "API_Gitlab_URL"
 const val ACCOUNT_WORDPRESS = "Accounts_OAuth_Wordpress"
 const val ACCOUNT_WORDPRESS_URL = "API_Wordpress_URL"
 
+// Video call
+const val JITSI_ENABLED = "Jitsi_Enabled"
+const val JISTI_ENABLE_CHANNELS = "Jisti_Enable_Channels"
+const val JITSI_SSL = "Jitsi_SSL"
+const val JITSI_DOMAIN = "Jitsi_Domain"
+const val JITSI_URL_ROOM_PREFIX = "Jitsi_URL_Room_Prefix"
+
 const val SITE_URL = "Site_Url"
 const val SITE_NAME = "Site_Name"
 const val FAVICON_196 = "Assets_favicon_192"
@@ -54,10 +63,13 @@ const val MESSAGE_READ_RECEIPT_STORE_USERS = "Message_Read_Receipt_Store_Users"
  * Extension functions for Public Settings.
  *
  * If you need to access a Setting, add a const val key above, add it to the filter on
- * ServerPresenter.kt and a extension function to access it
+ * RefreshSettingsInteractor.kt and a extension function to access it.
  */
-fun PublicSettings.isLdapAuthenticationEnabled(): Boolean = this[LDAP_ENABLE]?.value == true
 
+fun PublicSettings.uniqueIdentifier(): String? = this[UNIQUE_IDENTIFIER]?.value as String?
+
+// Authentication
+fun PublicSettings.isLdapAuthenticationEnabled(): Boolean = this[LDAP_ENABLE]?.value == true
 fun PublicSettings.isCasAuthenticationEnabled(): Boolean = this[CAS_ENABLE]?.value == true
 fun PublicSettings.casLoginUrl(): String = this[CAS_LOGIN_URL]?.value.toString()
 fun PublicSettings.isRegistrationEnabledForNewUsers(): Boolean = this[ACCOUNT_REGISTRATION]?.value == "Public"
@@ -74,6 +86,13 @@ fun PublicSettings.gitlabUrl(): String? = this[ACCOUNT_GITLAB_URL]?.value as Str
 fun PublicSettings.isWordpressAuthenticationEnabled(): Boolean = this[ACCOUNT_WORDPRESS]?.value == true
 fun PublicSettings.wordpressUrl(): String? = this[ACCOUNT_WORDPRESS_URL]?.value as String?
 
+// Video call
+fun PublicSettings.isJitsiEnabled(): Boolean = this[JITSI_ENABLED]?.value == true
+fun PublicSettings.isJitsiEnabledForChannels(): Boolean = this[JISTI_ENABLE_CHANNELS]?.value == true
+fun PublicSettings.isJitsiSSL(): Boolean = this[JITSI_SSL]?.value == true
+fun PublicSettings.jitsiDomain(): String? = this[JITSI_DOMAIN]?.value as String?
+fun PublicSettings.jitsiPrefix(): String? = this[JITSI_URL_ROOM_PREFIX]?.value as String?
+
 fun PublicSettings.useRealName(): Boolean = this[USE_REALNAME]?.value == true
 fun PublicSettings.useSpecialCharsOnRoom(): Boolean = this[ALLOW_ROOM_NAME_SPECIAL_CHARS]?.value == true
 fun PublicSettings.faviconLarge(): String? = this[FAVICON_512]?.value as String?

app/src/main/java/chat/rocket/android/server/infraestructure/ConnectionManager.kt

@@ -23,26 +23,31 @@ import chat.rocket.core.internal.rest.chatRooms
 import chat.rocket.core.model.Message
 import chat.rocket.core.model.Myself
 import chat.rocket.core.model.Room
-import kotlinx.coroutines.experimental.CommonPool
-import kotlinx.coroutines.experimental.Job
-import kotlinx.coroutines.experimental.channels.Channel
-import kotlinx.coroutines.experimental.channels.SendChannel
-import kotlinx.coroutines.experimental.channels.actor
-import kotlinx.coroutines.experimental.launch
-import kotlinx.coroutines.experimental.newSingleThreadContext
-import kotlinx.coroutines.experimental.selects.select
+import kotlinx.coroutines.CoroutineScope
+import kotlinx.coroutines.Dispatchers
+import kotlinx.coroutines.Job
+import kotlinx.coroutines.channels.Channel
+import kotlinx.coroutines.channels.SendChannel
+import kotlinx.coroutines.channels.actor
+import kotlinx.coroutines.isActive
+import kotlinx.coroutines.launch
+import kotlinx.coroutines.newSingleThreadContext
+import kotlinx.coroutines.selects.select
 import timber.log.Timber
 import java.util.concurrent.CopyOnWriteArrayList
-import kotlin.coroutines.experimental.CoroutineContext
+import kotlin.coroutines.CoroutineContext
 
 class ConnectionManager(
     internal val client: RocketChatClient,
     private val dbManager: DatabaseManager
-) {
+) : CoroutineScope {
+    private var connectJob : Job? = null
+    override val coroutineContext: CoroutineContext
+        get() = Dispatchers.IO
+
     val statusLiveData = MutableLiveData<State>()
     private val statusChannelList = CopyOnWriteArrayList<Channel<State>>()
     private val statusChannel = Channel<State>(Channel.CONFLATED)
-    private var connectJob: Job? = null
 
     private val roomMessagesChannels = LinkedHashMap<String, Channel<Message>>()
     private val userDataChannels = ArrayList<Channel<Myself>>()
@@ -60,7 +65,7 @@ class ConnectionManager(
     private val messagesContext = newSingleThreadContext("messagesContext")
 
     fun connect() {
-        if (connectJob?.isActive == true && (state !is State.Disconnected)) {
+        if (connectJob?.isActive == true && state !is State.Disconnected) {
             Timber.d("Already connected, just returning...")
             return
         }
@@ -78,32 +83,32 @@ class ConnectionManager(
                 when (status) {
                     is State.Connected -> {
                         dbManager.clearUsersStatus()
+
                         client.subscribeSubscriptions { _, id ->
                             Timber.d("Subscribed to subscriptions: $id")
                             subscriptionId = id
                         }
+
                         client.subscribeRooms { _, id ->
                             Timber.d("Subscribed to rooms: $id")
                             roomsId = id
                         }
+
                         client.subscribeUserData { _, id ->
                             Timber.d("Subscribed to the userData id: $id")
                             userDataId = id
                         }
+
                         client.subscribeActiveUsers { _, id ->
                             Timber.d("Subscribed to the activeUser id: $id")
                             activeUserId = id
                         }
 
                         resubscribeRooms()
+                        temporaryStatus?.let { client.setTemporaryStatus(it) }
 
-                        temporaryStatus?.let { status ->
-                            client.setTemporaryStatus(status)
-                        }
-                    }
-                    is State.Waiting -> {
-                        Timber.d("Connection in: ${status.seconds}")
                     }
+                    is State.Waiting -> Timber.d("Connection in: ${status.seconds}")
                 }
 
                 statusLiveData.postValue(status)
@@ -116,8 +121,9 @@ class ConnectionManager(
         }
 
         var totalBatchedUsers = 0
-        val userActor = createBatchActor<User>(activeUsersContext, parent = connectJob,
-                maxSize = 500, maxTime = 1000) { users ->
+        val userActor = createBatchActor<User>(
+            activeUsersContext, parent = connectJob, maxSize = 500, maxTime = 1000
+        ) { users ->
             totalBatchedUsers += users.size
             Timber.d("Processing Users batch: ${users.size} - $totalBatchedUsers")
 
@@ -125,8 +131,9 @@ class ConnectionManager(
             dbManager.processUsersBatch(users)
         }
 
-        val roomsActor = createBatchActor<StreamMessage<BaseRoom>>(roomsContext, parent = connectJob,
-                maxSize = 10) { batch ->
+        val roomsActor = createBatchActor<StreamMessage<BaseRoom>>(
+            roomsContext, parent = connectJob, maxSize = 10
+        ) { batch ->
             Timber.d("processing Stream batch: ${batch.size} - $batch")
             dbManager.processChatRoomsBatch(batch)
 
@@ -135,16 +142,15 @@ class ConnectionManager(
                 if (it.type == Type.Updated) {
                     if (it.data is Room) {
                         val room = it.data as Room
-                        roomsChannels[it.data.id]?.let { channel ->
-                            channel.offer(room)
-                        }
+                        roomsChannels[it.data.id]?.offer(room)
                     }
                 }
             }
         }
 
-        val messagesActor = createBatchActor<Message>(messagesContext, parent = connectJob,
-                maxSize = 100, maxTime = 500) { messages ->
+        val messagesActor = createBatchActor<Message>(
+            messagesContext, parent = connectJob, maxSize = 100, maxTime = 500
+        ) { messages ->
             Timber.d("Processing Messages batch: ${messages.size}")
             dbManager.processMessagesBatch(messages.distinctBy { it.id })
 
@@ -157,7 +163,7 @@ class ConnectionManager(
         }
 
         // stream-notify-user - ${userId}/rooms-changed
-        launch(parent = connectJob) {
+        launch {
             for (room in client.roomsChannel) {
                 Timber.d("GOT Room streamed")
                 roomsActor.send(room)
@@ -170,7 +176,7 @@ class ConnectionManager(
         }
 
         // stream-notify-user - ${userId}/subscriptions-changed
-        launch(parent = connectJob) {
+        launch {
             for (subscription in client.subscriptionsChannel) {
                 Timber.d("GOT Subscription streamed")
                 roomsActor.send(subscription)
@@ -178,7 +184,7 @@ class ConnectionManager(
         }
 
         // stream-room-messages - $roomId
-        launch(parent = connectJob) {
+        launch {
             for (message in client.messagesChannel) {
                 Timber.d("Received new Message for room ${message.roomId}")
                 messagesActor.send(message)
@@ -186,7 +192,7 @@ class ConnectionManager(
         }
 
         // userData
-        launch(parent = connectJob) {
+        launch {
             for (myself in client.userDataChannel) {
                 Timber.d("Got userData")
                 dbManager.updateSelfUser(myself)
@@ -197,7 +203,7 @@ class ConnectionManager(
         }
 
         // activeUsers
-        launch(parent = connectJob) {
+        launch {
             for (user in client.activeUsersChannel) {
                 userActor.send(user)
             }
@@ -231,7 +237,7 @@ class ConnectionManager(
     }
 
     private fun resubscribeRooms() {
-        roomMessagesChannels.toList().map { (roomId, channel) ->
+        roomMessagesChannels.toList().map { (roomId, _) ->
             client.subscribeRoomMessages(roomId) { _, id ->
                 Timber.d("Subscribed to $roomId: $id")
                 subscriptionIdMap[roomId] = id
@@ -286,16 +292,18 @@ class ConnectionManager(
         }
     }
 
-    private inline fun <T> createBatchActor(context: CoroutineContext = CommonPool,
-                                            parent: Job? = null,
-                                            maxSize: Int = 100,
-                                            maxTime: Int = 500,
-                                            crossinline block: (List<T>) -> Unit): SendChannel<T> {
-        return actor(context, parent = parent) {
+    private inline fun <T> createBatchActor(
+        context: CoroutineContext = Dispatchers.IO,
+        parent: Job? = null,
+        maxSize: Int = 100,
+        maxTime: Int = 500,
+        crossinline block: (List<T>) -> Unit
+    ): SendChannel<T> {
+        return actor(context) {
             val batch = ArrayList<T>(maxSize)
             var deadline = 0L // deadline for sending this batch to callback block
 
-            while(true) {
+            while (true) {
                 // when deadline is reached or size is exceeded, pass the batch to the callback block
                 val remainingTime = deadline - System.currentTimeMillis()
                 if (batch.isNotEmpty() && remainingTime <= 0 || batch.size >= maxSize) {

app/src/main/java/chat/rocket/android/server/infraestructure/DatabaseMessageMapper.kt

 package chat.rocket.android.server.infraestructure
 
 import chat.rocket.android.db.DatabaseManager
-import chat.rocket.android.db.model.AttachmentActionEntity
-import chat.rocket.android.db.model.AttachmentEntity
-import chat.rocket.android.db.model.FullMessage
-import chat.rocket.android.db.model.ReactionEntity
-import chat.rocket.android.db.model.UrlEntity
-import chat.rocket.android.db.model.UserEntity
+import chat.rocket.android.db.model.*
 import chat.rocket.android.util.retryDB
 import chat.rocket.common.model.SimpleRoom
 import chat.rocket.common.model.SimpleUser
@@ -14,7 +9,6 @@ import chat.rocket.core.model.Message
 import chat.rocket.core.model.Reactions
 import chat.rocket.core.model.attachment.Attachment
 import chat.rocket.core.model.attachment.Color
-import chat.rocket.core.model.attachment.DEFAULT_COLOR_STR
 import chat.rocket.core.model.attachment.Field
 import chat.rocket.core.model.attachment.actions.Action
 import chat.rocket.core.model.attachment.actions.ButtonAction
@@ -22,8 +16,8 @@ import chat.rocket.core.model.messageTypeOf
 import chat.rocket.core.model.url.Meta
 import chat.rocket.core.model.url.ParsedUrl
 import chat.rocket.core.model.url.Url
-import kotlinx.coroutines.experimental.CommonPool
-import kotlinx.coroutines.experimental.withContext
+import kotlinx.coroutines.Dispatchers
+import kotlinx.coroutines.withContext
 
 class DatabaseMessageMapper(private val dbManager: DatabaseManager) {
     suspend fun map(message: FullMessage): Message? = map(listOf(message)).firstOrNull()
@@ -58,7 +52,8 @@ class DatabaseMessageMapper(private val dbManager: DatabaseManager) {
                 val attachments = this.attachments?.let { mapAttachments(it).asReversed() }
                 val messageType = messageTypeOf(this.message.type)
 
-                list.add(Message(
+                list.add(
+                    Message(
                         id = this.message.id,
                         roomId = this.message.roomId,
                         message = this.message.message,
@@ -82,7 +77,8 @@ class DatabaseMessageMapper(private val dbManager: DatabaseManager) {
                         role = this.message.role,
                         synced = this.message.synced,
                         unread = this.message.unread
-                ))
+                    )
+                )
             }
         }
 
@@ -106,13 +102,19 @@ class DatabaseMessageMapper(private val dbManager: DatabaseManager) {
             val parsedUrl = url.hostname?.let {
                 ParsedUrl(host = it)
             }
-            val meta = if (!url.description.isNullOrEmpty() || !url.imageUrl.isNullOrEmpty() || !url.title.isNullOrEmpty()) {
-                val raw = HashMap<String, String>()
-                if (url.description != null) raw["ogDescription"] = url.description
-                if (url.title != null) raw["ogTitle"] = url.title
-                if (url.imageUrl != null) raw["ogImage"] = url.imageUrl
-                Meta(title = url.title,description = url.description, imageUrl = url.imageUrl, raw = raw)
-            } else null
+            val meta =
+                if (!url.description.isNullOrEmpty() || !url.imageUrl.isNullOrEmpty() || !url.title.isNullOrEmpty()) {
+                    val raw = HashMap<String, String>()
+                    if (url.description != null) raw["ogDescription"] = url.description
+                    if (url.title != null) raw["ogTitle"] = url.title
+                    if (url.imageUrl != null) raw["ogImage"] = url.imageUrl
+                    Meta(
+                        title = url.title,
+                        description = url.description,
+                        imageUrl = url.imageUrl,
+                        raw = raw
+                    )
+                } else null
 
             list.add(Url(url = url.url, meta = meta, parsedUrl = parsedUrl))
         }
@@ -135,7 +137,7 @@ class DatabaseMessageMapper(private val dbManager: DatabaseManager) {
         attachments.forEach { attachment ->
             with(attachment) {
                 val fields = if (hasFields) {
-                    withContext(CommonPool) {
+                    withContext(Dispatchers.IO) {
                         retryDB("getAttachmentFields(${attachment._id})") {
                             dbManager.messageDao().getAttachmentFields(attachment._id)
                         }
@@ -144,7 +146,7 @@ class DatabaseMessageMapper(private val dbManager: DatabaseManager) {
                     null
                 }
                 val actions = if (hasActions) {
-                    withContext(CommonPool) {
+                    withContext(Dispatchers.IO) {
                         retryDB("getAttachmentActions(${attachment._id})") {
                             dbManager.messageDao().getAttachmentActions(attachment._id)
                         }
@@ -152,8 +154,8 @@ class DatabaseMessageMapper(private val dbManager: DatabaseManager) {
                 } else {
                     null
                 }
-
-                val attachment = Attachment(
+                list.add(
+                    Attachment(
                         title = title,
                         type = type,
                         description = description,
@@ -179,10 +181,11 @@ class DatabaseMessageMapper(private val dbManager: DatabaseManager) {
                         authorLink = authorLink,
                         fields = fields,
                         fallback = fallback,
-                        buttonAlignment = if (actions != null && actions.isNotEmpty()) buttonAlignment ?: "vertical" else null,
+                        buttonAlignment = if (actions != null && actions.isNotEmpty()) buttonAlignment
+                            ?: "vertical" else null,
                         actions = actions
+                    )
                 )
-                list.add(attachment)
             }
         }
         return list
@@ -190,9 +193,11 @@ class DatabaseMessageMapper(private val dbManager: DatabaseManager) {
 
     private fun mapAction(action: AttachmentActionEntity): Action? {
         return when (action.type) {
-            "button" -> ButtonAction(action.type, action.text, action.url, action.isWebView,
-                    action.webViewHeightRatio, action.imageUrl, action.message,
-                    action.isMessageInChatWindow)
+            "button" -> ButtonAction(
+                action.type, action.text, action.url, action.isWebView,
+                action.webViewHeightRatio, action.imageUrl, action.message,
+                action.isMessageInChatWindow
+            )
             else -> null
         }
     }

app/src/main/java/chat/rocket/android/server/infraestructure/DatabaseMessagesRepository.kt

@@ -6,69 +6,70 @@ import chat.rocket.android.db.model.MessagesSync
 import chat.rocket.android.server.domain.MessagesRepository
 import chat.rocket.android.util.retryDB
 import chat.rocket.core.model.Message
-import kotlinx.coroutines.experimental.CommonPool
-import kotlinx.coroutines.experimental.withContext
+import kotlinx.coroutines.Dispatchers
+import kotlinx.coroutines.withContext
 
 class DatabaseMessagesRepository(
     private val dbManager: DatabaseManager,
     private val mapper: DatabaseMessageMapper
 ) : MessagesRepository {
 
-    override suspend fun getById(id: String): Message? = withContext(CommonPool) {
+    override suspend fun getById(id: String): Message? = withContext(Dispatchers.IO) {
         retryDB("getMessageById($id)") {
             dbManager.messageDao().getMessageById(id)?.let { message -> mapper.map(message) }
         }
     }
 
-    override suspend fun getByRoomId(roomId: String): List<Message> = withContext(CommonPool) {
+    override suspend fun getByRoomId(roomId: String): List<Message> = withContext(Dispatchers.IO) {
         // FIXME - investigate how to avoid this distinctBy here, since DAO is returning a lot of
         // duplicate rows (something related to our JOINS and relations on Room)
         retryDB("getMessagesByRoomId($roomId)") {
             dbManager.messageDao().getMessagesByRoomId(roomId)
-                    .distinctBy { it.message.message.id }
-                    .let { messages ->
-                        mapper.map(messages)
-                    }
+                .distinctBy { it.message.message.id }
+                .let { messages ->
+                    mapper.map(messages)
+                }
         }
     }
 
-    override suspend fun getRecentMessages(roomId: String, count: Long): List<Message> = withContext(CommonPool) {
-        retryDB("getRecentMessagesByRoomId($roomId, $count)") {
-            dbManager.messageDao().getRecentMessagesByRoomId(roomId, count)
+    override suspend fun getRecentMessages(roomId: String, count: Long): List<Message> =
+        withContext(Dispatchers.IO) {
+            retryDB("getRecentMessagesByRoomId($roomId, $count)") {
+                dbManager.messageDao().getRecentMessagesByRoomId(roomId, count)
                     .distinctBy { it.message.message.id }
                     .let { messages ->
                         mapper.map(messages)
                     }
+            }
         }
-    }
 
     override suspend fun save(message: Message) {
         dbManager.processMessagesBatch(listOf(message)).join()
     }
 
-    override suspend fun saveAll(messages: List<Message>) {
-        dbManager.processMessagesBatch(messages).join()
+    override suspend fun saveAll(newMessages: List<Message>) {
+        dbManager.processMessagesBatch(newMessages).join()
     }
 
     override suspend fun removeById(id: String) {
-        withContext(CommonPool) {
+        withContext(Dispatchers.IO) {
             retryDB("delete($id)") { dbManager.messageDao().delete(id) }
         }
     }
 
     override suspend fun removeByRoomId(roomId: String) {
-        withContext(CommonPool) {
+        withContext(Dispatchers.IO) {
             retryDB("deleteByRoomId($roomId)") {
                 dbManager.messageDao().deleteByRoomId(roomId)
             }
         }
     }
 
-    override suspend fun getAllUnsent(): List<Message> = withContext(CommonPool) {
+    override suspend fun getAllUnsent(): List<Message> = withContext(Dispatchers.IO) {
         retryDB("getUnsentMessages") {
             dbManager.messageDao().getUnsentMessages()
-                    .distinctBy { it.message.message.id }
-                    .let { mapper.map(it) }
+                .distinctBy { it.message.message.id }
+                .let { mapper.map(it) }
         }
     }
 
@@ -76,9 +77,9 @@ class DatabaseMessagesRepository(
         dbManager.sendOperation(Operation.SaveLastSync(MessagesSync(roomId, timeMillis)))
     }
 
-    override suspend fun getLastSyncDate(roomId: String): Long? = withContext(CommonPool) {
+    override suspend fun getLastSyncDate(roomId: String): Long? = withContext(Dispatchers.IO) {
         retryDB("getLastSync($roomId)") {
-            dbManager.messageDao().getLastSync(roomId)?.let { it.timestamp }
+            dbManager.messageDao().getLastSync(roomId)?.timestamp
         }
     }
 }
\ No newline at end of file

app/src/main/java/chat/rocket/android/server/infraestructure/SharedPreferencesAccountsRepository.kt

@@ -14,13 +14,8 @@ class SharedPreferencesAccountsRepository(
     private val moshi: Moshi
 ) : AccountsRepository {
 
-    override fun save(newAccount: Account) {
-        val accounts = load()
-
-        val newList = accounts.filter { account -> newAccount.serverUrl != account.serverUrl }
-            .toMutableList()
-        newList.add(0, newAccount)
-        save(newList)
+    override fun save(account: Account) {
+        save(load().filter { item -> item.serverUrl != item.serverUrl }.toMutableList().apply { add(0, account) })
     }
 
     override fun load(): List<Account> {
@@ -28,15 +23,11 @@ class SharedPreferencesAccountsRepository(
         val type = Types.newParameterizedType(List::class.java, Account::class.java)
         val adapter = moshi.adapter<List<Account>>(type)
 
-        return adapter.fromJson(json) ?: emptyList()
+        return json?.let { adapter.fromJson(it) ?: emptyList() } ?: emptyList()
     }
 
     override fun remove(serverUrl: String) {
-        val accounts = load()
-
-        val newList = accounts.filter { account -> serverUrl != account.serverUrl }
-            .toMutableList()
-        save(newList)
+        save(load().filter { account -> serverUrl != account.serverUrl }.toMutableList())
     }
 
     private fun save(accounts: List<Account>) {

app/src/main/java/chat/rocket/android/server/infraestructure/SharedPrefsBasicAuthRepository.kt

@@ -15,8 +15,7 @@ class SharedPrefsBasicAuthRepository(
 ) : BasicAuthRepository {
 
     override fun save(basicAuth: BasicAuth) {
-        val newList = load().filter { basicAuth -> basicAuth.host != basicAuth.host }
-            .toMutableList()
+        val newList = load().filter { auth -> auth.host != auth.host }.toMutableList()
         newList.add(0, basicAuth)
         save(newList)
     }
@@ -26,7 +25,7 @@ class SharedPrefsBasicAuthRepository(
         val type = Types.newParameterizedType(List::class.java, BasicAuth::class.java)
         val adapter = moshi.adapter<List<BasicAuth>>(type)
 
-        return adapter.fromJson(json) ?: emptyList()
+        return json?.let { adapter.fromJson(it) ?: emptyList() } ?: emptyList()
     }
 
     private fun save(basicAuths: List<BasicAuth>) {

app/src/main/java/chat/rocket/android/server/presentation/CheckServerPresenter.kt

@@ -45,10 +45,10 @@ import chat.rocket.core.internal.rest.serverInfo
 import chat.rocket.core.internal.rest.settingsOauth
 import chat.rocket.core.internal.rest.unregisterPushToken
 import chat.rocket.core.model.Myself
-import kotlinx.coroutines.experimental.CommonPool
-import kotlinx.coroutines.experimental.Job
-import kotlinx.coroutines.experimental.channels.Channel
-import kotlinx.coroutines.experimental.withContext
+import kotlinx.coroutines.Dispatchers
+import kotlinx.coroutines.Job
+import kotlinx.coroutines.channels.Channel
+import kotlinx.coroutines.withContext
 import timber.log.Timber
 
 private const val SERVICE_NAME_FACEBOOK = "facebook"
@@ -220,7 +220,7 @@ abstract class CheckServerPresenter constructor(
                 }
                 removeAccountInteractor?.remove(currentServer)
                 tokenRepository?.remove(currentServer)
-                withContext(CommonPool) { dbManager.logout() }
+                withContext(Dispatchers.IO) { dbManager.logout() }
                 navigator?.switchOrAddNewServer()
             } catch (ex: Exception) {
                 Timber.e(ex, "Error cleaning up the session...")

app/src/main/java/chat/rocket/android/server/ui/ChangeServerActivity.kt

@@ -26,7 +26,6 @@ fun Context.changeServerIntent(serverUrl: String? = null, chatRoomId: String? =
 
 class ChangeServerActivity : AppCompatActivity(), ChangeServerView {
     @Inject lateinit var presenter: ChangeServerPresenter
-
     var progress: ProgressDialog? = null
 
     override fun onCreate(savedInstanceState: Bundle?) {

app/src/main/java/chat/rocket/android/settings/di/SettingsFragmentModule.kt

@@ -7,7 +7,7 @@ import chat.rocket.android.settings.presentation.SettingsView
 import chat.rocket.android.settings.ui.SettingsFragment
 import dagger.Module
 import dagger.Provides
-import kotlinx.coroutines.experimental.Job
+import kotlinx.coroutines.Job
 
 @Module
 class SettingsFragmentModule {

app/src/main/java/chat/rocket/android/settings/password/di/PasswordFragmentModule.kt

@@ -7,7 +7,7 @@ import chat.rocket.android.settings.password.presentation.PasswordView
 import chat.rocket.android.settings.password.ui.PasswordFragment
 import dagger.Module
 import dagger.Provides
-import kotlinx.coroutines.experimental.Job
+import kotlinx.coroutines.Job
 
 @Module
 class PasswordFragmentModule {

app/src/main/java/chat/rocket/android/settings/password/ui/PasswordFragment.kt

@@ -106,18 +106,16 @@ class PasswordFragment : Fragment(), PasswordView, ActionMode.Callback {
 
     private fun finishActionMode() = actionMode?.finish()
 
-    private fun listenToChanges(): Disposable {
-        return Observables.combineLatest(
-            text_new_password.asObservable(),
-            text_confirm_password.asObservable()
-        ).subscribe {
-            val newPassword = it.first.toString()
-            val newPasswordConfirmation = it.second.toString()
-            if (newPassword.length > 5 && newPassword == newPasswordConfirmation) {
-                startActionMode()
-            } else {
-                finishActionMode()
-            }
+    private fun listenToChanges(): Disposable = Observables.combineLatest(
+        text_new_password.asObservable(),
+        text_confirm_password.asObservable()
+    ).subscribe {
+        val newPassword = it.first.toString()
+        val newPasswordConfirmation = it.second.toString()
+        if (newPassword.length > 5 && newPassword == newPasswordConfirmation) {
+            startActionMode()
+        } else {
+            finishActionMode()
         }
     }
 

app/src/main/java/chat/rocket/android/settings/ui/SettingsFragment.kt

@@ -115,8 +115,7 @@ class SettingsFragment : Fragment(), SettingsView, AdapterView.OnItemClickListen
     }
 
     private fun setupToolbar() {
-        (activity as AppCompatActivity?)?.supportActionBar?.title =
-                getString(R.string.title_settings)
+        (activity as AppCompatActivity?)?.supportActionBar?.title = getString(R.string.title_settings)
     }
 
     private fun shareApp() {

app/src/main/java/chat/rocket/android/userdetails/presentation/UserDetailsPresenter.kt

@@ -6,7 +6,9 @@ import chat.rocket.android.core.lifecycle.CancelStrategy
 import chat.rocket.android.db.DatabaseManager
 import chat.rocket.android.db.model.ChatRoomEntity
 import chat.rocket.android.db.model.UserEntity
-import chat.rocket.android.server.domain.GetConnectingServerInteractor
+import chat.rocket.android.server.domain.CurrentServerRepository
+import chat.rocket.android.server.domain.GetSettingsInteractor
+import chat.rocket.android.server.domain.isJitsiEnabled
 import chat.rocket.android.server.infraestructure.ConnectionManagerFactory
 import chat.rocket.android.util.extension.launchUI
 import chat.rocket.android.util.extensions.avatarUrl
@@ -14,8 +16,8 @@ import chat.rocket.android.util.retryIO
 import chat.rocket.common.model.RoomType
 import chat.rocket.common.util.ifNull
 import chat.rocket.core.internal.rest.createDirectMessage
-import kotlinx.coroutines.experimental.DefaultDispatcher
-import kotlinx.coroutines.experimental.withContext
+import kotlinx.coroutines.Dispatchers
+import kotlinx.coroutines.withContext
 import timber.log.Timber
 import javax.inject.Inject
 
@@ -24,13 +26,15 @@ class UserDetailsPresenter @Inject constructor(
     private val dbManager: DatabaseManager,
     private val strategy: CancelStrategy,
     private val navigator: ChatRoomNavigator,
-    serverInteractor: GetConnectingServerInteractor,
+    settingsInteractor: GetSettingsInteractor,
+    serverInteractor: CurrentServerRepository,
     factory: ConnectionManagerFactory
 ) {
     private var currentServer = serverInteractor.get()!!
     private val manager = factory.create(currentServer)
     private val client = manager.client
     private val interactor = FetchChatRoomsInteractor(client, dbManager)
+    private val settings = settingsInteractor.get(currentServer)
     private lateinit var userEntity: UserEntity
 
     fun loadUserDetails(userId: String) {
@@ -40,19 +44,20 @@ class UserDetailsPresenter @Inject constructor(
                 dbManager.getUser(userId)?.let {
                     userEntity = it
                     val avatarUrl =
-                        userEntity.username?.let { currentServer.avatarUrl(avatar = it) }
+                        userEntity.username?.let { username -> currentServer.avatarUrl(avatar = username) }
                     val username = userEntity.username
                     val name = userEntity.name
                     val utcOffset =
                         userEntity.utcOffset // TODO Convert UTC and display like the mockup
 
                     if (avatarUrl != null && username != null && name != null && utcOffset != null) {
-                        view.showUserDetails(
+                        view.showUserDetailsAndActions(
                             avatarUrl = avatarUrl,
                             name = name,
                             username = username,
                             status = userEntity.status,
-                            utcOffset = utcOffset.toString()
+                            utcOffset = utcOffset.toString(),
+                            isVideoCallAllowed = settings.isJitsiEnabled()
                         )
                     } else {
                         throw Exception()
@@ -76,7 +81,7 @@ class UserDetailsPresenter @Inject constructor(
             try {
                 view.showLoading()
 
-                withContext(DefaultDispatcher) {
+                withContext(Dispatchers.Default) {
                     val directMessage = retryIO("createDirectMessage($username") {
                         client.createDirectMessage(username)
                     }
@@ -118,4 +123,25 @@ class UserDetailsPresenter @Inject constructor(
             }
         }
     }
+
+    fun toVideoConference(username: String) {
+        launchUI(strategy) {
+            try {
+                withContext(Dispatchers.Default) {
+                    val directMessage = retryIO("createDirectMessage($username") {
+                        client.createDirectMessage(username)
+                    }
+                    navigator.toVideoConference(directMessage.id, RoomType.DIRECT_MESSAGE)
+                }
+            } catch (ex: Exception) {
+                Timber.e(ex)
+                ex.message?.let {
+                    view.showMessage(it)
+                }.ifNull {
+                    view.showGenericErrorMessage()
+                }
+            }
+        }
+    }
 }
+

app/src/main/java/chat/rocket/android/userdetails/presentation/UserDetailsView.kt

@@ -13,12 +13,14 @@ interface UserDetailsView : LoadingView, MessageView {
      * @param username The user's username.
      * @param status The user's status.
      * @param utcOffset The user's UTC offset.
+     * @param isVideoCallAllowed True if the video call is allowed, false otherwise.
      */
-    fun showUserDetails(
+    fun showUserDetailsAndActions(
         avatarUrl: String,
         name: String,
         username: String,
         status: String,
-        utcOffset: String
+        utcOffset: String,
+        isVideoCallAllowed: Boolean
     )
 }

app/src/main/java/chat/rocket/android/userdetails/ui/UserDetailsFragment.kt

@@ -29,11 +29,9 @@ import kotlinx.android.synthetic.main.app_bar_chat_room.*
 import kotlinx.android.synthetic.main.fragment_user_details.*
 import javax.inject.Inject
 
-fun newInstance(userId: String): Fragment {
-    return UserDetailsFragment().apply {
-        arguments = Bundle(1).apply {
-            putString(BUNDLE_USER_ID, userId)
-        }
+fun newInstance(userId: String): Fragment = UserDetailsFragment().apply {
+    arguments = Bundle(1).apply {
+        putString(BUNDLE_USER_ID, userId)
     }
 }
 
@@ -52,12 +50,10 @@ class UserDetailsFragment : Fragment(), UserDetailsView {
         super.onCreate(savedInstanceState)
         AndroidSupportInjection.inject(this)
 
-        val bundle = arguments
-        if (bundle != null) {
-            userId = bundle.getString(BUNDLE_USER_ID)
-        } else {
-            requireNotNull(bundle) { "no arguments supplied when the fragment was instantiated" }
+        arguments?.run {
+            userId = getString(BUNDLE_USER_ID, "")
         }
+            ?: requireNotNull(arguments) { "no arguments supplied when the fragment was instantiated" }
     }
 
     override fun onCreateView(
@@ -81,12 +77,13 @@ class UserDetailsFragment : Fragment(), UserDetailsView {
         super.onDestroyView()
     }
 
-    override fun showUserDetails(
+    override fun showUserDetailsAndActions(
         avatarUrl: String,
         name: String,
         username: String,
         status: String,
-        utcOffset: String
+        utcOffset: String,
+        isVideoCallAllowed: Boolean
     ) {
         val requestBuilder = Glide.with(this).load(avatarUrl)
 
@@ -104,6 +101,13 @@ class UserDetailsFragment : Fragment(), UserDetailsView {
 
         // We should also setup the user details listeners.
         text_message.setOnClickListener { presenter.createDirectMessage(username) }
+
+        if (isVideoCallAllowed) {
+            text_video_call.isVisible = true
+            text_video_call.setOnClickListener { presenter.toVideoConference(username) }
+        } else {
+            text_video_call.isVisible = false
+        }
     }
 
     override fun showLoading() {

app/src/main/java/chat/rocket/android/util/HttpLoggingInterceptor.kt

@@ -191,7 +191,7 @@ class HttpLoggingInterceptor constructor(private val logger: Logger) : Intercept
 
         val responseBody = response.body()
         val contentLength = responseBody!!.contentLength()
-        val bodySize = if (contentLength != -1L) contentLength.toString() + "-byte" else "unknown-length"
+        val bodySize = if (contentLength != -1L) "$contentLength-byte" else "unknown-length"
         val responseStr = if (response.message().isEmpty()) "" else " ${response.message()}"
         logger.log("<-- ${response.code()}$responseStr ${response.request().url()}"
                 + " (" + tookMs + "ms" + (if (!logHeaders) ", $bodySize body" else "") + ')'.toString())

app/src/main/java/chat/rocket/android/util/IO.kt

@@ -2,26 +2,25 @@ package chat.rocket.android.util
 
 import android.database.sqlite.SQLiteDatabaseLockedException
 import chat.rocket.common.RocketChatNetworkErrorException
-import kotlinx.coroutines.experimental.TimeoutCancellationException
-import kotlinx.coroutines.experimental.delay
-import kotlinx.coroutines.experimental.isActive
+import kotlinx.coroutines.delay
+import kotlinx.coroutines.isActive
 import timber.log.Timber
-import kotlin.coroutines.experimental.coroutineContext
+import kotlin.coroutines.coroutineContext
 
 const val DEFAULT_RETRY = 3
 private const val DEFAULT_DB_RETRY = 15
 
 suspend fun <T> retryIO(
-        description: String = "<missing description>",
-        times: Int = DEFAULT_RETRY,
-        initialDelay: Long = 100, // 0.1 second
-        maxDelay: Long = 1000,    // 1 second
-        factor: Double = 2.0,
-        block: suspend () -> T): T
-{
+    description: String = "<missing description>",
+    times: Int = DEFAULT_RETRY,
+    initialDelay: Long = 100, // 0.1 second
+    maxDelay: Long = 1000,    // 1 second
+    factor: Double = 2.0,
+    block: suspend () -> T
+): T {
     var currentDelay = initialDelay
     repeat(times - 1) { currentTry ->
-        if (!coroutineContext.isActive) throw TimeoutCancellationException("job canceled")
+        if (!coroutineContext.isActive) throw Exception("Job canceled when trying to execute retryIO")
         try {
             return block()
         } catch (e: RocketChatNetworkErrorException) {
@@ -29,26 +28,26 @@ suspend fun <T> retryIO(
             e.printStackTrace()
         }
 
-        if (!coroutineContext.isActive) throw TimeoutCancellationException("job canceled")
+        if (!coroutineContext.isActive) throw Exception("Job canceled when trying to execute retryIO")
         delay(currentDelay)
         currentDelay = (currentDelay * factor).toLong().coerceAtMost(maxDelay)
     }
 
-    if (!coroutineContext.isActive) throw TimeoutCancellationException("job canceled")
+    if (!coroutineContext.isActive) throw Exception("Job canceled when trying to execute retryIO")
     return block() // last attempt
 }
 
 suspend fun <T> retryDB(
-        description: String = "<missing description>",
-        times: Int = DEFAULT_DB_RETRY,
-        initialDelay: Long = 100, // 0.1 second
-        maxDelay: Long = 1500,    // 1.5 second
-        factor: Double = 1.2,
-        block: suspend () -> T): T
-{
+    description: String = "<missing description>",
+    times: Int = DEFAULT_DB_RETRY,
+    initialDelay: Long = 100, // 0.1 second
+    maxDelay: Long = 1500,    // 1.5 second
+    factor: Double = 1.2,
+    block: suspend () -> T
+): T {
     var currentDelay = initialDelay
     repeat(times - 1) { currentTry ->
-        if (!coroutineContext.isActive) throw TimeoutCancellationException("job canceled")
+        if (!coroutineContext.isActive) throw Exception("Job canceled when trying to execute retryDB")
         try {
             return block()
         } catch (e: SQLiteDatabaseLockedException) {
@@ -56,11 +55,11 @@ suspend fun <T> retryDB(
             e.printStackTrace()
         }
 
-        if (!coroutineContext.isActive) throw TimeoutCancellationException("job canceled")
+        if (!coroutineContext.isActive) throw Exception("Job canceled when trying to execute retryDB")
         delay(currentDelay)
         currentDelay = (currentDelay * factor).toLong().coerceAtMost(maxDelay)
     }
 
-    if (!coroutineContext.isActive) throw TimeoutCancellationException("job canceled")
+    if (!coroutineContext.isActive) throw Exception("Job canceled when trying to execute retryDB")
     return block() // last attempt
 }
\ No newline at end of file

app/src/main/java/chat/rocket/android/util/extensions/Fragment.kt

@@ -3,9 +3,10 @@ package chat.rocket.android.util.extensions
 import android.os.Looper
 import androidx.fragment.app.Fragment
 import androidx.fragment.app.FragmentActivity
-import kotlinx.coroutines.experimental.Job
-import kotlinx.coroutines.experimental.android.UI
-import kotlinx.coroutines.experimental.launch
+import kotlinx.coroutines.Dispatchers
+import kotlinx.coroutines.GlobalScope
+import kotlinx.coroutines.Job
+import kotlinx.coroutines.launch
 
 inline fun Fragment.ui(crossinline block: (activity: FragmentActivity) -> Unit): Job? {
     // Checking first for activity and view saves us from some synchronyzed and thread local checks
@@ -16,7 +17,7 @@ inline fun Fragment.ui(crossinline block: (activity: FragmentActivity) -> Unit):
             null
         } else {
             // Launch a Job on the UI context and check again if the activity and view are still valid
-            launch(UI) {
+            GlobalScope.launch(Dispatchers.Main) {
                 if (activity != null && view != null && context != null) {
                     block(activity!!)
                 }

app/src/main/java/chat/rocket/android/util/extensions/RocketChatClient.kt

@@ -7,15 +7,15 @@ import chat.rocket.android.util.retryIO
 import chat.rocket.core.internal.rest.registerPushToken
 import chat.rocket.core.model.Message
 import chat.rocket.core.model.asString
-import kotlinx.coroutines.experimental.CommonPool
-import kotlinx.coroutines.experimental.withContext
+import kotlinx.coroutines.Dispatchers
+import kotlinx.coroutines.withContext
 import timber.log.Timber
 
 suspend fun RocketChatClientFactory.registerPushToken(
     token: String,
     accounts: List<Account>
 ) {
-    withContext(CommonPool) {
+    withContext(Dispatchers.IO) {
         accounts.forEach { account ->
             try {
                 retryIO(description = "register push token: ${account.serverUrl}") {

app/src/main/java/chat/rocket/android/util/extensions/String.kt

@@ -69,11 +69,8 @@ fun String.userId(userId: String?): String? {
     return userId?.let { this.replace(it, "") }
 }
 
-fun String.lowercaseUrl(): String? {
-    val httpUrl = HttpUrl.parse(this)
-    val newScheme = httpUrl?.scheme()?.toLowerCase()
-
-    return httpUrl?.newBuilder()?.scheme(newScheme)?.build()?.toString()
+fun String.lowercaseUrl(): String? = HttpUrl.parse(this)?.run {
+    newBuilder().scheme(scheme().toLowerCase()).build().toString()
 }
 
 fun String?.isNotNullNorEmpty(): Boolean = this != null && this.isNotEmpty()

app/src/main/java/chat/rocket/android/util/extensions/Ui.kt

@@ -80,12 +80,10 @@ fun AppCompatActivity.toPreviousView() {
 }
 
 fun Activity.hideKeyboard() {
-    if (currentFocus != null) {
-        val imm = getSystemService(Context.INPUT_METHOD_SERVICE) as InputMethodManager
-        imm.hideSoftInputFromWindow(
-            currentFocus.windowToken,
-            InputMethodManager.RESULT_UNCHANGED_SHOWN
-        )
+    currentFocus?.run {
+        (getSystemService(Context.INPUT_METHOD_SERVICE) as InputMethodManager).also {
+            it.hideSoftInputFromWindow(windowToken, InputMethodManager.RESULT_UNCHANGED_SHOWN)
+        }
     }
 }
 

app/src/main/java/chat/rocket/android/util/extensions/Uri.kt

@@ -51,7 +51,7 @@ fun Uri.getFileSize(context: Context): Int {
 
 fun Uri.getMimeType(context: Context): String {
     return if (scheme == ContentResolver.SCHEME_CONTENT) {
-        context.contentResolver.getType(this)
+        context.contentResolver?.getType(this) ?: ""
     } else {
         val fileExtension = MimeTypeMap.getFileExtensionFromUrl(toString())
         if (fileExtension != null) {