Merge pull request #1779 from nextcloud/fix/noid/update-displayname-only-when-needed

update details only

2 files changed, 12 insertions, 2 deletions

diff --git a/lib/Service/AccountService.php b/lib/Service/AccountService.php
index a462962b..1fbfed90 100644
--- a/lib/Service/AccountService.php
+++ b/lib/Service/AccountService.php
@@ -318,7 +318,7 @@ class AccountService {
 		}
 
 		$this->addLocalActorDetailCount($actor);
-		$this->actorService->cacheLocalActor($actor);
+		$this->actorService->cacheLocalActorDetails($actor);
 	}
 
 
@@ -348,7 +348,7 @@ class AccountService {
 	 *
 	 * @throws NoUserException
 	 */
-	private function updateCacheLocalActorName(Person &$actor) {
+	private function updateCacheLocalActorName(Person $actor) {
 		$user = $this->userManager->get($actor->getUserId());
 		if ($user === null) {
 			throw new NoUserException();
diff --git a/lib/Service/ActorService.php b/lib/Service/ActorService.php
index 17cad663..3a84f27c 100644
--- a/lib/Service/ActorService.php
+++ b/lib/Service/ActorService.php
@@ -106,6 +106,16 @@ class ActorService {
 	 *
 	 * @throws ItemAlreadyExistsException
 	 */
+	public function cacheLocalActorDetails(Person $actor) {
+		$this->cacheActorsRequest->updateDetails($actor);
+	}
+
+
+	/**
+	 * @param Person $actor
+	 *
+	 * @throws ItemAlreadyExistsException
+	 */
 	public function save(Person $actor) {
 		$this->cacheDocumentIfNeeded($actor);
 		$this->cacheActorsRequest->save($actor);