PM-14379: Stop storing 'null' in as the word separator (#4219)

bitwarden · Nov 1, 2024 · 9240fb8 · 9240fb8
1 parent 2eb41e9
commit 9240fb8
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/app/src/main/java/com/x8bit/bitwarden/ui/tools/feature/generator/GeneratorViewModel.kt b/app/src/main/java/com/x8bit/bitwarden/ui/tools/feature/generator/GeneratorViewModel.kt
@@ -351,7 +351,7 @@ class GeneratorViewModel @Inject constructor(
                     GeneratorState.MainType.Passphrase(
                         numWords = max(options.numWords, minNumWords),
                         minNumWords = minNumWords,
-                        wordSeparator = options.wordSeparator.toCharArray().first(),
+                        wordSeparator = options.wordSeparator.toCharArray().firstOrNull(),
                         capitalize = options.allowCapitalize || policy.capitalize == true,
                         capitalizeEnabled = policy.capitalize != true,
                         includeNumber = options.allowIncludeNumber || policy.includeNumber == true,
@@ -462,7 +462,7 @@ class GeneratorViewModel @Inject constructor(
         val newOptions = options.copy(
             type = PasscodeGenerationOptions.PasscodeType.PASSPHRASE,
             numWords = passphrase.numWords,
-            wordSeparator = passphrase.wordSeparator.toString(),
+            wordSeparator = passphrase.wordSeparator?.toString().orEmpty(),
             allowCapitalize = passphrase.capitalize,
             allowIncludeNumber = passphrase.includeNumber,
         )