Merge pull request #27 from Talamasca/TriggerCampaignForUserAndAction…

…ListItem3 add ListID in Transport config
mediapart · Jun 15, 2017 · 76fb92d · 76fb92d
2 parents a9ff1b1 + 59d7955
commit 76fb92d
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 7 deletions.
diff --git a/src/Transport.php b/src/Transport.php
@@ -214,9 +214,9 @@ public function subscribe(Properties $user, $idKey = 'ID')
      * @param Properties $inputData
      * @param array $actionProperties
      */
-    public function TriggerCampaignForUserAndActionListItemWithResult($userId, Properties $inputData, array $actionProperties = [])
+    public function triggerCampaignForUserAndActionListItemWithResult($userId, Properties $inputData, array $actionProperties = [])
     {
-        $options['Actioncode'] = $actionProperties['Actioncode'];
+        $options['ActionCode'] = $actionProperties['Actioncode'];
         $options['ActionListID'] = $actionProperties['ActionListID'];
         $options['ActionListItemData'] = $inputData;
 
@@ -227,7 +227,7 @@ public function TriggerCampaignForUserAndActionListItemWithResult($userId, Prope
      * @param int $userId
      * @param Properties $inputData
      */
-    public function TriggerCampaignForUserWithResult($userId, Properties $inputData)
+    public function triggerCampaignForUserWithResult($userId, Properties $inputData)
     {
         $options['InputData'] = $inputData;
 
@@ -283,6 +283,7 @@ private function getOptions($userId)
     {
         $options = [
             'List' => $this->list,
+            'ListID' => $this->list,
             'UserID' => $userId,
             'GateName' => $this->campaign,
         ];

diff --git a/tests/TransportTest.php b/tests/TransportTest.php
@@ -175,6 +175,7 @@ protected function buildClientForTriggerCampaign($listId, $listName, $campaign,
             ->method('TriggerCampaignForUserWithResult')
             ->with($this->equalTo([
                 'List' => $listId,
+                'ListID' => $listId,
                 'UserID' => $userId,
                 'GateName' => $campaign,
                 'InputData' => $inputData,
@@ -189,9 +190,10 @@ protected function buildClientForTriggerCampaign($listId, $listName, $campaign,
                     $this->equalTo(
                         [
                             'List' => $listId,
+                            'ListID' => $listId,
                             'UserID' => $userId,
                             'GateName' => $campaign,
-                            'Actioncode' => $actionProperties['Actioncode'],
+                            'ActionCode' => $actionProperties['Actioncode'],
                             'ActionListID' => $actionProperties['ActionListID'],
                             'ActionListItemData' => $inputData,
                         ]
@@ -224,7 +226,7 @@ public function testTriggerCampaign()
 
         $transport = new Transport($client, $listName, $campaign);
         $transport->setLogger($logger);
-        $result = $transport->TriggerCampaignForUserWithResult($userId, $inputData);
+        $result = $transport->triggerCampaignForUserWithResult($userId, $inputData);
 
         $this->assertEquals($TriggerCampaignResult, $result);
     }
@@ -254,7 +256,7 @@ public function testTriggerCampaignActionList()
 
         $transport = new Transport($client, $listName, $campaign);
         $transport->setLogger($logger);
-        $result = $transport->TriggerCampaignForUserAndActionListItemWithResult($userId, $inputData, $actionProperties);
+        $result = $transport->triggerCampaignForUserAndActionListItemWithResult($userId, $inputData, $actionProperties);
 
         $this->assertEquals($TriggerCampaignResult, $result);
     }
@@ -284,7 +286,7 @@ public function testTriggerCampaignWithException()
 
         $this->setExpectedException('\Exception');
 
-        $result = $transport->TriggerCampaignForUserWithResult($userId, $inputData);
+        $result = $transport->triggerCampaignForUserWithResult($userId, $inputData);
     }
 
     protected function buildClientForSubscribe($listId, $listName, $campaign, $user, $userProperties, $GetUserByFilterResponseCode, $CreateUserResponseCode)