Convert http_build_query to json_encode (#94)

Author: oakbani

src/Optimizely/Optimizely.php

@@ -239,7 +239,7 @@ protected function sendImpressionEvent($experimentKey, $variationKey, $userId, $
             sprintf(
                 'Dispatching impression event to URL %s with params %s.',
                 $impressionEvent->getUrl(),
-                http_build_query($impressionEvent->getParams())
+                json_encode($impressionEvent->getParams())
             )
         );
 
@@ -356,7 +356,7 @@ public function track($eventKey, $userId, $attributes = null, $eventTags = null)
                 sprintf(
                     'Dispatching conversion event to URL %s with params %s.',
                     $conversionEvent->getUrl(),
-                    http_build_query($conversionEvent->getParams())
+                    json_encode($conversionEvent->getParams())
                 )
             );
 

tests/OptimizelyTest.php

@@ -890,7 +890,7 @@ public function testTrackNoAttributesNoEventValue()
             ->method('log')
             ->with(
                 Logger::DEBUG,
-                'Dispatching conversion event to URL logx.optimizely.com/track with params param1=val1.'
+                'Dispatching conversion event to URL logx.optimizely.com/track with params {"param1":"val1"}.'
             );
 
         $optlyObject = new Optimizely($this->datafile, new ValidEventDispatcher(), $this->loggerMock);
@@ -1028,7 +1028,7 @@ public function testTrackNoAttributesNoEventValueAfterSetForcedVariation()
             ->method('log')
             ->with(
                 Logger::DEBUG,
-                'Dispatching conversion event to URL logx.optimizely.com/track with params param1=val1.'
+                'Dispatching conversion event to URL logx.optimizely.com/track with params {"param1":"val1"}.'
             );
 
         $optlyObject = new Optimizely($this->datafile, new ValidEventDispatcher(), $this->loggerMock);
@@ -1168,7 +1168,7 @@ public function testTrackWithAttributesNoEventValue()
             ->method('log')
             ->with(
                 Logger::DEBUG,
-                'Dispatching conversion event to URL logx.optimizely.com/track with params param1=val1.'
+                'Dispatching conversion event to URL logx.optimizely.com/track with params {"param1":"val1"}.'
             );
 
         $optlyObject = new Optimizely($this->datafile, new ValidEventDispatcher(), $this->loggerMock);
@@ -1304,7 +1304,7 @@ public function testTrackNoAttributesWithDeprecatedEventValue()
             ->method('log')
             ->with(
                 Logger::DEBUG,
-                'Dispatching conversion event to URL logx.optimizely.com/track with params param1=val1.'
+                'Dispatching conversion event to URL logx.optimizely.com/track with params {"param1":"val1"}.'
             );
 
         $optlyObject = new Optimizely($this->datafile, new ValidEventDispatcher(), $this->loggerMock);
@@ -1430,7 +1430,7 @@ public function testTrackNoAttributesWithEventValue()
             ->method('log')
             ->with(
                 Logger::DEBUG,
-                'Dispatching conversion event to URL logx.optimizely.com/track with params param1=val1.'
+                'Dispatching conversion event to URL logx.optimizely.com/track with params {"param1":"val1"}.'
             );
 
         $optlyObject = new Optimizely($this->datafile, new ValidEventDispatcher(), $this->loggerMock);
@@ -1525,7 +1525,7 @@ public function testTrackNoAttributesWithInvalidEventValue()
             ->with(Logger::INFO, 'Tracking event "purchase" for user "test_user".');
         $this->loggerMock->expects($this->at($callIndex++))
             ->method('log')
-            ->with(Logger::DEBUG, 'Dispatching conversion event to URL logx.optimizely.com/track with params param1=val1.');
+            ->with(Logger::DEBUG, 'Dispatching conversion event to URL logx.optimizely.com/track with params {"param1":"val1"}.');
 
         $optlyObject = new Optimizely($this->datafile, new ValidEventDispatcher(), $this->loggerMock);
 
@@ -1669,7 +1669,7 @@ public function testTrackWithAttributesWithDeprecatedEventValue()
             ->method('log')
             ->with(
                 Logger::DEBUG,
-                'Dispatching conversion event to URL logx.optimizely.com/track with params param1=val1.'
+                'Dispatching conversion event to URL logx.optimizely.com/track with params {"param1":"val1"}.'
             );
 
 
@@ -1809,7 +1809,7 @@ public function testTrackWithAttributesWithEventValue()
             ->method('log')
             ->with(
                 Logger::DEBUG,
-                'Dispatching conversion event to URL logx.optimizely.com/track with params param1=val1.'
+                'Dispatching conversion event to URL logx.optimizely.com/track with params {"param1":"val1"}.'
             );
 
         $optlyObject = new Optimizely($this->datafile, new ValidEventDispatcher(), $this->loggerMock);
@@ -3094,7 +3094,7 @@ public function testSendImpressionEventWithNoAttributes()
             ->method('log')
             ->with(
                 Logger::DEBUG,
-                'Dispatching impression event to URL logx.optimizely.com/decision with params param1=val1&param2=val2.'
+                'Dispatching impression event to URL logx.optimizely.com/decision with params {"param1":"val1","param2":"val2"}.'
             );
 
         $optlyObject->sendImpressionEvent('group_experiment_1', 'group_exp_1_var_2', 'user_1', null);
@@ -3173,7 +3173,7 @@ public function testSendImpressionEventWithAttributes()
             ->method('log')
             ->with(
                 Logger::DEBUG,
-                'Dispatching impression event to URL logx.optimizely.com/decision with params param1=val1.'
+                'Dispatching impression event to URL logx.optimizely.com/decision with params {"param1":"val1"}.'
             );
 
         $eventBuilder = new \ReflectionProperty(Optimizely::class, '_eventBuilder');