Coding standards

[friendica.git] / src / Worker / RemoveContact.php

diff --git a/src/Worker/RemoveContact.php b/src/Worker/RemoveContact.php
index a84cb0d53760701f9eb9a825b5a24c22fbf3255d..13cbbec8ba18cb4add35ed2eb97406b88aaca5a4 100644 (file)
--- a/src/Worker/RemoveContact.php
+++ b/src/Worker/RemoveContact.php
@@ -1,6 +1,6 @@
 <?php
 /**
- * @copyright Copyright (C) 2020, Friendica
+ * @copyright Copyright (C) 2010-2021, the Friendica project
  *
  * @license GNU AGPL version 3 or any later version
  *
@@ -23,37 +23,53 @@ namespace Friendica\Worker;
 
 use Friendica\Core\Logger;
 use Friendica\Database\DBA;
-use Friendica\Model\Item;
+use Friendica\Database\DBStructure;
 use Friendica\Model\Photo;
+use Friendica\Model\Post;
 
 /**
  * Removes orphaned data from deleted contacts
  */
 class RemoveContact {
        public static function execute($id) {
+               if (empty($id)) {
+                       return;
+               }
+
                // Only delete if the contact is to be deleted
-               $contact = DBA::selectFirst('contact', ['uid'], ['deleted' => true, 'id' => $id]);
+               $contact = DBA::selectFirst('contact', ['id', 'uid', 'url', 'nick', 'name'], ['deleted' => true, 'id' => $id]);
                if (!DBA::isResult($contact)) {
                        return;
                }
 
-               Logger::info('Start deleting contact', ['id' => $id]);
+               Logger::info('Start deleting contact', ['contact' => $contact]);
+
                // Now we delete the contact and all depending tables
-               if ($contact['uid'] == 0) {
-                       DBA::delete('post-tag', ['cid' => $id]);
-                       $condition = ["`author-id` = ? OR `owner-id` = ? OR `causer-id` = ? OR `contact-id` = ?",
-                               $id, $id, $id, $id];
-               } else {
-                       $condition = ['uid' => $contact['uid'], 'contact-id' => $id];
+               DBA::delete('post-tag', ['cid' => $id]);
+
+               if (DBStructure::existsTable('item')) {
+                       DBA::delete('item', ['author-id' => $id]);
+                       DBA::delete('item', ['owner-id' => $id]);
+                       DBA::delete('item', ['causer-id' => $id]);
+                       DBA::delete('item', ['contact-id' => $id]);
                }
-               do {
-                       $items = Item::select(['id', 'guid'], $condition, ['limit' => 100]);
-                       while ($item = Item::fetch($items)) {
-                               Logger::info('Delete removed contact item', ['id' => $item['id'], 'guid' => $item['guid']]);
-                               DBA::delete('item', ['id' => $item['id']]);
-                       }
-                       DBA::close($items);
-               } while (Item::exists($condition));
+
+               DBA::delete('mail', ['contact-id' => $id]);
+
+               Post\ThreadUser::delete(['author-id' => $id]);
+               Post\ThreadUser::delete(['owner-id' => $id]);
+               Post\ThreadUser::delete(['causer-id' => $id]);
+               Post\ThreadUser::delete(['contact-id' => $id]);
+               Post\Thread::delete(['author-id' => $id]);
+               Post\Thread::delete(['owner-id' => $id]);
+               Post\Thread::delete(['causer-id' => $id]);
+               Post\User::delete(['author-id' => $id]);
+               Post\User::delete(['owner-id' => $id]);
+               Post\User::delete(['causer-id' => $id]);
+               Post\User::delete(['contact-id' => $id]);
+               Post::delete(['author-id' => $id]);
+               Post::delete(['owner-id' => $id]);
+               Post::delete(['causer-id' => $id]);
 
                Photo::delete(['contact-id' => $id]);
                $ret = DBA::delete('contact', ['id' => $id]);