X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;f=src%2FWorker%2FExpire.php;h=b6096b5849405d11b2f68fec6a81c245157a44ad;hb=cbfc3d0c28e0b55c3ccf1ddfd321798a8c2532e8;hp=e963d6d3519d43ae6e278ae26de6b0793ac3fe7c;hpb=fa11959919dcef30780f6280f1e5257c873fa8b6;p=friendica.git diff --git a/src/Worker/Expire.php b/src/Worker/Expire.php index e963d6d351..b6096b5849 100644 --- a/src/Worker/Expire.php +++ b/src/Worker/Expire.php @@ -6,43 +6,59 @@ namespace Friendica\Worker; +use Friendica\BaseObject; +use Friendica\Core\Addon; use Friendica\Core\Config; use Friendica\Core\Worker; -use Friendica\Database\DBM; -use dba; +use Friendica\Database\DBA; +use Friendica\Model\Item; require_once 'include/dba.php'; -class Expire { - public static function execute($param = '', $hook_name = '') { - global $a; +class Expire +{ + public static function execute($param = '', $hook_name = '') + { + $a = BaseObject::getApp(); - require_once 'include/datetime.php'; require_once 'include/items.php'; - load_hooks(); + Addon::loadHooks(); if ($param == 'delete') { logger('Delete expired items', LOGGER_DEBUG); // physically remove anything that has been deleted for more than two months - $r = dba::p("SELECT `id` FROM `item` WHERE `deleted` AND `changed` < UTC_TIMESTAMP() - INTERVAL 60 DAY"); - while ($row = dba::fetch($r)) { - dba::delete('item', array('id' => $row['id'])); + $condition = ["`deleted` AND `changed` < UTC_TIMESTAMP() - INTERVAL 60 DAY"]; + $rows = DBA::select('item', ['id'], $condition); + while ($row = DBA::fetch($rows)) { + DBA::delete('item', ['id' => $row['id']]); } - dba::close($r); + DBA::close($rows); - logger('Delete expired items - done', LOGGER_DEBUG); + // Normally we shouldn't have orphaned data at all. + // If we do have some, then we have to check why. + logger('Deleting orphaned item activities - start', LOGGER_DEBUG); + $condition = ["NOT EXISTS (SELECT `iaid` FROM `item` WHERE `item`.`iaid` = `item-activity`.`id`)"]; + DBA::delete('item-activity', $condition); + logger('Orphaned item activities deleted: ' . DBA::affectedRows(), LOGGER_DEBUG); + + logger('Deleting orphaned item content - start', LOGGER_DEBUG); + $condition = ["NOT EXISTS (SELECT `icid` FROM `item` WHERE `item`.`icid` = `item-content`.`id`)"]; + DBA::delete('item-content', $condition); + logger('Orphaned item content deleted: ' . DBA::affectedRows(), LOGGER_DEBUG); // make this optional as it could have a performance impact on large sites if (intval(Config::get('system', 'optimize_items'))) { - dba::e("OPTIMIZE TABLE `item`"); + DBA::e("OPTIMIZE TABLE `item`"); } + + logger('Delete expired items - done', LOGGER_DEBUG); return; } elseif (intval($param) > 0) { - $user = dba::select('user', array('uid', 'username', 'expire'), array('uid' => $param), array('limit' => 1)); - if (DBM::is_result($user)) { + $user = DBA::selectFirst('user', ['uid', 'username', 'expire'], ['uid' => $param]); + if (DBA::isResult($user)) { logger('Expire items for user '.$user['uid'].' ('.$user['username'].') - interval: '.$user['expire'], LOGGER_DEBUG); - item_expire($user['uid'], $user['expire']); + Item::expire($user['uid'], $user['expire']); logger('Expire items for user '.$user['uid'].' ('.$user['username'].') - done ', LOGGER_DEBUG); } return; @@ -50,7 +66,7 @@ class Expire { foreach ($a->hooks["expire"] as $hook) { if ($hook[1] == $hook_name) { logger("Calling expire hook '" . $hook[1] . "'", LOGGER_DEBUG); - call_single_hook($a, $hook_name, $hook, $data); + Addon::callSingleHook($a, $hook_name, $hook, $data); } } return; @@ -58,23 +74,23 @@ class Expire { logger('expire: start'); - Worker::add(array('priority' => $a->queue['priority'], 'created' => $a->queue['created'], 'dont_fork' => true), + Worker::add(['priority' => $a->queue['priority'], 'created' => $a->queue['created'], 'dont_fork' => true], 'Expire', 'delete'); - $r = dba::p("SELECT `uid`, `username` FROM `user` WHERE `expire` != 0"); - while ($row = dba::fetch($r)) { + $r = DBA::p("SELECT `uid`, `username` FROM `user` WHERE `expire` != 0"); + while ($row = DBA::fetch($r)) { logger('Calling expiry for user '.$row['uid'].' ('.$row['username'].')', LOGGER_DEBUG); - Worker::add(array('priority' => $a->queue['priority'], 'created' => $a->queue['created'], 'dont_fork' => true), + Worker::add(['priority' => $a->queue['priority'], 'created' => $a->queue['created'], 'dont_fork' => true], 'Expire', (int)$row['uid']); } - dba::close($r); + DBA::close($r); logger('expire: calling hooks'); if (is_array($a->hooks) && array_key_exists('expire', $a->hooks)) { foreach ($a->hooks['expire'] as $hook) { logger("Calling expire hook for '" . $hook[1] . "'", LOGGER_DEBUG); - Worker::add(array('priority' => $a->queue['priority'], 'created' => $a->queue['created'], 'dont_fork' => true), + Worker::add(['priority' => $a->queue['priority'], 'created' => $a->queue['created'], 'dont_fork' => true], 'Expire', 'hook', $hook[1]); } }