Update function names
Removing underscores; I'm fine with marking private functions, but I'd like to make it consistent across the codebasepull/647/head
parent
97e7a8283b
commit
c3e7339019
|
@ -393,7 +393,7 @@ function findClosestElement(target, elements) {
|
||||||
return closestElement;
|
return closestElement;
|
||||||
}
|
}
|
||||||
|
|
||||||
async function _filterSearchResult(matchingSite, searchResult, searchEngine, countFiltered, storage, reorderedHrefs) {
|
async function filterSearchResult(matchingSite, searchResult, searchEngine, countFiltered, storage, reorderedHrefs) {
|
||||||
// Get user's settings for the wiki
|
// Get user's settings for the wiki
|
||||||
let id = matchingSite['id'];
|
let id = matchingSite['id'];
|
||||||
let searchFilterSetting = 'replace';
|
let searchFilterSetting = 'replace';
|
||||||
|
@ -497,7 +497,7 @@ async function _filterSearchResult(matchingSite, searchResult, searchEngine, cou
|
||||||
return countFiltered;
|
return countFiltered;
|
||||||
}
|
}
|
||||||
|
|
||||||
async function _reorderDestinationSearchResult(resultsFirstChild, searchResult) {
|
async function reorderDestinationSearchResult(resultsFirstChild, searchResult) {
|
||||||
// Find containing element for the search result
|
// Find containing element for the search result
|
||||||
const closestJsController = searchResult.closest('div[jscontroller]');
|
const closestJsController = searchResult.closest('div[jscontroller]');
|
||||||
const closestDataDiv = searchResult.closest('div[data-hveid].g') || searchResult.closest('div[data-hveid]');
|
const closestDataDiv = searchResult.closest('div[data-hveid].g') || searchResult.closest('div[data-hveid]');
|
||||||
|
@ -565,7 +565,7 @@ async function reorderSearchResults(searchResults, searchEngine, storage) {
|
||||||
console.debug('Indie Wiki Buddy is not re-ordering results, as an indie wiki is already the first result.');
|
console.debug('Indie Wiki Buddy is not re-ordering results, as an indie wiki is already the first result.');
|
||||||
break;
|
break;
|
||||||
} else {
|
} else {
|
||||||
await _reorderDestinationSearchResult(resultsFirstChild, searchResult);
|
await reorderDestinationSearchResult(resultsFirstChild, searchResult);
|
||||||
reorderedHrefs.push(searchResultLink);
|
reorderedHrefs.push(searchResultLink);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
@ -615,7 +615,7 @@ async function filterSearchResults(searchResults, searchEngine, storage, reorder
|
||||||
// Handle source -> destination filtering
|
// Handle source -> destination filtering
|
||||||
let matchingSource = await commonFunctionFindMatchingSite(searchResultLink, crossLanguageSetting);
|
let matchingSource = await commonFunctionFindMatchingSite(searchResultLink, crossLanguageSetting);
|
||||||
if (matchingSource) {
|
if (matchingSource) {
|
||||||
countFiltered = await _filterSearchResult(matchingSource, searchResult, searchEngine, countFiltered, storage, reorderedHrefs);
|
countFiltered = await filterSearchResult(matchingSource, searchResult, searchEngine, countFiltered, storage, reorderedHrefs);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
} catch (e) {
|
} catch (e) {
|
||||||
|
|
Loading…
Reference in New Issue