refactor: Remove unnecessary return type from `handleProfileClick`

This commit is contained in:
Nik Clayton 2023-09-19 22:41:54 +02:00
parent 8c95472d42
commit 26ff8e5bad
1 changed files with 8 additions and 5 deletions

View File

@ -488,7 +488,10 @@ class MainActivity : BottomSheetActivity(), ActionButtonActivity, HasAndroidInje
header = AccountHeaderView(this).apply {
headerBackgroundScaleType = ImageView.ScaleType.CENTER_CROP
currentHiddenInList = true
onAccountHeaderListener = { _: View?, profile: IProfile, current: Boolean -> handleProfileClick(profile, current) }
onAccountHeaderListener = { _: View?, profile: IProfile, current: Boolean ->
handleProfileClick(profile, current)
false
}
addProfile(
ProfileSettingDrawerItem().apply {
identifier = DRAWER_ITEM_ADD_ACCOUNT
@ -854,23 +857,23 @@ class MainActivity : BottomSheetActivity(), ActionButtonActivity, HasAndroidInje
}
}
private fun handleProfileClick(profile: IProfile, current: Boolean): Boolean {
private fun handleProfileClick(profile: IProfile, current: Boolean) {
val activeAccount = accountManager.activeAccount
// open profile when active image was clicked
if (current && activeAccount != null) {
val intent = AccountActivity.getIntent(this, activeAccount.accountId)
startActivityWithSlideInAnimation(intent)
return false
return
}
// open LoginActivity to add new account
if (profile.identifier == DRAWER_ITEM_ADD_ACCOUNT) {
startActivityWithSlideInAnimation(LoginActivity.getIntent(this, LoginActivity.MODE_ADDITIONAL_LOGIN))
return false
return
}
// change Account
changeAccount(profile.identifier, null)
return false
return
}
private fun changeAccount(newSelectedId: Long, forward: Intent?) {