Skip to content

Commit

Permalink
fix: Remove assert not null when it's not needed (#2206)
Browse files Browse the repository at this point in the history
  • Loading branch information
NicolasBourdin88 authored Feb 27, 2025
2 parents a60fa9b + a974979 commit fa84db9
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions app/src/main/java/com/infomaniak/mail/views/AvatarView.kt
Original file line number Diff line number Diff line change
Expand Up @@ -164,7 +164,7 @@ class AvatarView @JvmOverloads constructor(
loadAvatarByDisplayType(avatarDisplayType, correspondent, bimi, contactsFromViewModel)
}

private fun updateStateAndLoadUserAvatar(user: User, correspondent: Correspondent, bimi: Bimi) {
private fun updateStateAndLoadUserAvatar(user: User, correspondent: Correspondent?, bimi: Bimi?) {
state.update(correspondent, bimi)
loadUserAvatar(user)
}
Expand Down Expand Up @@ -206,7 +206,7 @@ class AvatarView @JvmOverloads constructor(
when (avatarDisplayType) {
AvatarDisplayType.UNKNOWN_CORRESPONDENT -> loadUnknownUserAvatar()
AvatarDisplayType.USER_AVATAR ->
AccountUtils.currentUser?.let { updateStateAndLoadUserAvatar(user = it, correspondent!!, bimi!!) }
AccountUtils.currentUser?.let { updateStateAndLoadUserAvatar(user = it, correspondent, bimi) }
AvatarDisplayType.CUSTOM_AVATAR,
AvatarDisplayType.INITIALS -> loadAvatarUsingDictionary(correspondent!!, contacts, bimi)
AvatarDisplayType.BIMI -> loadBimiAvatar(correspondent!!, bimi!!)
Expand Down

0 comments on commit fa84db9

Please sign in to comment.