X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;f=src%2FWorker%2FMergeContact.php;h=2378d98e173f07ce0a9789855d5e82a03737da6e;hb=34030a736d5e0aa2195c0f472cf69f863c161d83;hp=85b27ef4ac9f977dd53ce9237a1201a733cbc175;hpb=097620b62799c96d610d73410ec07a6b8cdf82f0;p=friendica.git

diff --git a/src/Worker/MergeContact.php b/src/Worker/MergeContact.php
index 85b27ef4ac..2378d98e17 100644
--- a/src/Worker/MergeContact.php
+++ b/src/Worker/MergeContact.php
@@ -1,6 +1,6 @@
 <?php
 /**
- * @copyright Copyright (C) 2020, Friendica
+ * @copyright Copyright (C) 2010-2022, the Friendica project
  *
  * @license GNU AGPL version 3 or any later version
  *
@@ -23,34 +23,55 @@ namespace Friendica\Worker;
 
 use Friendica\Core\Logger;
 use Friendica\Database\DBA;
+use Friendica\Database\DBStructure;
 
 class MergeContact
 {
-	public static function execute($first, $dup_id, $uid)
+	/**
+	 * Replace all occurences of the given contact id and replace it
+	 *
+	 * @param integer $new_cid
+	 * @param integer $old_cid
+	 * @param integer $uid
+	 */
+	public static function execute(int $new_cid, int $old_cid, int $uid)
 	{
-		if (empty($first) || empty($dup_id) || ($first == $dup_id)) {
+		if (empty($new_cid) || empty($old_cid) || ($new_cid == $old_cid)) {
 			// Invalid request
 			return;
 		}
 
-		Logger::info('Handling duplicate', ['search' => $dup_id, 'replace' => $first]);
+		Logger::info('Handling duplicate', ['search' => $old_cid, 'replace' => $new_cid]);
 
-		// Search and replace
-		DBA::update('item', ['contact-id' => $first], ['contact-id' => $dup_id]);
-		DBA::update('thread', ['contact-id' => $first], ['contact-id' => $dup_id]);
-		DBA::update('mail', ['contact-id' => $first], ['contact-id' => $dup_id]);
-		DBA::update('photo', ['contact-id' => $first], ['contact-id' => $dup_id]);
-		DBA::update('event', ['cid' => $first], ['cid' => $dup_id]);
+		foreach (['item', 'thread', 'post-user', 'post-thread-user'] as $table) {
+			if (DBStructure::existsTable($table)) {
+				DBA::update($table, ['contact-id' => $new_cid], ['contact-id' => $old_cid]);
+			}
+		}
+		DBA::update('mail', ['contact-id' => $new_cid], ['contact-id' => $old_cid]);
+		DBA::update('photo', ['contact-id' => $new_cid], ['contact-id' => $old_cid]);
+		DBA::update('event', ['cid' => $new_cid], ['cid' => $old_cid]);
+
+		// These fields only contain public contact entries (uid = 0)
 		if ($uid == 0) {
-			DBA::update('item', ['author-id' => $first], ['author-id' => $dup_id]);
-			DBA::update('item', ['owner-id' => $first], ['owner-id' => $dup_id]);
-			DBA::update('thread', ['author-id' => $first], ['author-id' => $dup_id]);
-			DBA::update('thread', ['owner-id' => $first], ['owner-id' => $dup_id]);
+			DBA::update('post-tag', ['cid' => $new_cid], ['cid' => $old_cid]);
+			DBA::delete('post-tag', ['cid' => $old_cid]);
+			foreach (['item', 'post', 'post-thread', 'post-user', 'post-thread-user'] as $table) {
+				if (DBStructure::existsTable($table)) {
+					DBA::update($table, ['author-id' => $new_cid], ['author-id' => $old_cid]);
+					DBA::update($table, ['owner-id' => $new_cid], ['owner-id' => $old_cid]);
+					DBA::update($table, ['causer-id' => $new_cid], ['causer-id' => $old_cid]);
+				}
+			}
+			if (DBStructure::existsTable('thread')) {
+				DBA::update('thread', ['author-id' => $new_cid], ['author-id' => $old_cid]);
+				DBA::update('thread', ['owner-id' => $new_cid], ['owner-id' => $old_cid]);
+			}
 		} else {
 			/// @todo Check if some other data needs to be adjusted as well, possibly the "rel" status?
 		}
 
 		// Remove the duplicate
-		DBA::delete('contact', ['id' => $dup_id]);
+		DBA::delete('contact', ['id' => $old_cid]);
 	}
 }