fixed issue when exclude keywords is empty

newfold-labs · Dec 16, 2024 · dc9112c · dc9112c
1 parent f94146f
commit dc9112c
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 4 deletions.
diff --git a/includes/LinkPrefetch.php b/includes/LinkPrefetch.php
@@ -73,7 +73,7 @@ public static function getDefaultSettings() {
 			'instantClick'    => false,
 			'activeOnMobile'  => false,
 			'mobileBehavior'  => 'touchstart',
-			'ignoreKeywords'  => 'wp-admin,#,?',
+			'ignoreKeywords'  => '#,?',
 		);
 	}
 

diff --git a/scripts/linkPrefetch.js b/scripts/linkPrefetch.js
@@ -18,7 +18,7 @@ document.addEventListener( 'DOMContentLoaded', () => {
 				activeOnDesktop: config.activeOnDesktop,
 				behavior: config.behavior,
 				hoverDelay: parseInt( config.hoverDelay ) || 60,
-				ignoreKeywords: config.ignoreKeywords.split( ',' ),
+				ignoreKeywords: config.ignoreKeywords ? config.ignoreKeywords.split( ',' ) : '',
 				instantClick: config.instantClick,
 				mobileActive: config.activeOnMobile,
 				isMobile: config.isMobile,
@@ -138,13 +138,19 @@ document.addEventListener( 'DOMContentLoaded', () => {
 			if ( ! url || this.prefetchedUrls.has( url ) ) {
 				return false;
 			}
+
 			if (
 				url.replace( /\/$/, '' ) ===
 					location.href.replace( /\/$/, '' ) ||
-				this.ignoreKeywords.some( ( k ) => url.includes( k ) )
+					(
+						this.ignoreKeywords !== '' &&
+						this.ignoreKeywords.some( ( k ) => url.includes( k ) )
+					)
+
 			) {
 				return false;
 			}
+
 			return true;
 		}
 
@@ -164,7 +170,7 @@ document.addEventListener( 'DOMContentLoaded', () => {
 		instantClick: true,
 		activeOnMobile: true,
 		mobileBehavior: 'viewport',
-		ignoreKeywords: 'wp-admin,#,?',
+		ignoreKeywords: '#,?',
 	};
 
 	new LinkPrefetcher( config ).init();