Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

AdForm adapter: result bid missing bidId from _bidsRequested #943

Closed
PsychodelEKS opened this issue Jan 20, 2017 · 2 comments
Closed

AdForm adapter: result bid missing bidId from _bidsRequested #943

PsychodelEKS opened this issue Jan 20, 2017 · 2 comments

Comments

@PsychodelEKS
Copy link

Type of issue

bug

Description

AdForm adapter served bids (passed to bidsBackHandler) do not have any attribute to match them with bids from _bidsRequested.

Steps to reproduce

  1. Configure adform adapter to bid any placement
  2. register any fucntion as a bidsBackHandler to observe passed results
  3. get bid from adform

Expected results

adform bid contains adId which matches bidId from corresponding requsted bid (_bidsRequested)

Actual results

there is adId, but it does not match _bidsRequested[*].bidId

Platform details

independant

Other information

Made a hack myself, though there may be better solution (I used additional attribute requestBidId not to interfere with anything):

@@ -78,12 +78,14 @@ function AdformAdapter() {
           bidObject.width = adItem.width;
           bidObject.height = adItem.height;
           bidObject.dealId = adItem.deal_id;
-          bidmanager.addBidResponse(bid.placementCode, bidObject);
         } else {
           bidObject = bidfactory.createBid(2);
           bidObject.bidderCode = bidder;
-          bidmanager.addBidResponse(bid.placementCode, bidObject);
         }
+
+        bidObject.requestBidId = bid.bidId;
+
+        bidmanager.addBidResponse(bid.placementCode, bidObject);
       }
     };
@mkendall07
Copy link
Member

@prebid/adform
Can you please check?

@matthewlane
Copy link
Collaborator

matthewlane commented Feb 7, 2017

#947 to match Adform bids has been merged into master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants