Don't store blank usernames.

This commit is contained in:
Greyson Parrelli 2023-08-25 14:34:54 -04:00
parent ae3071d318
commit 57b9571d86
2 changed files with 9 additions and 2 deletions

View file

@ -362,7 +362,14 @@ internal class AccountValues internal constructor(store: KeyValueStore) : Signal
get() = !isPrimaryDevice
/** The local user's full username (nickname.discriminator), if set. */
var username: String? by stringValue(KEY_USERNAME, null)
var username: String?
get() {
val value = getString(KEY_USERNAME, null)
return if (value.isNullOrBlank()) null else value
}
set(value) {
putString(KEY_USERNAME, value)
}
/** The local user's username link components, if set. */
var usernameLink: UsernameLinkComponents?

View file

@ -32,7 +32,7 @@ internal class CopyUsernameToSignalStoreMigrationJob(
val self = Recipient.self()
if (self.username.isEmpty) {
if (self.username.isEmpty || self.username.get().isBlank()) {
Log.i(TAG, "No username set, skipping.")
return
}