Merge pull request #8 from tevfik6/CXAPPS-329

CXAPPS-329: Fix Handling of Fields in Request for Upsert API

Author: MichaelJ2324

.gitignore

@@ -10,3 +10,4 @@ docs/
 /.phive/
 /tools/
 /.php-cs-fixer.cache
+.vscode/

src/Endpoint/Abstracts/AbstractSugarBeanEndpoint.php

@@ -169,7 +169,7 @@ protected function configureRequest(Request $request, $data): Request
                 $this->_uploadFile['field'] => $this->_uploadFile['path'],
             ]);
             $data = null;
-        } elseif ($this->getCurrentAction() === self::MODEL_ACTION_RETRIEVE) {
+        } elseif (in_array($this->getCurrentAction(), [self::MODEL_ACTION_RETRIEVE, self::BEAN_ACTION_UPSERT])) {
             $data = $this->configureFieldsDataProps($data);
         }
 

tests/Endpoint/IntegrateTest.php

@@ -99,6 +99,26 @@ public function testUpsert(): void
         $this->assertEquals('test', $endpoint->foobar_c);
         $this->assertEquals('Prospect', $endpoint['account_type']);
     }
+    public function testUpsertWithSetField(): void
+    {
+        $this->client->mockResponses->append(new Response('201'), new Response('201'));
+        $endpoint = new Integrate();
+        $endpoint->setClient($this->client);
+        $endpoint->setModule('Accounts');
+        $endpoint['name'] = 'Test Account';
+        $endpoint['sync_key'] = 'test';
+        $endpoint->upsert();
+        $request = $this->client->mockResponses->getLastRequest();
+        $body = json_decode($request->getBody()->getContents(), true);
+        $this->assertArrayNotHasKey('fields', $body);
+
+        $endpoint->setFields(['foobar', 'bar']);
+        $endpoint->upsert();
+        $request = $this->client->mockResponses->getLastRequest();
+        $body = json_decode($request->getBody()->getContents(), true);
+        $this->assertArrayHasKey('fields', $body);
+        $this->assertEquals("foobar,bar", $body['fields']);
+    }
 
     /**
      * @covers ::getSyncKey