OneTrust_CookieConesnt_Changes: Addressed review comments

analytics.js

@@ -195,19 +195,13 @@ class Analytics {
 
       // If cookie consent object is return we filter according to consents given by user
       // else we do not consider any filtering for cookie consent.
-      if (this.cookieConsent) {
-        this.clientIntegrations = this.clientIntegrations.filter((intg) => {
-          return (
-            // remove from the list which don't have support yet in SDK
-            integrations[intg.name] != undefined &&
-            this.cookieConsent.isEnabled(intg.config)
-          );
-        });
-      } else {
-        this.clientIntegrations = this.clientIntegrations.filter((intg) => {
-          return integrations[intg.name] != undefined;
-        });
-      }
+      this.clientIntegrations = this.clientIntegrations.filter((intg) => {
+        return (
+          integrations[intg.name] != undefined &&
+          (!this.cookieConsent ||
+            (this.cookieConsent && this.cookieConsent.isEnabled(intg.config)))
+        );
+      });
       this.init(this.clientIntegrations);
     } catch (error) {
       handleError(error);

cookieConsent/CookieConsentFactory.js

@@ -1,8 +1,5 @@
-import * as OneTrust from "./OneTrust";
+import { OneTrust } from "./OneTrust";
 
-const cookieConsent = {
-  OneTrust: OneTrust.default,
-};
 class CookieConsentFactory {
   static initialize(sourceConfig, options) {
     /**
@@ -15,19 +12,14 @@ class CookieConsentFactory {
      *
      */
 
-    if (
-      options.cookieConsentManager &&
-      options.cookieConsentManager.oneTrust &&
-      options.cookieConsentManager.oneTrust.enabled
-    ) {
+    if (options.cookieConsentManager?.oneTrust?.enabled) {
       // This is P1. When we have an ui in source side to turn on/off of cookie consent
       // if (sourceConfig &&
       //     sourceConfig.cookieConsentManager &&
       // sourceConfig.cookieConsentManager.oneTrust &&
       // sourceConfig.cookieConsentManager.oneTrustenabled) {
 
-      const CookieConsentClass = cookieConsent.OneTrust;
-      return new CookieConsentClass();
+      return new OneTrust();
 
       // }
     }

cookieConsent/OneTrust/index.js

@@ -1,3 +1,3 @@
 import OneTrust from "./browser";
 
-export default OneTrust;
+export { OneTrust };