(*) Rename EdgeDomain$domain into topDomain
This variable had a very confusing name, and was dangerously easy to use in the wrong place with the result of getting something that only works as expected half the time. Ideally this class needs an overhaul, the assumptions it makes about domain names aren't great.
This commit is contained in:
parent
4801c47273
commit
edf9aa2c23
@ -115,8 +115,9 @@ public class StackexchangeSideloader implements SideloadSource {
|
||||
ret.words = keywordExtractor.extractKeywords(dld, url);
|
||||
ret.words.addAllSyntheticTerms(List.of(
|
||||
"site:" + domainName,
|
||||
"site:" + url.domain.domain,
|
||||
url.domain.domain
|
||||
"site:" + url.domain.topDomain,
|
||||
url.domain.topDomain,
|
||||
domainName
|
||||
));
|
||||
|
||||
if (!post.tags().isBlank()) {
|
||||
|
Loading…
Reference in New Issue
Block a user