Merge pull request keepassxreboot#2106 from keepassxreboot/fix/dont_c…

…lear_username Do not clear the username field if the entry has no username set
async-harsh · Feb 2, 2024 · d713bf4 · d713bf4
2 parents 8bcd8b5 + b785221
commit d713bf4
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/keepassxc-browser/content/fill.js b/keepassxc-browser/content/fill.js
@@ -256,7 +256,8 @@ kpxcFill.fillInCredentials = async function(combination, predefinedUsername, uui
     }
 
     // Fill username
-    if (combination.username && (!combination.username.value || combination.username.value !== usernameValue)) {
+    if (combination.username && usernameValue &&
+        (!combination.username.value || combination.username.value !== usernameValue)) {
         if (!passOnly) {
             kpxc.setValueWithChange(combination.username, usernameValue);
         }