[FSSDK-9001] Fix: ODP should be enabled by default when initialized optimizely using OptimizelyFactory (#337)

mnoman09 · web-flow · commit c5592e746fc1 · 2023-03-18T03:55:51.000+05:00
* using default odp manager when initializing optimizely using optimizelyFactory to enable it by default
* Added use odp check
diff --git a/OptimizelySDK/OptimizelyFactory.cs b/OptimizelySDK/OptimizelyFactory.cs
@@ -1,5 +1,5 @@
 ﻿/* 
- * Copyright 2019-2021, Optimizely
+ * Copyright 2019-2021, 2023, Optimizely
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use file except in compliance with the License.
@@ -14,6 +14,10 @@
  * limitations under the License.
  */
 
+#if !(NET35 || NET40 || NETSTANDARD1_6)
+#define USE_ODP
+#endif
+
 using System;
 #if !NETSTANDARD1_6 && !NET35
 using System.Configuration;
@@ -27,6 +31,11 @@
 using OptimizelySDK.Logger;
 using OptimizelySDK.Notifications;
 
+#if USE_ODP
+using OptimizelySDK.Odp;
+#endif
+
+
 namespace OptimizelySDK
 {
     /// <summary>
@@ -215,8 +224,18 @@ public static Optimizely NewDefaultInstance(ProjectConfigManager configManager,
             UserProfileService userprofileService = null, EventProcessor eventProcessor = null
         )
         {
+#if USE_ODP
+            var odpManager = new OdpManager.Builder()
+                                .WithErrorHandler(errorHandler)
+                                .WithLogger(logger)
+                                .Build();
+            return new Optimizely(configManager, notificationCenter, eventDispatcher, logger,
+                errorHandler, userprofileService, eventProcessor, null, odpManager);
+#else
             return new Optimizely(configManager, notificationCenter, eventDispatcher, logger,
                 errorHandler, userprofileService, eventProcessor);
+#endif
+
         }
     }
 }
