fix: Set mapped consent name from server to lowercase

mparticle-integrations · Apr 12, 2024 · 284fd7c · 284fd7c
1 parent fd09f40
commit 284fd7c
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 6 deletions.
diff --git a/src/GoogleAdWordsEventForwarder.js b/src/GoogleAdWordsEventForwarder.js
@@ -399,7 +399,11 @@
 
             for (var i = 0; i <= mappings.length - 1; i++) {
                 var mappingEntry = mappings[i];
-                var mpMappedConsentName = mappingEntry.map;
+                // Although consent purposes can be inputted into the UI in any casing
+                // the SDK will automatically lowercase them to prevent pseudo-duplicate
+                // consent purposes, so we call `toLowerCase` on the consentMapping purposes here
+                var mpMappedConsentName = mappingEntry.map.toLowerCase();
+                // var mpMappedConsentName = mappingEntry.map;
                 var googleMappedConsentName = mappingEntry.value;
 
                 if (

diff --git a/test/src/tests.js b/test/src/tests.js
@@ -1135,25 +1135,25 @@ describe('Adwords forwarder', function () {
             var consentMap = [
                 {
                     jsmap: null,
-                    map: 'some_consent',
+                    map: 'Some_consent',
                     maptype: 'ConsentPurposes',
                     value: 'ad_user_data',
                 },
                 {
                     jsmap: null,
-                    map: 'storage_consent',
+                    map: 'Storage_consent',
                     maptype: 'ConsentPurposes',
                     value: 'analytics_storage',
                 },
                 {
                     jsmap: null,
-                    map: 'other_test_consent',
+                    map: 'Other_test_consent',
                     maptype: 'ConsentPurposes',
                     value: 'ad_storage',
                 },
                 {
                     jsmap: null,
-                    map: 'test_consent',
+                    map: 'Test_consent',
                     maptype: 'ConsentPurposes',
                     value: 'ad_personalization',
                 },
@@ -1204,7 +1204,7 @@ describe('Adwords forwarder', function () {
                         conversionId: 'AW-123123123',
                         enableGtag: 'True',
                         consentMappingWeb:
-                            '[{&quot;jsmap&quot;:null,&quot;map&quot;:&quot;some_consent&quot;,&quot;maptype&quot;:&quot;ConsentPurposes&quot;,&quot;value&quot;:&quot;ad_user_data&quot;},{&quot;jsmap&quot;:null,&quot;map&quot;:&quot;storage_consent&quot;,&quot;maptype&quot;:&quot;ConsentPurposes&quot;,&quot;value&quot;:&quot;analytics_storage&quot;},{&quot;jsmap&quot;:null,&quot;map&quot;:&quot;other_test_consent&quot;,&quot;maptype&quot;:&quot;ConsentPurposes&quot;,&quot;value&quot;:&quot;ad_storage&quot;},{&quot;jsmap&quot;:null,&quot;map&quot;:&quot;test_consent&quot;,&quot;maptype&quot;:&quot;ConsentPurposes&quot;,&quot;value&quot;:&quot;ad_personalization&quot;}]',
+                            '[{&quot;jsmap&quot;:null,&quot;map&quot;:&quot;Some_consent&quot;,&quot;maptype&quot;:&quot;ConsentPurposes&quot;,&quot;value&quot;:&quot;ad_user_data&quot;},{&quot;jsmap&quot;:null,&quot;map&quot;:&quot;Storage_consent&quot;,&quot;maptype&quot;:&quot;ConsentPurposes&quot;,&quot;value&quot;:&quot;analytics_storage&quot;},{&quot;jsmap&quot;:null,&quot;map&quot;:&quot;Other_test_consent&quot;,&quot;maptype&quot;:&quot;ConsentPurposes&quot;,&quot;value&quot;:&quot;ad_storage&quot;},{&quot;jsmap&quot;:null,&quot;map&quot;:&quot;Test_consent&quot;,&quot;maptype&quot;:&quot;ConsentPurposes&quot;,&quot;value&quot;:&quot;ad_personalization&quot;}]',
                     },
                     reportService.cb,
                     true