From c3569c4390c674e7995f59e4e8d8bd8c735a422a Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?S=C3=A9bastien=20Millet?= <sebastien.millet@dailymotion.com>
Date: Fri, 27 Sep 2024 16:54:34 +0200
Subject: [PATCH] Dailymotion bid adapter: Process both ORTB2 sources of
 category (#12279)

## Type of change
- [x] Updated bidder adapter

## Description of change
This fixes a naming issue and correctly reports categories from the main ORTB2 site or app objects along with the one coming from the content object.
---
 modules/dailymotionBidAdapter.js                | 10 ++++++----
 test/spec/modules/dailymotionBidAdapter_spec.js |  7 ++++++-
 2 files changed, 12 insertions(+), 5 deletions(-)

diff --git a/modules/dailymotionBidAdapter.js b/modules/dailymotionBidAdapter.js
index 04baaebafb9..791fbccda5f 100644
--- a/modules/dailymotionBidAdapter.js
+++ b/modules/dailymotionBidAdapter.js
@@ -17,10 +17,11 @@ function getVideoMetadata(bidRequest, bidderRequest) {
 
   // As per oRTB 2.5 spec, "A bid request must not contain both an App and a Site object."
   // See section 3.2.14
+  const siteOrAppObj = deepAccess(bidderRequest, 'ortb2.site')
+    ? deepAccess(bidderRequest, 'ortb2.site')
+    : deepAccess(bidderRequest, 'ortb2.app');
   // Content object is either from Object: Site or Object: App
-  const contentObj = deepAccess(bidderRequest, 'ortb2.site')
-    ? deepAccess(bidderRequest, 'ortb2.site.content')
-    : deepAccess(bidderRequest, 'ortb2.app.content');
+  const contentObj = deepAccess(siteOrAppObj, 'content')
 
   const parsedContentData = {
     // Store as object keys to ensure uniqueness
@@ -70,7 +71,8 @@ function getVideoMetadata(bidRequest, bidderRequest) {
       ? videoParams.isCreatedForKids
       : null,
     context: {
-      siteOrAppCat: deepAccess(contentObj, 'cat', []),
+      siteOrAppCat: deepAccess(siteOrAppObj, 'cat', []),
+      siteOrAppContentCat: deepAccess(contentObj, 'cat', []),
       videoViewsInSession: (
         typeof videoParams.videoViewsInSession === 'number' &&
         videoParams.videoViewsInSession >= 0
diff --git a/test/spec/modules/dailymotionBidAdapter_spec.js b/test/spec/modules/dailymotionBidAdapter_spec.js
index e0f572c06ba..2a276a06b15 100644
--- a/test/spec/modules/dailymotionBidAdapter_spec.js
+++ b/test/spec/modules/dailymotionBidAdapter_spec.js
@@ -211,6 +211,7 @@ describe('dailymotionBidAdapterTests', () => {
       isCreatedForKids: bidRequestData[0].params.video.isCreatedForKids,
       context: {
         siteOrAppCat: [],
+        siteOrAppContentCat: [],
         videoViewsInSession: bidRequestData[0].params.video.videoViewsInSession,
         autoplay: bidRequestData[0].params.video.autoplay,
         playerName: bidRequestData[0].params.video.playerName,
@@ -1551,6 +1552,7 @@ describe('dailymotionBidAdapterTests', () => {
       isCreatedForKids: null,
       context: {
         siteOrAppCat: [],
+        siteOrAppContentCat: [],
         videoViewsInSession: null,
         autoplay: null,
         playerName: 'dailymotion',
@@ -1605,6 +1607,7 @@ describe('dailymotionBidAdapterTests', () => {
           coppa: 0,
         },
         site: {
+          cat: ['IAB-1'],
           content: {
             id: '54321',
             language: 'FR',
@@ -1729,7 +1732,8 @@ describe('dailymotionBidAdapterTests', () => {
       livestream: !!bidderRequestData.ortb2.site.content.livestream,
       isCreatedForKids: bidRequestData[0].params.video.isCreatedForKids,
       context: {
-        siteOrAppCat: bidderRequestData.ortb2.site.content.cat,
+        siteOrAppCat: bidderRequestData.ortb2.site.cat,
+        siteOrAppContentCat: bidderRequestData.ortb2.site.content.cat,
         videoViewsInSession: bidRequestData[0].params.video.videoViewsInSession,
         autoplay: bidRequestData[0].params.video.autoplay,
         playerName: bidRequestData[0].params.video.playerName,
@@ -1821,6 +1825,7 @@ describe('dailymotionBidAdapterTests', () => {
       isCreatedForKids: null,
       context: {
         siteOrAppCat: [],
+        siteOrAppContentCat: [],
         videoViewsInSession: null,
         autoplay: null,
         playerName: '',