[bug] fix search breaking when searching links with empty url or name

Author: zombieFox

js/search.js

@@ -55,9 +55,16 @@ var search = (function(){
 };
 searchedBookmarks.total=bookmarks.get().length;
 bookmarks.get().forEach(function(arrayItem,index){
-if(arrayItem.url.replace(/^https?\:\/\//i,"").replace(/\/$/,"").toLowerCase().includes(searchInput.value.toLowerCase().replace(/\s/g,""))||arrayItem.name.toLowerCase().includes(searchInput.value.toLowerCase().replace(/\s/g,""))){
-varbookmarkDataCopy=JSON.parse(JSON.stringify(arrayItem));
-searchedBookmarks.matching.push(bookmarkDataCopy);
+if(arrayItem.url!=null){
+if(arrayItem.url.replace(/^https?\:\/\//i,"").replace(/\/$/,"").toLowerCase().includes(searchInput.value.toLowerCase().replace(/\s/g,""))){
+varbookmarkDataCopy=JSON.parse(JSON.stringify(arrayItem));
+searchedBookmarks.matching.push(bookmarkDataCopy);
+};
+}elseif(arrayItem.name!=null){
+if(arrayItem.name.toLowerCase().includes(searchInput.value.toLowerCase().replace(/\s/g,""))){
+varbookmarkDataCopy=JSON.parse(JSON.stringify(arrayItem));
+searchedBookmarks.matching.push(bookmarkDataCopy);
+};
 };
 });
 returnsearchedBookmarks;

js/version.js

@@ -1,6 +1,6 @@
 varversion=(function(){
 
-varcurrent="3.16.2";
+varcurrent="3.16.3";
 
 varcompare=function(a,b){
 varpa=a.split(".");

manifest.json

@@ -2,7 +2,7 @@
 "name": "nightTab",
 "short_name": "nightTab",
 "description": "A neutral new tab page accented with a chosen colour. Customise the layout, style, background and bookmarks in nightTab.",
-"version": "3.16.2",
+"version": "3.16.3",
 "manifest_version": 2,
 "chrome_url_overrides":{
 "newtab": "index.html"