nextcloud / server

apps/contactsinteraction/lib/Listeners/ContactInteractionListener.php

Indentation +127 added lines, -127 removed lines

@@ -44,131 +44,131 @@
 
 class ContactInteractionListener implements IEventListener {
 
-	use TTransactional;
-
-	private RecentContactMapper $mapper;
-	private CardSearchDao $cardSearchDao;
-	private IUserManager $userManager;
-	private IDBConnection $dbConnection;
-	private ITimeFactory $timeFactory;
-	private IL10N $l10n;
-	private LoggerInterface $logger;
-
-	public function __construct(RecentContactMapper $mapper,
-								CardSearchDao $cardSearchDao,
-								IUserManager $userManager,
-								IDBConnection $connection,
-								ITimeFactory $timeFactory,
-								IL10N $l10nFactory,
-								LoggerInterface $logger) {
-		$this->mapper = $mapper;
-		$this->cardSearchDao = $cardSearchDao;
-		$this->userManager = $userManager;
-		$this->dbConnection = $connection;
-		$this->timeFactory = $timeFactory;
-		$this->l10n = $l10nFactory;
-		$this->logger = $logger;
-	}
-
-	public function handle(Event $event): void {
-		if (!($event instanceof ContactInteractedWithEvent)) {
-			return;
-		}
-
-		if ($event->getUid() === null && $event->getEmail() === null && $event->getFederatedCloudId() === null) {
-			$this->logger->warning("Contact interaction event has no user identifier set");
-			return;
-		}
-
-		if ($event->getUid() !== null && $event->getUid() === $event->getActor()->getUID()) {
-			$this->logger->info("Ignoring contact interaction with self");
-			return;
-		}
-
-		$this->atomic(function () use ($event) {
-			$uid = $event->getUid();
-			$email = $event->getEmail();
-			$federatedCloudId = $event->getFederatedCloudId();
-			$existing = $this->mapper->findMatch(
-				$event->getActor(),
-				$uid,
-				$email,
-				$federatedCloudId
-			);
-			if (!empty($existing)) {
-				$now = $this->timeFactory->getTime();
-				foreach ($existing as $c) {
-					$c->setLastContact($now);
-					$this->mapper->update($c);
-				}
-
-				return;
-			}
-
-			$contact = new RecentContact();
-			$contact->setActorUid($event->getActor()->getUID());
-			if ($uid !== null) {
-				$contact->setUid($uid);
-			}
-			if ($email !== null) {
-				$contact->setEmail($email);
-			}
-			if ($federatedCloudId !== null) {
-				$contact->setFederatedCloudId($federatedCloudId);
-			}
-			$contact->setLastContact($this->timeFactory->getTime());
-
-			$copy = $this->cardSearchDao->findExisting(
-				$event->getActor(),
-				$uid,
-				$email,
-				$federatedCloudId
-			);
-			if ($copy !== null) {
-				try {
-					$parsed = Reader::read($copy, Reader::OPTION_FORGIVING);
-					$parsed->CATEGORIES = $this->l10n->t('Recently contacted');
-					$contact->setCard($parsed->serialize());
-				} catch (Throwable $e) {
-					$this->logger->warning(
-						'Could not parse card to add recent category: ' . $e->getMessage(),
-						[
-							'exception' => $e,
-						]);
-					$contact->setCard($copy);
-				}
-			} else {
-				$contact->setCard($this->generateCard($contact));
-			}
-			$this->mapper->insert($contact);
-		}, $this->dbConnection);
-	}
-
-	private function getDisplayName(?string $uid): ?string {
-		if ($uid === null) {
-			return null;
-		}
-		if (($user = $this->userManager->get($uid)) === null) {
-			return null;
-		}
-
-		return $user->getDisplayName();
-	}
-
-	private function generateCard(RecentContact $contact): string {
-		$props = [
-			'URI' => UUIDUtil::getUUID(),
-			'FN' => $this->getDisplayName($contact->getUid()) ?? $contact->getEmail() ?? $contact->getFederatedCloudId(),
-			'CATEGORIES' => $this->l10n->t('Recently contacted'),
-		];
-
-		if ($contact->getEmail() !== null) {
-			$props['EMAIL'] = $contact->getEmail();
-		}
-		if ($contact->getFederatedCloudId() !== null) {
-			$props['CLOUD'] = $contact->getFederatedCloudId();
-		}
-
-		return (new VCard($props))->serialize();
-	}
+    use TTransactional;
+
+    private RecentContactMapper $mapper;
+    private CardSearchDao $cardSearchDao;
+    private IUserManager $userManager;
+    private IDBConnection $dbConnection;
+    private ITimeFactory $timeFactory;
+    private IL10N $l10n;
+    private LoggerInterface $logger;
+
+    public function __construct(RecentContactMapper $mapper,
+                                CardSearchDao $cardSearchDao,
+                                IUserManager $userManager,
+                                IDBConnection $connection,
+                                ITimeFactory $timeFactory,
+                                IL10N $l10nFactory,
+                                LoggerInterface $logger) {
+        $this->mapper = $mapper;
+        $this->cardSearchDao = $cardSearchDao;
+        $this->userManager = $userManager;
+        $this->dbConnection = $connection;
+        $this->timeFactory = $timeFactory;
+        $this->l10n = $l10nFactory;
+        $this->logger = $logger;
+    }
+
+    public function handle(Event $event): void {
+        if (!($event instanceof ContactInteractedWithEvent)) {
+            return;
+        }
+
+        if ($event->getUid() === null && $event->getEmail() === null && $event->getFederatedCloudId() === null) {
+            $this->logger->warning("Contact interaction event has no user identifier set");
+            return;
+        }
+
+        if ($event->getUid() !== null && $event->getUid() === $event->getActor()->getUID()) {
+            $this->logger->info("Ignoring contact interaction with self");
+            return;
+        }
+
+        $this->atomic(function () use ($event) {
+            $uid = $event->getUid();
+            $email = $event->getEmail();
+            $federatedCloudId = $event->getFederatedCloudId();
+            $existing = $this->mapper->findMatch(
+                $event->getActor(),
+                $uid,
+                $email,
+                $federatedCloudId
+            );
+            if (!empty($existing)) {
+                $now = $this->timeFactory->getTime();
+                foreach ($existing as $c) {
+                    $c->setLastContact($now);
+                    $this->mapper->update($c);
+                }
+
+                return;
+            }
+
+            $contact = new RecentContact();
+            $contact->setActorUid($event->getActor()->getUID());
+            if ($uid !== null) {
+                $contact->setUid($uid);
+            }
+            if ($email !== null) {
+                $contact->setEmail($email);
+            }
+            if ($federatedCloudId !== null) {
+                $contact->setFederatedCloudId($federatedCloudId);
+            }
+            $contact->setLastContact($this->timeFactory->getTime());
+
+            $copy = $this->cardSearchDao->findExisting(
+                $event->getActor(),
+                $uid,
+                $email,
+                $federatedCloudId
+            );
+            if ($copy !== null) {
+                try {
+                    $parsed = Reader::read($copy, Reader::OPTION_FORGIVING);
+                    $parsed->CATEGORIES = $this->l10n->t('Recently contacted');
+                    $contact->setCard($parsed->serialize());
+                } catch (Throwable $e) {
+                    $this->logger->warning(
+                        'Could not parse card to add recent category: ' . $e->getMessage(),
+                        [
+                            'exception' => $e,
+                        ]);
+                    $contact->setCard($copy);
+                }
+            } else {
+                $contact->setCard($this->generateCard($contact));
+            }
+            $this->mapper->insert($contact);
+        }, $this->dbConnection);
+    }
+
+    private function getDisplayName(?string $uid): ?string {
+        if ($uid === null) {
+            return null;
+        }
+        if (($user = $this->userManager->get($uid)) === null) {
+            return null;
+        }
+
+        return $user->getDisplayName();
+    }
+
+    private function generateCard(RecentContact $contact): string {
+        $props = [
+            'URI' => UUIDUtil::getUUID(),
+            'FN' => $this->getDisplayName($contact->getUid()) ?? $contact->getEmail() ?? $contact->getFederatedCloudId(),
+            'CATEGORIES' => $this->l10n->t('Recently contacted'),
+        ];
+
+        if ($contact->getEmail() !== null) {
+            $props['EMAIL'] = $contact->getEmail();
+        }
+        if ($contact->getFederatedCloudId() !== null) {
+            $props['CLOUD'] = $contact->getFederatedCloudId();
+        }
+
+        return (new VCard($props))->serialize();
+    }
 }

Spacing +2 added lines, -2 removed lines

@@ -85,7 +85,7 @@ 
 			return;
 		}
 
-		$this->atomic(function () use ($event) {
+		$this->atomic(function() use ($event) {
 			$uid = $event->getUid();
 			$email = $event->getEmail();
 			$federatedCloudId = $event->getFederatedCloudId();
@@ -131,7 +131,7 @@ 
 					$contact->setCard($parsed->serialize());
 				} catch (Throwable $e) {
 					$this->logger->warning(
-						'Could not parse card to add recent category: ' . $e->getMessage(),
+						'Could not parse card to add recent category: '.$e->getMessage(),
 						[
 							'exception' => $e,
 						]);