Skip to content

Commit 24d5a19

Browse files
committed
Ensure get lang only for amazon urls
1 parent 66a7424 commit 24d5a19

File tree

1 file changed

+9
-8
lines changed

1 file changed

+9
-8
lines changed

packages/metascraper-amazon/index.js

Lines changed: 9 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -20,22 +20,23 @@ const SUFFIX_LANGUAGES = {
2020
'it': 'it'
2121
}
2222

23-
const wrap = rule => ({ htmlDom, url }) => isAmazonUrl(url) && rule(htmlDom)
24-
25-
const wrapUrl = rule => ({ htmlDom, url }) => {
26-
const value = wrap(rule)({htmlDom, url})
27-
return isUrl(value) && value
28-
}
29-
3023
const getDomainLanguage = url => {
3124
const {host} = new URL(url)
3225
const suffix = host.replace('www.', '').split('.')
3326
suffix.shift()
3427
return SUFFIX_LANGUAGES[suffix.join('.')]
3528
}
3629

30+
const createWrap = fn => rule => ({ htmlDom, url }) => {
31+
const value = isAmazonUrl(url) && rule(htmlDom)
32+
return !fn ? value : fn(value) && value
33+
}
34+
35+
const wrap = createWrap()
36+
const wrapUrl = createWrap(value => isUrl(value))
37+
3738
module.exports = () => ({
38-
lang: [({ htmlDom: $, meta, url }) => getDomainLanguage(url)],
39+
lang: [({ htmlDom: $, meta, url }) => isAmazonUrl(url) && getDomainLanguage(url)],
3940
author: [
4041
wrap($ => titleize($('.contributorNameID').text())),
4142
wrap($ => titleize($('#bylineInfo').text())),

0 commit comments

Comments
 (0)