]> git.mxchange.org Git - friendica.git/blobdiff - src/Model/Post.php
Posts per author/server on the community pages (#13764)
[friendica.git] / src / Model / Post.php
index 5cafdf25aea1623f9795ea80ba649d81948609ae..2e8bbe22e3650fdc7ffd265f5c3428d37b2b608d 100644 (file)
@@ -1,6 +1,6 @@
 <?php
 /**
- * @copyright Copyright (C) 2020, Friendica
+ * @copyright Copyright (C) 2010-2023, the Friendica project
  *
  * @license GNU AGPL version 3 or any later version
  *
 
 namespace Friendica\Model;
 
+use BadMethodCallException;
 use Friendica\Core\Logger;
+use Friendica\Database\Database;
 use Friendica\Database\DBA;
-use Friendica\Database\DBStructure;
+use Friendica\DI;
 use Friendica\Protocol\Activity;
 
 class Post
 {
+       /**
+        * Insert a new post entry
+        *
+        * @param integer $uri_id
+        * @param array   $fields
+        * @return bool   Success of the insert process
+        * @throws \Exception
+        */
+       public static function insert(int $uri_id, array $data = []): bool
+       {
+               if (empty($uri_id)) {
+                       throw new BadMethodCallException('Empty URI_id');
+               }
+
+               $fields = DI::dbaDefinition()->truncateFieldsForTable('post', $data);
+
+               // Additionally assign the key fields
+               $fields['uri-id'] = $uri_id;
+
+               return DBA::insert('post', $fields, Database::INSERT_IGNORE);
+       }
+
        /**
         * Fetch a single post row
         *
@@ -65,45 +89,51 @@ class Post
                        }
                }
 
+               if (array_key_exists('extid', $row) && is_null($row['extid'])) {
+                       $row['extid'] = '';
+               }
+
                return $row;
        }
 
        /**
-        * Fills an array with data from an post query
+        * Fills an array with data from a post query
         *
-        * @param object $stmt statement object
-        * @param bool   $do_close
+        * @param object|bool $stmt Return value from Database->select
         * @return array Data array
+        * @throws \Exception
         */
-       public static function toArray($stmt, $do_close = true) {
+       public static function toArray($stmt): array
+       {
                if (is_bool($stmt)) {
-                       return $stmt;
+                       return [];
                }
 
                $data = [];
                while ($row = self::fetch($stmt)) {
                        $data[] = $row;
                }
-               if ($do_close) {
-                       DBA::close($stmt);
-               }
+
+               DBA::close($stmt);
+
                return $data;
        }
 
        /**
-        * Check if post data exists
+        * Check if post-user-view records exists
         *
         * @param array $condition array of fields for condition
         *
         * @return boolean Are there rows for that condition?
         * @throws \Exception
         */
-       public static function exists($condition) {
-               return DBA::exists('post-view', $condition);
+       public static function exists(array $condition): bool
+       {
+               return DBA::exists('post-user-view', $condition);
        }
 
        /**
-        * Counts the posts satisfying the provided condition
+        * Counts the post-user-view records satisfying the provided condition
         *
         * @param array        $condition array of fields for condition
         * @param array        $params    Array of several parameters
@@ -118,22 +148,65 @@ class Post
         * $count = Post::count($condition);
         * @throws \Exception
         */
-       public static function count(array $condition = [], array $params = [])
+       public static function count(array $condition = [], array $params = []): int
+       {
+               return DBA::count('post-user-view', $condition, $params);
+       }
+
+       /**
+        * Counts the post-thread-user-view records satisfying the provided condition
+        *
+        * @param array        $condition array of fields for condition
+        * @param array        $params    Array of several parameters
+        *
+        * @return int
+        *
+        * Example:
+        * $condition = ["uid" => 1, "network" => 'dspr'];
+        * or:
+        * $condition = ["`uid` = ? AND `network` IN (?, ?)", 1, 'dfrn', 'dspr'];
+        *
+        * $count = Post::count($condition);
+        * @throws \Exception
+        */
+       public static function countThread(array $condition = [], array $params = []): int
+       {
+               return DBA::count('post-thread-user-view', $condition, $params);
+       }
+
+       /**
+        * Counts the post-view records satisfying the provided condition
+        *
+        * @param array        $condition array of fields for condition
+        * @param array        $params    Array of several parameters
+        *
+        * @return int
+        *
+        * Example:
+        * $condition = ["network" => 'dspr'];
+        * or:
+        * $condition = ["`network` IN (?, ?)", 1, 'dfrn', 'dspr'];
+        *
+        * $count = Post::count($condition);
+        * @throws \Exception
+        */
+       public static function countPosts(array $condition = [], array $params = []): int
        {
                return DBA::count('post-view', $condition, $params);
        }
 
        /**
-        * Retrieve a single record from the post table and returns it in an associative array
+        * Retrieve a single record from the post-user-view view and returns it in an associative array
         *
         * @param array $fields
         * @param array $condition
         * @param array $params
+        * @param bool  $user_mode true = post-user-view, false = post-view
         * @return bool|array
         * @throws \Exception
         * @see   DBA::select
         */
-       public static function selectFirst(array $fields = [], array $condition = [], $params = [])
+       public static function selectFirst(array $fields = [], array $condition = [], array $params = [])
        {
                $params['limit'] = 1;
 
@@ -149,7 +222,97 @@ class Post
        }
 
        /**
-        * Select rows from the post table and returns them as an array
+        * Retrieve a single record from the post-user-view view and returns it in an associative array
+        * When the requested record is a reshare activity, the system fetches the reshared original post.
+        * Otherwise the function reacts similar to selectFirst
+        *
+        * @param array $fields
+        * @param array $condition
+        * @param array $params
+        * @param bool  $user_mode true = post-user-view, false = post-view
+        * @return bool|array
+        * @throws \Exception
+        * @see   DBA::select
+        */
+       public static function selectOriginal(array $fields = [], array $condition = [], array $params = [])
+       {
+               $original_fields = $fields;
+               $remove = [];
+               if (!empty($fields)) {
+                       foreach (['gravity', 'verb', 'thr-parent-id', 'uid'] as $field) {
+                               if (!in_array($field, $fields)) {
+                                       $fields[] = $field;
+                                       $remove[] = $field;
+                               }
+                       }
+               }
+               $result = self::selectFirst($fields, $condition, $params);
+               if (empty($result)) {
+                       return $result;
+               }
+
+               if (($result['gravity'] != Item::GRAVITY_ACTIVITY) || ($result['verb'] != Activity::ANNOUNCE)) {
+                       foreach ($remove as $field) {
+                               unset($result[$field]);
+                       }
+                       return $result;
+               }
+
+               return self::selectFirst($original_fields, ['uri-id' => $result['thr-parent-id'], 'uid' => [0, $result['uid']]], $params);
+       }
+
+       /**
+        * Retrieve a single record from the post-view view and returns it in an associative array
+        *
+        * @param array $fields
+        * @param array $condition
+        * @param array $params
+        * @return bool|array
+        * @throws \Exception
+        * @see   DBA::select
+        */
+       public static function selectFirstPost(array $fields = [], array $condition = [], array $params = [])
+       {
+               $params['limit'] = 1;
+
+               $result = self::selectPosts($fields, $condition, $params);
+
+               if (is_bool($result)) {
+                       return $result;
+               } else {
+                       $row = self::fetch($result);
+                       DBA::close($result);
+                       return $row;
+               }
+       }
+
+       /**
+        * Retrieve a single record from the post-thread-user-view view and returns it in an associative array
+        *
+        * @param array $fields
+        * @param array $condition
+        * @param array $params
+        * @return bool|array
+        * @throws \Exception
+        * @see   DBA::select
+        */
+       public static function selectFirstThread(array $fields = [], array $condition = [], array $params = [])
+       {
+               $params['limit'] = 1;
+
+               $result = self::selectThread($fields, $condition, $params);
+
+               if (is_bool($result)) {
+                       return $result;
+               } else {
+                       $row = self::fetch($result);
+                       DBA::close($result);
+                       return $row;
+               }
+       }
+
+       /**
+        * Select rows from the post-user-view view and returns them as an array
         *
         * @param array $selected  Array of selected fields, empty for all
         * @param array $condition Array of fields for condition
@@ -158,7 +321,7 @@ class Post
         * @return array
         * @throws \Exception
         */
-       public static function selectToArray(array $fields = [], array $condition = [], $params = [])
+       public static function selectToArray(array $fields = [], array $condition = [], array $params = [])
        {
                $result = self::select($fields, $condition, $params);
 
@@ -178,7 +341,7 @@ class Post
        /**
         * Select rows from the given view
         *
-        * @param string $view      View (post-view or post-thread-view)
+        * @param string $view      View (post-user-view or post-thread-user-view)
         * @param array  $selected  Array of selected fields, empty for all
         * @param array  $condition Array of fields for condition
         * @param array  $params    Array of several parameters
@@ -186,16 +349,13 @@ class Post
         * @return boolean|object
         * @throws \Exception
         */
-       private static function selectView(string $view, array $selected = [], array $condition = [], $params = [])
+       private static function selectView(string $view, array $selected = [], array $condition = [], array $params = [])
        {
                if (empty($selected)) {
-                       $selected = array_merge(['author-addr', 'author-nick', 'owner-addr', 'owner-nick', 'causer-addr', 'causer-nick',
-                               'causer-network', 'photo', 'name-date', 'uri-date', 'avatar-date', 'thumb', 'dfrn-id',
-                               'parent-guid', 'parent-network', 'parent-author-id', 'parent-author-link', 'parent-author-name',
-                               'parent-author-network', 'signed_text', 'language', 'raw-body'], Item::DISPLAY_FIELDLIST, Item::ITEM_FIELDLIST);
-                       
-                       if ($view == 'post-thread-view') {
-                               $selected = array_merge($selected, ['ignored', 'iid']);
+                       $selected = array_merge(Item::DISPLAY_FIELDLIST, Item::ITEM_FIELDLIST);
+
+                       if ($view == 'post-thread-user-view') {
+                               $selected = array_merge($selected, ['ignored']);
                        }
                }
 
@@ -205,7 +365,7 @@ class Post
        }
 
        /**
-        * Select rows from the post table
+        * Select rows from the post-user-view view
         *
         * @param array $selected  Array of selected fields, empty for all
         * @param array $condition Array of fields for condition
@@ -214,13 +374,43 @@ class Post
         * @return boolean|object
         * @throws \Exception
         */
-       public static function select(array $selected = [], array $condition = [], $params = [])
+       public static function select(array $selected = [], array $condition = [], array $params = [])
+       {
+               return self::selectView('post-user-view', $selected, $condition, $params);
+       }
+
+       /**
+        * Select rows from the post-view view
+        *
+        * @param array $selected  Array of selected fields, empty for all
+        * @param array $condition Array of fields for condition
+        * @param array $params    Array of several parameters
+        *
+        * @return boolean|object
+        * @throws \Exception
+        */
+       public static function selectPosts(array $selected = [], array $condition = [], array $params = [])
        {
                return self::selectView('post-view', $selected, $condition, $params);
        }
 
        /**
-        * Select rows from the post table
+        * Select rows from the post-thread-user-view view
+        *
+        * @param array $selected  Array of selected fields, empty for all
+        * @param array $condition Array of fields for condition
+        * @param array $params    Array of several parameters
+        *
+        * @return boolean|object
+        * @throws \Exception
+        */
+       public static function selectThread(array $selected = [], array $condition = [], array $params = [])
+       {
+               return self::selectView('post-thread-user-view', $selected, $condition, $params);
+       }
+
+       /**
+        * Select rows from the post-thread-view view
         *
         * @param array $selected  Array of selected fields, empty for all
         * @param array $condition Array of fields for condition
@@ -229,7 +419,7 @@ class Post
         * @return boolean|object
         * @throws \Exception
         */
-       public static function selectThread(array $selected = [], array $condition = [], $params = [])
+       public static function selectPostThread(array $selected = [], array $condition = [], array $params = [])
        {
                return self::selectView('post-thread-view', $selected, $condition, $params);
        }
@@ -237,7 +427,7 @@ class Post
        /**
         * Select rows from the given view for a given user
         *
-        * @param string  $view      View (post-view or post-thread-view)
+        * @param string  $view      View (post-user-view or post-thread-user-view)
         * @param integer $uid       User ID
         * @param array   $selected  Array of selected fields, empty for all
         * @param array   $condition Array of fields for condition
@@ -246,39 +436,27 @@ class Post
         * @return boolean|object
         * @throws \Exception
         */
-       private static function selectViewForUser(string $view, $uid, array $selected = [], array $condition = [], $params = [])
+       private static function selectViewForUser(string $view, int $uid, array $selected = [], array $condition = [], array $params = [])
        {
                if (empty($selected)) {
                        $selected = Item::DISPLAY_FIELDLIST;
                }
 
                $condition = DBA::mergeConditions($condition,
-                       ["`visible` AND NOT `deleted` AND NOT `moderated`
+                       ["`visible` AND NOT `deleted`
                        AND NOT `author-blocked` AND NOT `owner-blocked`
-                       AND (NOT `causer-blocked` OR `causer-id` = ?) AND NOT `contact-blocked`
+                       AND (NOT `causer-blocked` OR `causer-id` = ? OR `causer-id` IS NULL) AND NOT `contact-blocked`
                        AND ((NOT `contact-readonly` AND NOT `contact-pending` AND (`contact-rel` IN (?, ?)))
-                               OR `self` OR `gravity` != ? OR `contact-uid` = ?)
-                       AND NOT EXISTS (SELECT `uri-id` FROM `post-user` WHERE `hidden` AND `uri-id` = `" . $view . "`.`uri-id` AND `uid` = ?)
-                       AND NOT EXISTS (SELECT `cid` FROM `user-contact` WHERE `uid` = ? AND `cid` = `author-id` AND `blocked`)
-                       AND NOT EXISTS (SELECT `cid` FROM `user-contact` WHERE `uid` = ? AND `cid` = `owner-id` AND `blocked`)
-                       AND NOT EXISTS (SELECT `cid` FROM `user-contact` WHERE `uid` = ? AND `cid` = `author-id` AND `ignored` AND `gravity` = ?)
-                       AND NOT EXISTS (SELECT `cid` FROM `user-contact` WHERE `uid` = ? AND `cid` = `owner-id` AND `ignored` AND `gravity` = ?)",
-                       0, Contact::SHARING, Contact::FRIEND, GRAVITY_PARENT, 0, $uid, $uid, $uid, $uid, GRAVITY_PARENT, $uid, GRAVITY_PARENT]);
-
-               $select_string = '';
-
-               if (in_array('pinned', $selected)) {
-                       $selected = array_flip($selected);
-                       unset($selected['pinned']);
-                       $selected = array_flip($selected);      
-
-                       $select_string = "(SELECT `pinned` FROM `post-thread-user` WHERE `uri-id` = `" . $view . "`.`uri-id` AND uid=`" . $view . "`.`uid`) AS `pinned`, ";
-               }
+                               OR `self` OR `contact-uid` = ?)
+                       AND NOT EXISTS(SELECT `uri-id` FROM `post-user`    WHERE `uid` = ? AND `uri-id` = " . DBA::quoteIdentifier($view) . ".`uri-id` AND `hidden`)
+                       AND NOT EXISTS(SELECT `cid`    FROM `user-contact` WHERE `uid` = ? AND `cid` IN (`author-id`, `owner-id`) AND (`blocked` OR `ignored`))
+                       AND NOT EXISTS(SELECT `gsid`   FROM `user-gserver` WHERE `uid` = ? AND `gsid` IN (`author-gsid`, `owner-gsid`, `causer-gsid`) AND `ignored`)",
+                               0, Contact::SHARING, Contact::FRIEND, 0, $uid, $uid, $uid]);
 
-               $select_string .= implode(', ', array_map([DBA::class, 'quoteIdentifier'], $selected));
+               $select_string = implode(', ', array_map([DBA::class, 'quoteIdentifier'], $selected));
 
                $condition_string = DBA::buildCondition($condition);
-               $param_string = DBA::buildParameter($params);
+               $param_string     = DBA::buildParameter($params);
 
                $sql = "SELECT " . $select_string . " FROM `" . $view . "` " . $condition_string . $param_string;
                $sql = DBA::cleanQuery($sql);
@@ -287,7 +465,7 @@ class Post
        }
 
        /**
-        * Select rows from the post view for a given user
+        * Select rows from the post-user-view view for a given user
         *
         * @param integer $uid       User ID
         * @param array   $selected  Array of selected fields, empty for all
@@ -297,13 +475,46 @@ class Post
         * @return boolean|object
         * @throws \Exception
         */
-       public static function selectForUser($uid, array $selected = [], array $condition = [], $params = [])
+       public static function selectForUser(int $uid, array $selected = [], array $condition = [], array $params = [])
+       {
+               return self::selectViewForUser('post-user-view', $uid, $selected, $condition, $params);
+       }
+
+       /**
+        * Select rows from the post-view view for a given user
+        *
+        * @param integer $uid       User ID
+        * @param array   $selected  Array of selected fields, empty for all
+        * @param array   $condition Array of fields for condition
+        * @param array   $params    Array of several parameters
+        *
+        * @return boolean|object
+        * @throws \Exception
+        */
+       public static function selectPostsForUser(int $uid, array $selected = [], array $condition = [], array $params = [])
        {
                return self::selectViewForUser('post-view', $uid, $selected, $condition, $params);
        }
 
-               /**
-        * Select rows from the post view for a given user
+       /**
+        * Select rows from the post-timeline-view view for a given user
+        * This function is used for API calls.
+        *
+        * @param integer $uid       User ID
+        * @param array   $selected  Array of selected fields, empty for all
+        * @param array   $condition Array of fields for condition
+        * @param array   $params    Array of several parameters
+        *
+        * @return boolean|object
+        * @throws \Exception
+        */
+       public static function selectTimelineForUser(int $uid, array $selected = [], array $condition = [], array $params = [])
+       {
+               return self::selectViewForUser('post-timeline-view', $uid, $selected, $condition, $params);
+       }
+
+       /**
+        * Select rows from the post-thread-user-view view for a given user
         *
         * @param integer $uid       User ID
         * @param array   $selected  Array of selected fields, empty for all
@@ -313,13 +524,13 @@ class Post
         * @return boolean|object
         * @throws \Exception
         */
-       public static function selectThreadForUser($uid, array $selected = [], array $condition = [], $params = [])
+       public static function selectThreadForUser(int $uid, array $selected = [], array $condition = [], array $params = [])
        {
-               return self::selectViewForUser('post-thread-view', $uid, $selected, $condition, $params);
+               return self::selectViewForUser('post-thread-user-view', $uid, $selected, $condition, $params);
        }
 
        /**
-        * Retrieve a single record from the post view for a given user and returns it in an associative array
+        * Retrieve a single record from the post-user-view view for a given user and returns it in an associative array
         *
         * @param integer $uid User ID
         * @param array   $selected
@@ -329,7 +540,7 @@ class Post
         * @throws \Exception
         * @see   DBA::select
         */
-       public static function selectFirstForUser($uid, array $selected = [], array $condition = [], $params = [])
+       public static function selectFirstForUser(int $uid, array $selected = [], array $condition = [], array $params = [])
        {
                $params['limit'] = 1;
 
@@ -345,36 +556,43 @@ class Post
        }
 
        /**
-        * Select pinned rows from the item table for a given user
+        * Retrieve a single record from the post-user-view view for a given user and returns it in an associative array
+        * When the requested record is a reshare activity, the system fetches the reshared original post.
+        * Otherwise the function reacts similar to selectFirstForUser
         *
-        * @param integer $uid       User ID
-        * @param array   $selected  Array of selected fields, empty for all
-        * @param array   $condition Array of fields for condition
-        * @param array   $params    Array of several parameters
-        *
-        * @return boolean|object
+        * @param integer $uid User ID
+        * @param array   $selected
+        * @param array   $condition
+        * @param array   $params
+        * @return bool|array
         * @throws \Exception
+        * @see   DBA::select
         */
-       public static function selectPinned(int $uid, array $selected = [], array $condition = [], $params = [])
+       public static function selectOriginalForUser(int $uid, array $selected = [], array $condition = [], array $params = [])
        {
-               $postthreaduser = DBA::select('post-thread-user', ['uri-id'], ['uid' => $uid, 'pinned' => true]);
-               if (!DBA::isResult($postthreaduser)) {
-                       return $postthreaduser;
+               $original_selected = $selected;
+               $remove = [];
+               if (!empty($selected)) {
+                       foreach (['gravity', 'verb', 'thr-parent-id'] as $field) {
+                               if (!in_array($field, $selected)) {
+                                       $selected[] = $field;
+                                       $remove[]   = $field;
+                               }
+                       }
                }
-       
-               $pinned = [];
-               while ($useritem = DBA::fetch($postthreaduser)) {
-                       $pinned[] = $useritem['uri-id'];
+               $result = self::selectFirstForUser($uid, $selected, $condition, $params);
+               if (empty($result)) {
+                       return $result;
                }
-               DBA::close($postthreaduser);
 
-               if (empty($pinned)) {
-                       return [];
+               if (($result['gravity'] != Item::GRAVITY_ACTIVITY) || ($result['verb'] != Activity::ANNOUNCE)) {
+                       foreach ($remove as $field) {
+                               unset($result[$field]);
+                       }
+                       return $result;
                }
 
-               $condition = DBA::mergeConditions(['uri-id' => $pinned, 'uid' => $uid, 'gravity' => GRAVITY_PARENT], $condition);
-
-               return self::selectForUser($uid, $selected, $condition, $params);
+               return self::selectFirstForUser($uid, $original_selected, ['uri-id' => $result['thr-parent-id'], 'uid' => [0, $uid]], $params);
        }
 
        /**
@@ -392,7 +610,7 @@ class Post
        {
                $affected = 0;
 
-               Logger::info('Start Update', ['fields' => $fields, 'condition' => $condition]);
+               Logger::info('Start Update', ['fields' => $fields, 'condition' => $condition, 'uid' => DI::userSession()->getLocalUserId()]);
 
                // Don't allow changes to fields that are responsible for the relation between the records
                unset($fields['id']);
@@ -405,95 +623,117 @@ class Post
                unset($fields['parent-uri']);
                unset($fields['parent-uri-id']);
 
-               $thread_condition = DBA::mergeConditions($condition, ['gravity' => GRAVITY_PARENT]);
+               $thread_condition = DBA::mergeConditions($condition, ['gravity' => Item::GRAVITY_PARENT]);
 
                // To ensure the data integrity we do it in an transaction
                DBA::transaction();
 
-               $update_fields = DBStructure::getFieldsForTable('post-user', $fields);
+               $update_fields = DI::dbaDefinition()->truncateFieldsForTable('post-user', $fields);
                if (!empty($update_fields)) {
-                       $rows = DBA::selectToArray('post-view', ['post-user-id'], $condition);
-                       $puids = array_column($rows, 'post-user-id');
-                       if (!DBA::update('post-user', $update_fields, ['id' => $puids])) {
-                               DBA::rollback();
-                               Logger::notice('Updating post-user failed', ['fields' => $update_fields, 'condition' => $condition]);
-                               return false;
+                       $affected_count = 0;
+                       $posts          = DBA::select('post-user-view', ['post-user-id'], $condition);
+                       while ($rows = DBA::toArray($posts, false, 100)) {
+                               $puids = array_column($rows, 'post-user-id');
+                               if (!DBA::update('post-user', $update_fields, ['id' => $puids])) {
+                                       DBA::rollback();
+                                       Logger::warning('Updating post-user failed', ['fields' => $update_fields, 'condition' => $condition]);
+                                       return false;
+                               }
+                               $affected_count += DBA::affectedRows();
                        }
-                       $affected = DBA::affectedRows();                        
+                       DBA::close($posts);
+                       $affected = $affected_count;
                }
 
-               $update_fields = DBStructure::getFieldsForTable('post-content', $fields);
+               $update_fields = DI::dbaDefinition()->truncateFieldsForTable('post-content', $fields);
                if (!empty($update_fields)) {
-                       $rows = DBA::selectToArray('post-view', ['uri-id'], $condition, ['group_by' => ['uri-id']]);
-                       $uriids = array_column($rows, 'uri-id');
-                       if (!DBA::update('post-content', $update_fields, ['uri-id' => $uriids])) {
-                               DBA::rollback();
-                               Logger::notice('Updating post-content failed', ['fields' => $update_fields, 'condition' => $condition]);
-                               return false;
+                       $affected_count = 0;
+                       $posts          = DBA::select('post-user-view', ['uri-id'], $condition, ['group_by' => ['uri-id']]);
+                       while ($rows = DBA::toArray($posts, false, 100)) {
+                               $uriids = array_column($rows, 'uri-id');
+                               if (!DBA::update('post-content', $update_fields, ['uri-id' => $uriids])) {
+                                       DBA::rollback();
+                                       Logger::warning('Updating post-content failed', ['fields' => $update_fields, 'condition' => $condition]);
+                                       return false;
+                               }
+                               $affected_count += DBA::affectedRows();
                        }
-                       $affected = max($affected, DBA::affectedRows());
+                       DBA::close($posts);
+                       $affected = max($affected, $affected_count);
                }
 
-               $update_fields = Post\DeliveryData::extractFields($fields);
+               $update_fields = DI::dbaDefinition()->truncateFieldsForTable('post', $fields);
                if (!empty($update_fields)) {
-                       if (empty($uriids)) {
-                               $rows = DBA::selectToArray('post-view', ['uri-id'], $condition, ['group_by' => ['uri-id']]);
+                       $affected_count = 0;
+                       $posts          = DBA::select('post-user-view', ['uri-id'], $condition, ['group_by' => ['uri-id']]);
+                       while ($rows = DBA::toArray($posts, false, 100)) {
                                $uriids = array_column($rows, 'uri-id');
+
+                               // Only delete the "post" entry when all "post-user" entries are deleted
+                               if (!empty($update_fields['deleted']) && DBA::exists('post-user', ['uri-id' => $uriids, 'deleted' => false])) {
+                                       unset($update_fields['deleted']);
+                               }
+
+                               if (!DBA::update('post', $update_fields, ['uri-id' => $uriids])) {
+                                       DBA::rollback();
+                                       Logger::warning('Updating post failed', ['fields' => $update_fields, 'condition' => $condition]);
+                                       return false;
+                               }
+                               $affected_count += DBA::affectedRows();
                        }
-                       if (!DBA::update('post-delivery-data', $update_fields, ['uri-id' => $uriids])) {
-                               DBA::rollback();
-                               Logger::notice('Updating post-delivery-data failed', ['fields' => $update_fields, 'condition' => $condition]);
-                               return false;
-                       }
-                       $affected = max($affected, DBA::affectedRows());
+                       DBA::close($posts);
+                       $affected = max($affected, $affected_count);
                }
 
-               $update_fields = DBStructure::getFieldsForTable('post-thread-user', $fields);
+               $update_fields = Post\DeliveryData::extractFields($fields);
                if (!empty($update_fields)) {
-                       $rows = DBA::selectToArray('post-view', ['post-user-id'], $thread_condition);
-                       $thread_puids = array_column($rows, 'post-user-id');
-
-                       $post_thread_condition = DBA::collapseCondition(['id' => $thread_puids]);
-
-                       $post_thread_condition[0] = "EXISTS(SELECT `id` FROM `post-user` WHERE " .
-                               $post_thread_condition[0] . " AND `uri-id` = `post-thread-user`.`uri-id` AND `uid` = `post-thread-user`.`uid`)";
-                       Logger::info('Test2-start', ['condition' => $post_thread_condition]);
-                               if (!DBA::update('post-thread-user', $update_fields, $post_thread_condition)) {
-                               DBA::rollback();
-                               Logger::notice('Updating post-thread-user failed', ['fields' => $update_fields, 'condition' => $condition]);
-                               return false;
+                       $affected_count = 0;
+                       $posts          = DBA::select('post-user-view', ['uri-id'], $condition, ['group_by' => ['uri-id']]);
+                       while ($rows = DBA::toArray($posts, false, 100)) {
+                               $uriids = array_column($rows, 'uri-id');
+                               if (!DBA::update('post-delivery-data', $update_fields, ['uri-id' => $uriids])) {
+                                       DBA::rollback();
+                                       Logger::warning('Updating post-delivery-data failed', ['fields' => $update_fields, 'condition' => $condition]);
+                                       return false;
+                               }
+                               $affected_count += DBA::affectedRows();
                        }
-                       Logger::info('Test2-end');
-                       $affected = max($affected, DBA::affectedRows());
+                       DBA::close($posts);
+                       $affected = max($affected, $affected_count);
                }
 
-               $update_fields = DBStructure::getFieldsForTable('thread', $fields);
+               $update_fields = DI::dbaDefinition()->truncateFieldsForTable('post-thread', $fields);
                if (!empty($update_fields)) {
-                       $rows = DBA::selectToArray('post-view', ['id'], $thread_condition);
-                       $ids = array_column($rows, 'id');
-                       if (!DBA::update('thread', $update_fields, ['iid' => $ids])) {
-                               DBA::rollback();
-                               Logger::notice('Updating thread failed', ['fields' => $update_fields, 'condition' => $thread_condition]);
-                               return false;
+                       $affected_count = 0;
+                       $posts          = DBA::select('post-user-view', ['uri-id'], $thread_condition, ['group_by' => ['uri-id']]);
+                       while ($rows = DBA::toArray($posts, false, 100)) {
+                               $uriids = array_column($rows, 'uri-id');
+                               if (!DBA::update('post-thread', $update_fields, ['uri-id' => $uriids])) {
+                                       DBA::rollback();
+                                       Logger::warning('Updating post-thread failed', ['fields' => $update_fields, 'condition' => $condition]);
+                                       return false;
+                               }
+                               $affected_count += DBA::affectedRows();
                        }
-                       $affected = max($affected, DBA::affectedRows());
+                       DBA::close($posts);
+                       $affected = max($affected, $affected_count);
                }
 
-               $update_fields = [];
-               foreach (Item::USED_FIELDLIST as $field) {
-                       if (array_key_exists($field, $fields)) {
-                               $update_fields[$field] = $fields[$field];
-                       }
-               }
+               $update_fields = DI::dbaDefinition()->truncateFieldsForTable('post-thread-user', $fields);
                if (!empty($update_fields)) {
-                       $rows = DBA::selectToArray('post-view', ['id'], $condition, []);
-                       $ids = array_column($rows, 'id');
-                       if (!DBA::update('item', $update_fields, ['id' => $ids])) {
-                               DBA::rollback();
-                               Logger::notice('Updating item failed', ['fields' => $update_fields, 'condition' => $condition]);
-                               return false;
+                       $affected_count = 0;
+                       $posts          = DBA::select('post-user-view', ['post-user-id'], $thread_condition);
+                       while ($rows = DBA::toArray($posts, false, 100)) {
+                               $thread_puids = array_column($rows, 'post-user-id');
+                               if (!DBA::update('post-thread-user', $update_fields, ['post-user-id' => $thread_puids])) {
+                                       DBA::rollback();
+                                       Logger::warning('Updating post-thread-user failed', ['fields' => $update_fields, 'condition' => $condition]);
+                                       return false;
+                               }
+                               $affected_count += DBA::affectedRows();
                        }
-                       $affected = max($affected, DBA::affectedRows());
+                       DBA::close($posts);
+                       $affected = max($affected, $affected_count);
                }
 
                DBA::commit();
@@ -501,4 +741,20 @@ class Post
                Logger::info('Updated posts', ['rows' => $affected]);
                return $affected;
        }
+
+       /**
+        * Delete a row from the post table
+        *
+        * @param array        $conditions Field condition(s)
+        * @param array        $options
+        *                           - cascade: If true we delete records in other tables that depend on the one we're deleting through
+        *                           relations (default: true)
+        *
+        * @return boolean was the delete successful?
+        * @throws \Exception
+        */
+       public static function delete(array $conditions, array $options = []): bool
+       {
+               return DBA::delete('post', $conditions, $options);
+       }
 }