provide a site-wide expiration policy without loopholes - still need work to reduce the impact of expiration queries.

This commit is contained in:
friendica 2015-03-24 15:53:55 -07:00
parent abf642be7f
commit 3d60008842
3 changed files with 30 additions and 12 deletions

View File

@ -7,7 +7,11 @@ function expire_run($argv, $argc){
cli_startup();
$r = q("select id from item where (item_restrict & %d)>0 and not (item_restrict & %d)>0 and changed < %s - INTERVAL %s",
// perform final cleanup on previously delete items
$r = q("select id from item where (item_restrict & %d) > 0 and (item_restrict & %d) = 0
and changed < %s - INTERVAL %s",
intval(ITEM_DELETED),
intval(ITEM_PENDING_REMOVE),
db_utcnow(), db_quoteinterval('10 DAY')
@ -19,6 +23,7 @@ function expire_run($argv, $argc){
}
// physically remove anything that has been deleted for more than two months
// FIXME - this is a wretchedly inefficient query
$r = q("delete from item where ( item_restrict & %d ) > 0 and changed < %s - INTERVAL %s",
intval(ITEM_PENDING_REMOVE),
@ -32,12 +37,28 @@ function expire_run($argv, $argc){
logger('expire: start', LOGGER_DEBUG);
$site_expire = get_config('system', 'default_expire_days');
if(intval($site_expire)) {
$r = q("SELECT channel_id, channel_address, channel_pageflags, channel_expire_days from channel where true");
}
else {
$r = q("SELECT channel_id, channel_address, channel_pageflags, channel_expire_days from channel where channel_expire_days != 0");
}
$r = q("SELECT channel_id, channel_address, channel_expire_days from channel where channel_expire_days != 0");
if($r && count($r)) {
if($r) {
foreach($r as $rr) {
// expire the sys channel separately
if($rr['channel_pageflags'] & PAGE_SYSTEM)
continue;
// if the site expiration is non-zero and less than person expiration, use that
logger('Expire: ' . $rr['channel_address'] . ' interval: ' . $rr['channel_expire_days'], LOGGER_DEBUG);
item_expire($rr['channel_id'],$rr['channel_expire_days']);
item_expire($rr['channel_id'],
((intval($site_expire) && intval($site_expire) < intval($rr['channel_expire_days']))
? $site_expire
: $rr['channel_expire_days'])
);
}
}
@ -48,14 +69,13 @@ function expire_run($argv, $argc){
// this should probably just fetch the channel_expire_days from the sys channel,
// but there's no convenient way to set it.
$expire_days = get_config('externals','expire_days');
$expire_days = get_config('system','sys_expire_days');
if($expire_days === false)
$expire_days = 30;
if($expire_days)
item_expire($x['channel_id'],$expire_days);
item_expire($x['channel_id'],(($site_expire && $site_expire < $expire_days) ? $site_expire : $expire_days));
}
return;
}

View File

@ -249,9 +249,7 @@ function create_identity($arr) {
}
}
$expire = get_config('system', 'default_expire_days');
$expire = (($expire===false)? '0': $expire);
$expire = 0;
$r = q("insert into channel ( channel_account_id, channel_primary,
channel_name, channel_address, channel_guid, channel_guid_sig,