gppControl: accept flat section data

prebid · Nov 14, 2024 · 87d46df · 87d46df
1 parent 62307a3
commit 87d46df
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/libraries/mspa/activityControls.js b/libraries/mspa/activityControls.js
@@ -7,6 +7,7 @@ import {
 } from '../../src/activities/activities.js';
 import {gppDataHandler} from '../../src/adapterManager.js';
 import {logInfo} from '../../src/utils.js';
+import {isPlainObject} from "lodash/lang.js";
 
 // default interpretation for MSPA consent(s):
 // https://docs.prebid.org/features/mspa-usnat.html
@@ -105,7 +106,7 @@ export function mspaRule(sids, getConsent, denies, applicableSids = () => gppDat
 }
 
 function flatSection(subsections) {
-  if (subsections == null) return subsections;
+  if (!Array.isArray(subsections)) return subsections;
   return subsections.reduceRight((subsection, consent) => {
     return Object.assign(consent, subsection);
   }, {});

diff --git a/test/spec/libraries/mspa/activityControls_spec.js b/test/spec/libraries/mspa/activityControls_spec.js
@@ -229,6 +229,13 @@ describe('setupRules', () => {
     sinon.assert.calledWith(rules.mockActivity, {mock: 'consent'})
   });
 
+  it('should accept already flattened section data', () => {
+    consent.parsedSections.mockApi = {flat: 'consent'};
+    runSetup('mockApi', [1]);
+    isAllowed('mockActivity', {});
+    sinon.assert.calledWith(rules.mockActivity, {flat: 'consent'})
+  })
+
   it('should not choke when no consent data is available', () => {
     consent = null;
     runSetup('mockApi', [1]);