Remove null params

Author: csrbarber

lib/UserManagement.php

@@ -234,11 +234,17 @@ public function createOrganizationMembership($userId, $organizationId, $roleSlug
 
         $params = [
             "organization_id" => $organizationId,
-            "user_id" => $userId,
-            "role_slug" => $roleSlug,
-            "role_slugs" => $roleSlugs
+            "user_id" => $userId
         ];
 
+        if (!is_null($roleSlug)) {
+            $params["role_slug"] = $roleSlug;
+        }
+
+        if (!is_null($roleSlugs)) {
+            $params["role_slugs"] = $roleSlugs;
+        }
+
         $response = Client::request(
             Client::METHOD_POST,
             $path,
@@ -313,10 +319,15 @@ public function updateOrganizationMembership($organizationMembershipId, $roleSlu
     {
         $path = "user_management/organization_memberships/{$organizationMembershipId}";
 
-        $params = [
-            "role_slug" => $roleSlug,
-            "role_slugs" => $roleSlugs
-        ];
+        $params = [];
+
+        if (!is_null($roleSlug)) {
+            $params["role_slug"] = $roleSlug;
+        }
+
+        if (!is_null($roleSlugs)) {
+            $params["role_slugs"] = $roleSlugs;
+        }
 
         $response = Client::request(
             Client::METHOD_PUT,

tests/WorkOS/UserManagementTest.php

@@ -894,7 +894,6 @@ public function testCreateOrganizationMembership()
             "organization_id" => $orgId,
             "user_id" => $userId,
             "role_slug" => $roleSlug,
-            "role_slugs" => null,
         ];
 
         $this->mockRequest(
@@ -925,7 +924,6 @@ public function testCreateOrganizationMembershipWithRoleSlugs()
         $params = [
             "organization_id" => $orgId,
             "user_id" => $userId,
-            "role_slug" => null,
             "role_slugs" => $roleSlugs,
         ];
 
@@ -1104,7 +1102,7 @@ public function testUpdateOrganizationMembership()
             Client::METHOD_PUT,
             $path,
             null,
-            ["role_slug" => $roleSlug, "role_slugs" => null],
+            ["role_slug" => $roleSlug],
             true,
             $result
         );
@@ -1125,7 +1123,7 @@ public function testUpdateOrganizationMembershipWithRoleSlugs()
             Client::METHOD_PUT,
             $path,
             null,
-            ["role_slug" => null, "role_slugs" => $roleSlugs],
+            ["role_slugs" => $roleSlugs],
             true,
             $result
         );