vwo-sdk 1.36.0 → 1.37.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 43fec6e51e5bb0377e3714ca0fa90bc1cd11c26c29c99af86e0b8fec858af4b1
-  data.tar.gz: 6b63f796cc23e812e65a731cfeefddfcee46c1bb1d24ece33b9bdecbc619bc62
+  metadata.gz: ea72e150de33227c3e911fd61478a8815fb8f42f788df4a65a6751a1da7c25fb
+  data.tar.gz: c88b275817a6ef0ca80efc03287e57fd1488be9390e7a028a4dc812cca93abc0
 SHA512:
-  metadata.gz: 0357c8d8c220348265463a1adaabd0480c9424cf8082d05541643c6ccdbdfbf2692e603a08acfd1b71d4762d31649ba1e5fd5469707f1e01251d96329509a6e2
-  data.tar.gz: e30bd9f514a53da482f5e7d38dda6a3ced309fa027afe4d9148cfb464fff6acf3da0e012574dbf38712390ab24daa0148a42ce76f83fb8d264ecfb88c8bfb84a
+  metadata.gz: 294fab2cbb559b6869e547dd0f55b9bb5ac54910ba61d497f9d205fd37c405bc206b6264912bc0600306dd0986d875c034be0e4342260f3d0aa8a647ad0a67e3
+  data.tar.gz: d25b2e2642a1e178eeeba2b0f7f7e82188040df6116a6a14bb13abfba7cf8e0b2ee28c5161936cc9773663f55cdca9d371b72deea94393ac1c9d6a20cf9f383b

data/lib/vwo/constants.rb

@@ -19,7 +19,7 @@ class VWO
     SEED_VALUE = 1
     MAX_TRAFFIC_PERCENT = 100
     MAX_TRAFFIC_VALUE = 10_000
-    MAX_RANGE = 10000
+    MAX_RANGE = 10_000
     STATUS_RUNNING = 'RUNNING'
     # rubocop:disable Style/ExpandPathArguments
     LIBRARY_PATH =  File.expand_path('../..', __FILE__)
@@ -27,7 +27,7 @@ class VWO
     HTTP_PROTOCOL = 'http://'
     HTTPS_PROTOCOL = 'https://'
     URL_NAMESPACE = '6ba7b811-9dad-11d1-80b4-00c04fd430c8'
-    SDK_VERSION = '1.36.0'
+    SDK_VERSION = '1.37.1'
     SDK_NAME = 'ruby'
     VWO_DELIMITER = '_vwo_'
     MAX_EVENTS_PER_REQUEST = 5000

data/lib/vwo/core/bucketer.rb

@@ -31,7 +31,7 @@ class VWO
       # Source - https://stackoverflow.com/a/20766900/2494535
       U_MAX_32_BIT = 0xFFFFFFFF
       MAX_HASH_VALUE = 2**32
-      FILE = FileNameEnum::Bucketer
+      FILE = FileNameEnum::BUCKETER
 
       def initialize
         @logger = VWO::Utils::Logger
@@ -49,16 +49,14 @@ class VWO
             LogLevelEnum::ERROR,
             'USER_ID_INVALID',
             {
-            '{file}' => FILE,
-            '{userId}' => user_id
+              '{file}' => FILE,
+              '{userId}' => user_id
             }
           )
           return false
         end
 
-        if campaign.nil?
-          return false
-        end
+        return false if campaign.nil?
 
         traffic_allocation = campaign['percentTraffic']
         value_assigned_to_user = get_bucket_value_for_user(user_id, campaign, nil, disable_logs)
@@ -92,21 +90,17 @@ class VWO
             LogLevelEnum::ERROR,
             'USER_ID_INVALID',
             {
-            '{file}' => FILE,
-            '{userId}' => user_id
+              '{file}' => FILE,
+              '{userId}' => user_id
             }
           )
           return
         end
 
-        unless campaign
-          return
-        end
+        return unless campaign
 
         user_id_for_hash_value = user_id
-        if campaign['isBucketingSeedEnabled']
-          user_id_for_hash_value = campaign['id'].to_s + "_" + user_id
-        end
+        user_id_for_hash_value = "#{campaign['id']}_#{user_id}" if campaign['isBucketingSeedEnabled']
         hash_value = MurmurHash3::V32.str_hash(user_id_for_hash_value, SEED_VALUE) & U_MAX_32_BIT
         normalize = MAX_TRAFFIC_VALUE.to_f / campaign['percentTraffic']
         multiplier = normalize / 100
@@ -120,12 +114,12 @@ class VWO
           LogLevelEnum::DEBUG,
           'USER_CAMPAIGN_BUCKET_VALUES',
           {
-              '{file}' => FILE,
-              '{campaignKey}' => campaign['key'],
-              '{userId}' => user_id,
-              '{percentTraffic}' => campaign['percentTraffic'],
-              '{hashValue}' => hash_value,
-              '{bucketValue}' => bucket_value,
+            '{file}' => FILE,
+            '{campaignKey}' => campaign['key'],
+            '{userId}' => user_id,
+            '{percentTraffic}' => campaign['percentTraffic'],
+            '{hashValue}' => hash_value,
+            '{bucketValue}' => bucket_value
           },
           disable_logs
         )
@@ -156,9 +150,9 @@ class VWO
       def get_bucket_value_for_user(user_id, campaign = {}, group_id = nil, disable_logs = false)
         user_id_for_hash_value = user_id
         if group_id
-          user_id_for_hash_value = group_id.to_s + "_" + user_id
+          user_id_for_hash_value = "#{group_id}_#{user_id}"
         elsif campaign['isBucketingSeedEnabled']
-          user_id_for_hash_value = campaign['id'].to_s + "_" + user_id
+          user_id_for_hash_value = "#{campaign['id']}_#{user_id}"
         end
         hash_value = MurmurHash3::V32.str_hash(user_id_for_hash_value, SEED_VALUE) & U_MAX_32_BIT
         bucket_value = get_bucket_value(hash_value, MAX_TRAFFIC_PERCENT)
@@ -198,11 +192,9 @@ class VWO
       # @return[Hash|nil]
       #
       def get_campaign_using_range(range_for_campaigns, campaigns)
-        range_for_campaigns = range_for_campaigns * 100
+        range_for_campaigns *= 100
         campaigns.each do |campaign|
-          if campaign["max_range"] && campaign["max_range"] >= range_for_campaigns && campaign["min_range"] <= range_for_campaigns
-            return campaign
-          end
+          return campaign if campaign['max_range'] && campaign['max_range'] >= range_for_campaigns && campaign['min_range'] <= range_for_campaigns
         end
         nil
       end