Skip to content

Commit c9f3a95

Browse files
committed
convert bidders: eplanning
1 parent 2a93dab commit c9f3a95

File tree

3 files changed

+14
-12
lines changed

3 files changed

+14
-12
lines changed

modules/engageyaBidAdapter.js

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,9 +13,10 @@ function getPageUrl(bidRequest, bidderRequest) {
1313
if (bidRequest.params.pageUrl && bidRequest.params.pageUrl != '[PAGE_URL]') {
1414
return bidRequest.params.pageUrl;
1515
}
16-
if (bidderRequest && bidderRequest.refererInfo && bidderRequest.refererInfo.referer) {
17-
return bidderRequest.refererInfo.referer;
16+
if (bidderRequest && bidderRequest.refererInfo && bidderRequest.refererInfo.page) {
17+
return bidderRequest.refererInfo.page;
1818
}
19+
// TODO: does this fallback make sense?
1920
const pageUrl = (isInIframe() && document.referrer)
2021
? document.referrer
2122
: window.location.href;

modules/eplanningBidAdapter.js

Lines changed: 7 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@ import { isEmpty, getWindowSelf, parseSizesInput } from '../src/utils.js';
22
import { getGlobal } from '../src/prebidGlobal.js';
33
import { registerBidder } from '../src/adapters/bidderFactory.js';
44
import { getStorageManager } from '../src/storageManager.js';
5+
import {parseDomain} from '../src/refererDetection.js';
56

67
const BIDDER_CODE = 'eplanning';
78
export const storage = getStorageManager({bidderCode: BIDDER_CODE});
@@ -36,18 +37,16 @@ export const spec = {
3637
const urlConfig = getUrlConfig(bidRequests);
3738
const pcrs = getCharset();
3839
const spaces = getSpaces(bidRequests, urlConfig.ml);
39-
const pageUrl = bidderRequest.refererInfo.referer;
40-
const getDomain = (url) => {
41-
let anchor = document.createElement('a');
42-
anchor.href = url;
43-
return anchor.hostname;
44-
}
40+
// TODO: do the fallbacks make sense here?
41+
const pageUrl = bidderRequest.refererInfo.page || bidderRequest.refererInfo.topmostLocation;
42+
const domain = bidderRequest.refererInfo.domain || window.location.hostname
4543
if (urlConfig.t) {
4644
url = 'https://' + urlConfig.isv + '/layers/t_pbjs_2.json';
4745
params = {};
4846
} else {
49-
url = 'https://' + (urlConfig.sv || DEFAULT_SV) + '/pbjs/1/' + urlConfig.ci + '/' + dfpClientId + '/' + getDomain(pageUrl) + '/' + sec;
50-
const referrerUrl = bidderRequest.refererInfo.referer.reachedTop ? window.top.document.referrer : bidderRequest.refererInfo.referer;
47+
url = 'https://' + (urlConfig.sv || DEFAULT_SV) + '/pbjs/1/' + urlConfig.ci + '/' + dfpClientId + '/' + domain + '/' + sec;
48+
// TODO: does the fallback make sense here?
49+
const referrerUrl = bidderRequest.refererInfo.ref || bidderRequest.refererInfo.topmostLocation
5150

5251
if (storage.hasLocalStorage()) {
5352
registerViewabilityAllBids(bidRequests);

test/spec/modules/eplanningBidAdapter_spec.js

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -293,7 +293,9 @@ describe('E-Planning Adapter', function () {
293293
const refererUrl = 'https://localhost';
294294
const bidderRequest = {
295295
refererInfo: {
296-
referer: refererUrl
296+
page: refererUrl,
297+
domain: 'localhost',
298+
ref: refererUrl,
297299
},
298300
gdprConsent: {
299301
gdprApplies: 1,
@@ -467,7 +469,7 @@ describe('E-Planning Adapter', function () {
467469

468470
it('should return ur parameter with current window url', function () {
469471
const ur = spec.buildRequests(bidRequests, bidderRequest).data.ur;
470-
expect(ur).to.equal(bidderRequest.refererInfo.referer);
472+
expect(ur).to.equal(bidderRequest.refererInfo.page);
471473
});
472474

473475
it('should return fr parameter when there is a referrer', function () {

0 commit comments

Comments
 (0)