From b10a12559b9273a5a9267583facb9a1b7395b273 Mon Sep 17 00:00:00 2001
From: Alexander Grund <Flamefire@users.noreply.github.com>
Date: Sat, 27 Apr 2024 12:41:39 +0200
Subject: [PATCH] Remove duplicate notification

The `passwordFillIsAllowed` function should only do a check. The logging is done in the caller
---
 keepassxc-browser/content/fill.js | 7 +------
 1 file changed, 1 insertion(+), 6 deletions(-)

diff --git a/keepassxc-browser/content/fill.js b/keepassxc-browser/content/fill.js
index c777f3e..a758a36 100644
--- a/keepassxc-browser/content/fill.js
+++ b/keepassxc-browser/content/fill.js
@@ -348,10 +348,5 @@ const passwordFillIsAllowed = function(elem) {
         return true;
     }
 
-    if (elem?.getLowerCaseAttribute('type') !== 'password') {
-        kpxcUI.createNotification('warning', tr('fieldsPasswordFillNotAccepted'));
-        return false;
-    }
-
-    return true;
+    return elem?.getLowerCaseAttribute('type') === 'password';
 };