Skip to content

Commit 40567dd

Browse files
committed
Refactor
1 parent b53deae commit 40567dd

1 file changed

Lines changed: 9 additions & 17 deletions

File tree

src/modules/tracker.js

Lines changed: 9 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -67,28 +67,20 @@ function applyParams(parameters, options) {
6767

6868
if (search) {
6969
try {
70+
const utmParamKeys = ['utm_source', 'utm_medium', 'utm_campaign'];
71+
const utmQueryParamStrArr = [];
7072
const searchParams = new URLSearchParams(search);
71-
const utmSource = searchParams.get('utm_source');
72-
const utmMedium = searchParams.get('utm_medium');
73-
const utmCampaign = searchParams.get('utm_campaign');
7473

75-
// Add UTM parameters to origin_referrer if they exist
76-
if (utmSource || utmMedium || utmCampaign) {
77-
aggregateParams.origin_referrer += '?';
74+
utmParamKeys.forEach((key) => {
75+
const utmParamValue = searchParams.get(key);
7876

79-
if (utmSource) {
80-
aggregateParams.origin_referrer += `utm_source=${utmSource}`;
81-
}
82-
83-
if (utmMedium) {
84-
if (utmSource) aggregateParams.origin_referrer += '&';
85-
aggregateParams.origin_referrer += `utm_medium=${utmMedium}`;
77+
if (utmParamValue) {
78+
utmQueryParamStrArr.push(`${key}=${utmParamValue}`);
8679
}
80+
});
8781

88-
if (utmCampaign) {
89-
if (utmSource || utmMedium) aggregateParams.origin_referrer += '&';
90-
aggregateParams.origin_referrer += `utm_campaign=${utmCampaign}`;
91-
}
82+
if (utmQueryParamStrArr.length) {
83+
aggregateParams.origin_referrer += `?${utmQueryParamStrArr.join('&')}`;
9284
}
9385
} catch (e) {
9486
// Do nothing

0 commit comments

Comments
 (0)