Nodarx/people_manager
1
0
Fork 0
Code Issues Pull Requests Actions Projects Releases Activity

Merge pull request 'fix: service create / update' (#5) from fix/service-create-update into main

Browse Source 
Reviewed-on: #5
This commit was merged in pull request #5.
This commit is contained in:
Sebastian Krupinski
2026-03-04 03:05:17 +00:00
parent c7d6657254 a4967b9c19
commit bbf0283ac5
1 changed files with 4 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
lib/Manager.php
View File

@@ -177,11 +177,8 @@ class Manager {
throw new InvalidArgumentException("Provider '$providerId' does not support service creation"); throw new InvalidArgumentException("Provider '$providerId' does not support service creation");
} }
// Create a fresh service instance // Create a service instance with provided data
$service = $provider->serviceFresh(); $service = $provider->serviceFresh()->jsonDeserialize($data);
// Deserialize the data into the service
$service->jsonDeserialize($data);
// Create the service // Create the service
$serviceId = $provider->serviceCreate($tenantId, $userId, $service); $serviceId = $provider->serviceCreate($tenantId, $userId, $service);
@@ -218,8 +215,8 @@ class Manager {
throw new InvalidArgumentException("Service '$serviceId' not found"); throw new InvalidArgumentException("Service '$serviceId' not found");
} }
// Update with new data // Create a service instance with updated data
$service->jsonDeserialize($data); $service = $provider->serviceFresh()->jsonDeserialize($data);
// Modify the service // Modify the service
$provider->serviceModify($tenantId, $userId, $service); $provider->serviceModify($tenantId, $userId, $service);