diff --git a/resources/lib/UnityGroup.php b/resources/lib/UnityGroup.php
index 0c374fae0..3b8881aa0 100644
--- a/resources/lib/UnityGroup.php
+++ b/resources/lib/UnityGroup.php
@@ -196,10 +196,6 @@ public function approveUser(UnityUser $new_user, bool $send_mail = true): void
         \ensure($new_user->exists());
         $this->addMemberUID($new_user->uid);
         $this->SQL->removeRequest($new_user->uid, $this->gid);
-        $this->SQL->addLog(
-            "approved_user",
-            _json_encode(["user" => $new_user->uid, "group" => $this->gid]),
-        );
         if ($send_mail) {
             $this->MAILER->sendMail($new_user->getMail(), "group_user_added", [
                 "group" => $this->gid,
@@ -255,10 +251,6 @@ public function removeUser(UnityUser $new_user, bool $send_mail = true): void
             throw new Exception("Cannot delete group owner from group. Disable group instead");
         }
         $this->removeMemberUID($new_user->uid);
-        $this->SQL->addLog(
-            "removed_user",
-            _json_encode(["user" => $new_user->uid, "group" => $this->gid]),
-        );
         if ($send_mail) {
             $this->MAILER->sendMail($new_user->getMail(), "group_user_removed", [
                 "group" => $this->gid,
@@ -482,12 +474,19 @@ public function getManagerUIDs(): array
         return $this->entry->getAttribute("managerUid");
     }
 
+    public function addMemberUID(string $uid): void
+    {
+        parent::addMemberUID($uid);
+        $this->SQL->addLog("added_user", _json_encode(["user" => $uid, "group" => $this->gid]));
+    }
+
     public function removeMemberUID(string $uid): void
     {
         if ($this->managerUIDExists($uid)) {
             $this->removeManagerUID($uid);
         }
         parent::removeMemberUID($uid);
+        $this->SQL->addLog("removed_user", _json_encode(["user" => $uid, "group" => $this->gid]));
     }
 
     public function addPlusAddressToMail(string $mail): string