Openx: use bidfloor if set - prebid.js adapter behavior

prebid · May 7, 2021 · f14d9c4 · f14d9c4
1 parent ded8eba
commit f14d9c4
Showing 1 changed file with 12 additions and 2 deletions.
diff --git a/src/main/java/org/prebid/server/bidder/openx/OpenxBidder.java b/src/main/java/org/prebid/server/bidder/openx/OpenxBidder.java
@@ -10,7 +10,6 @@
 import com.iab.openrtb.response.SeatBid;
 import io.vertx.core.http.HttpMethod;
 import org.apache.commons.collections4.CollectionUtils;
-import org.apache.commons.lang3.ObjectUtils;
 import org.apache.commons.lang3.StringUtils;
 import org.prebid.server.bidder.Bidder;
 import org.prebid.server.bidder.model.BidderBid;
@@ -32,6 +31,7 @@
 import org.prebid.server.proto.openrtb.ext.response.BidType;
 import org.prebid.server.util.HttpUtil;
 
+import java.math.BigDecimal;
 import java.util.ArrayList;
 import java.util.Collection;
 import java.util.Collections;
@@ -160,7 +160,7 @@ private Imp makeImp(Imp imp) {
         final ExtImpPrebid prebidImpExt = impExt.getPrebid();
         final Imp.ImpBuilder impBuilder = imp.toBuilder()
                 .tagid(openxImpExt.getUnit())
-                .bidfloor(ObjectUtils.defaultIfNull(imp.getBidfloor(), openxImpExt.getCustomFloor()))
+                .bidfloor(resolveBidFloor(imp, openxImpExt))
                 .ext(makeImpExt(openxImpExt.getCustomParams()));
 
         if (resolveImpType(imp) == OpenxImpType.video
@@ -173,6 +173,16 @@ private Imp makeImp(Imp imp) {
         return impBuilder.build();
     }
 
+    private BigDecimal resolveBidFloor(Imp imp, ExtImpOpenx openxExtImp) {
+        BigDecimal impBidFloor = imp.getBidfloor();
+        BigDecimal customFloor = openxExtImp.getCustomFloor();
+        if (impBidFloor == null || impBidFloor.compareTo(BigDecimal.ZERO) == 0
+                && customFloor.compareTo(BigDecimal.ZERO) > 0) {
+            return customFloor;
+        }
+        return impBidFloor;
+    }
+
     private ExtRequest makeReqExt(Imp imp) {
         final ExtImpOpenx openxImpExt = parseOpenxExt(imp).getBidder();
         return mapper.fillExtension(