More "item" traces removed

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

diff --git a/src/Worker/Expire.php b/src/Worker/Expire.php
index 2c419c401af9ef7b1268a573b08e095146667e77..31c0af0397ae039e689e18e1977be4058840b588 100644 (file)
--- a/src/Worker/Expire.php
+++ b/src/Worker/Expire.php
@@ -44,25 +44,26 @@ class Expire
                        Logger::log('Delete expired items', Logger::DEBUG);
                        // physically remove anything that has been deleted for more than two months
                        $condition = ["`deleted` AND `changed` < UTC_TIMESTAMP() - INTERVAL 60 DAY"];
-                       $rows = DBA::select('item', ['id', 'guid', 'uri-id', 'uid'],  $condition);
-                       while ($row = DBA::fetch($rows)) {
+                       $rows = Post::select(['id', 'guid', 'uri-id', 'uid'],  $condition);
+                       while ($row = Post::fetch($rows)) {
                                Logger::info('Delete expired item', ['id' => $row['id'], 'guid' => $row['guid']]);
                                DBA::delete('item', ['id' => $row['id']]);
                                Post\User::delete(['uri-id' => $row['uri-id'], 'uid' => $row['uid']]);
+                               Post\ThreadUser::delete(['uri-id' => $row['uri-id'], 'uid' => $row['uid']]);
                        }
                        DBA::close($rows);
 
-                       // Normally we shouldn't have orphaned data at all.
-                       // If we do have some, then we have to check why.
-                       Logger::log('Deleting orphaned item content - start', Logger::DEBUG);
-                       $condition = ["NOT EXISTS (SELECT `uri-id` FROM `item` WHERE `item`.`uri-id` = `post-content`.`uri-id`)"];
+                       Logger::info('Deleting orphaned post-content - start');
+                       /// @todo Replace "item with "post-user" in the future when "item" is removed
+                       $condition = ["NOT EXISTS (SELECT `uri-id` FROM `post-user` WHERE `post-user`.`uri-id` = `post-content`.`uri-id`)"];
                        DBA::delete('post-content', $condition);
-                       Logger::log('Orphaned item content deleted: ' . DBA::affectedRows(), Logger::DEBUG);
+                       Logger::info('Orphaned post-content deleted', ['rows' => DBA::affectedRows()]);
 
-                       // make this optional as it could have a performance impact on large sites
-                       if (intval(DI::config()->get('system', 'optimize_items'))) {
-                               DBA::e("OPTIMIZE TABLE `item`");
-                       }
+                       Logger::info('Deleting orphaned post-thread - start');
+                       /// @todo Replace "item with "post-user" in the future when "item" is removed
+                       $condition = ["NOT EXISTS (SELECT `uri-id` FROM `post-user` WHERE `post-user`.`uri-id` = `post-thread`.`uri-id`)"];
+                       DBA::delete('post-thread', $condition);
+                       Logger::info('Orphaned item content deleted', ['rows' => DBA::affectedRows()]);
 
                        Logger::log('Delete expired items - done', Logger::DEBUG);
                        return;