Skip to content

Commit

Permalink
GumGum: use mediaTypes.banner.sizes (prebid#4416)
Browse files Browse the repository at this point in the history
* adds digitrust module, mods gdpr from bool to int

* update unit test

* only send gdprconsent if present

* uses mediaTypes before trying bidRequest sizes

* removes use of deprecated method
  • Loading branch information
susyt authored and jsnellbaker committed Nov 7, 2019
1 parent 00abeec commit 04ba41a
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 3 deletions.
5 changes: 3 additions & 2 deletions modules/gumgumBidAdapter.js
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ function _getBrowserParams(topWindowUrl) {
try {
topWindow = global.top;
topScreen = topWindow.screen;
topUrl = topWindowUrl || utils.getTopWindowUrl();
topUrl = topWindowUrl || '';
} catch (error) {
utils.logError(error);
return browserParams
Expand Down Expand Up @@ -145,6 +145,7 @@ function buildRequests (validBidRequests, bidderRequest) {
userId = {}
} = bidRequest;
const data = {};
const sizes = bidRequest.mediaTypes && bidRequest.mediaTypes.banner && bidRequest.mediaTypes.banner.sizes;
const topWindowUrl = bidderRequest && bidderRequest.refererInfo && bidderRequest.refererInfo.referer;
if (pageViewId) {
data.pv = pageViewId
Expand Down Expand Up @@ -177,7 +178,7 @@ function buildRequests (validBidRequests, bidderRequest) {
tId: transactionId,
pi: data.pi,
selector: params.selector,
sizes: bidRequest.sizes || bidRequest.mediatype[banner].sizes,
sizes: sizes || bidRequest.sizes,
url: BID_ENDPOINT,
method: 'GET',
data: Object.assign(data, _getBrowserParams(topWindowUrl), _getDigiTrustQueryParams(userId), _getTradeDeskIDParam(userId))
Expand Down
6 changes: 5 additions & 1 deletion test/spec/modules/gumgumBidAdapter_spec.js
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,11 @@ describe('gumgumAdapter', function () {
'bidfloor': 0.05
},
'adUnitCode': 'adunit-code',
'sizes': [[300, 250], [300, 600], [1, 1]],
'mediaTypes': {
'banner': {
sizes: [[300, 250], [300, 600], [1, 1]]
}
},
'bidId': '30b31c1838de1e',
'bidderRequestId': '22edbae2733bf6',
'auctionId': '1d1a030790a475',
Expand Down

0 comments on commit 04ba41a

Please sign in to comment.