change to immutable

amocrm · Nov 21, 2023 · 9818fc3 · 9818fc3
1 parent 9e7765d
commit 9818fc3
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 7 deletions.
diff --git a/README.md b/README.md
@@ -106,11 +106,11 @@ $accessToken = $apiClient->getOAuthClient()->getAccessTokenByCode($_GET['code'])
 ### Авторизация с правами конкретного пользователя аккаунта
 Начиная с версии 1.4.0 появилась возможность авторизоваться с правами конкретного пользователя, если токен был выпущен администратором аккаунта.
 
-Для авторизации под пользователем аккаунта - необходимо задать ID пользователя у объекта типа ```\AmoCRM\Client\AmoCRMApiClient```.
+Для авторизации под пользователем аккаунта - необходимо задать ID пользователя у объекта типа ```\AmoCRM\Client\AmoCRMApiClient```. Метод вернет новый объект с установленным контекстом.
 
 ```php
 $apiClient = new \AmoCRM\Client\AmoCRMApiClient($clientId, $clientSecret, $redirectUri);
-$apiClient->setContextUserId(123);
+$apiClientWithContext = $apiClient->withContextUserId(123);
 ```
 
 

diff --git a/examples/user_context_actions.php b/examples/user_context_actions.php
@@ -23,11 +23,11 @@ function (AccessTokenInterface $accessToken, string $baseDomain) {
     );
 
 $contextUserId = 123;
-$apiClient->setContextUserId($contextUserId);
+$apiClientWithContext = $apiClient->withContextUserId($contextUserId);
 
 //Получим свойства аккаунта и сравним юезра
 try {
-    $account = $apiClient->account()->getCurrent();
+    $account = $apiClientWithContext->account()->getCurrent();
 
     echo 'Текущий юзер, тот кого вы передали? - ' . ($account->getCurrentUserId() === $contextUserId ? 'да' : 'нет');
 } catch (AmoCRMApiException $e) {

diff --git a/src/AmoCRM/Client/AmoCRMApiClient.php b/src/AmoCRM/Client/AmoCRMApiClient.php
@@ -115,15 +115,17 @@ public function getContextUserId(): ?int
 
     /**
      * Для админских токеном можно задать пользователя аккаунта, в контексте которого будет сделан запрос
+     * Метод возвращает новый объект апи клиента с установленным контекстом
      * @param int|null $contextUserId
      *
      * @return $this
      */
-    public function setContextUserId(?int $contextUserId): AmoCRMApiClient
+    public function withContextUserId(?int $contextUserId): AmoCRMApiClient
     {
-        $this->contextUserId = $contextUserId;
+        $apiClient = clone $this;
+        $apiClient->contextUserId = $contextUserId;
 
-        return $this;
+        return $apiClient;
     }
 
     /**