Rename elementContainsText to doesElementContainText and refactor it

AdguardTeam · May 23, 2024 · a47011e · a47011e
1 parent 7fab2a8
commit a47011e
Showing 1 changed file with 7 additions and 7 deletions.
diff --git a/src/scriptlets/trusted-click-element.js b/src/scriptlets/trusted-click-element.js
@@ -228,21 +228,21 @@ export function trustedClickElement(source, selectors, extraMatch = '', delay =
         }
     }
 
-    const parsedTextToMatch = textMatches ? toRegExp(textMatches) : '';
+    const textMatchRegexp = textMatches ? toRegExp(textMatches) : null;
 
     /**
      * Checks if an element contains the specified text.
      *
      * @param {Element} element - The element to check.
-     * @param {string} textMatch - The text to match.
+     * @param {string} matchRegexp - The text to match.
      * @returns {boolean} Returns true if the element contains the specified text, otherwise false.
      */
-    const elementContainsText = (element, textMatch) => {
-        if (!element || !textMatch) {
+    const doesElementContainText = (element, matchRegexp) => {
+        const { textContent } = element;
+        if (!textContent) {
             return false;
         }
-        const elementText = element.textContent;
-        return parsedTextToMatch.test(elementText);
+        return matchRegexp.test(textContent);
     };
 
     /**
@@ -285,7 +285,7 @@ export function trustedClickElement(source, selectors, extraMatch = '', delay =
             }
             // Skip already clicked elements
             if (!elementObj.clicked) {
-                if (parsedTextToMatch && !elementContainsText(elementObj.element, parsedTextToMatch)) {
+                if (textMatchRegexp && !doesElementContainText(elementObj.element, textMatchRegexp)) {
                     continue;
                 }
                 elementObj.element.click();