1
1
Fork 0

Merge pull request #5110 from annando/nodeinfo

Nodeinfo: The number of comments is now calculated more reliable
This commit is contained in:
Hypolite Petovan 2018-05-22 17:20:06 -04:00 committed by GitHub
commit 44caf05f06
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
3 changed files with 35 additions and 62 deletions

View file

@ -10,6 +10,7 @@ use Friendica\Core\Addon;
use Friendica\Core\System; use Friendica\Core\System;
use Friendica\Core\Config; use Friendica\Core\Config;
use Friendica\Util\Network; use Friendica\Util\Network;
require_once 'include/dba.php';
function nodeinfo_wellknown(App $a) { function nodeinfo_wellknown(App $a) {
$nodeinfo = ['links' => [['rel' => 'http://nodeinfo.diaspora.software/ns/schema/1.0', $nodeinfo = ['links' => [['rel' => 'http://nodeinfo.diaspora.software/ns/schema/1.0',
@ -153,7 +154,7 @@ function nodeinfo_cron() {
$a = get_app(); $a = get_app();
// If the addon 'statistics_json' is enabled then disable it and actrivate nodeinfo. // If the addon 'statistics_json' is enabled then disable it and activate nodeinfo.
if (Addon::isEnabled('statistics_json')) { if (Addon::isEnabled('statistics_json')) {
Config::set('system', 'nodeinfo', true); Config::set('system', 'nodeinfo', true);
@ -176,16 +177,7 @@ function nodeinfo_cron() {
if (!Config::get('system', 'nodeinfo')) { if (!Config::get('system', 'nodeinfo')) {
return; return;
} }
$last = Config::get('nodeinfo', 'last_calucation');
if ($last) {
// Calculate every 24 hours
$next = $last + (24 * 60 * 60);
if ($next > time()) {
logger('calculation intervall not reached');
return;
}
}
logger('cron_start'); logger('cron_start');
$users = q("SELECT `user`.`uid`, `user`.`login_date`, `contact`.`last-item` $users = q("SELECT `user`.`uid`, `user`.`login_date`, `contact`.`last-item`
@ -214,35 +206,19 @@ function nodeinfo_cron() {
} }
} }
Config::set('nodeinfo', 'total_users', $total_users); Config::set('nodeinfo', 'total_users', $total_users);
logger('total_users: '.$total_users, LOGGER_DEBUG);
Config::set('nodeinfo', 'active_users_halfyear', $active_users_halfyear); Config::set('nodeinfo', 'active_users_halfyear', $active_users_halfyear);
Config::set('nodeinfo', 'active_users_monthly', $active_users_monthly); Config::set('nodeinfo', 'active_users_monthly', $active_users_monthly);
logger('total_users: ' . $total_users . '/' . $active_users_halfyear. '/' . $active_users_monthly, LOGGER_DEBUG);
} }
$posts = q("SELECT COUNT(*) AS `local_posts` FROM `thread` WHERE `thread`.`wall` AND `thread`.`uid` != 0"); $local_posts = dba::count('thread', ["`wall` AND NOT `deleted` AND `uid` != 0"]);
if (!is_array($posts)) {
$local_posts = -1;
} else {
$local_posts = $posts[0]['local_posts'];
}
Config::set('nodeinfo', 'local_posts', $local_posts); Config::set('nodeinfo', 'local_posts', $local_posts);
logger('local_posts: ' . $local_posts, LOGGER_DEBUG);
logger('local_posts: '.$local_posts, LOGGER_DEBUG); $local_comments = dba::count('item', ["`origin` AND `id` != `parent` AND NOT `deleted` AND `uid` != 0"]);
$posts = q("SELECT COUNT(*) AS `local_comments` FROM `contact`
INNER JOIN `item` ON `item`.`contact-id` = `contact`.`id` AND `item`.`uid` = `contact`.`uid` AND
`item`.`id` != `item`.`parent` AND `item`.`network` IN ('%s', '%s', '%s')
WHERE `contact`.`self`",
dbesc(NETWORK_OSTATUS), dbesc(NETWORK_DIASPORA), dbesc(NETWORK_DFRN));
if (!is_array($posts)) {
$local_comments = -1;
} else {
$local_comments = $posts[0]['local_comments'];
}
Config::set('nodeinfo', 'local_comments', $local_comments); Config::set('nodeinfo', 'local_comments', $local_comments);
logger('local_comments: ' . $local_comments, LOGGER_DEBUG);
// Now trying to register // Now trying to register
$url = 'http://the-federation.info/register/'.$a->get_hostname(); $url = 'http://the-federation.info/register/'.$a->get_hostname();
@ -251,5 +227,4 @@ function nodeinfo_cron() {
logger('registering answer: '.$ret, LOGGER_DEBUG); logger('registering answer: '.$ret, LOGGER_DEBUG);
logger('cron_end'); logger('cron_end');
Config::set('nodeinfo', 'last_calucation', time());
} }

View file

@ -58,9 +58,6 @@ Class Cron {
// Call possible post update functions // Call possible post update functions
Worker::add(PRIORITY_LOW, "CronJobs", "post_update"); Worker::add(PRIORITY_LOW, "CronJobs", "post_update");
// update nodeinfo data
Worker::add(PRIORITY_LOW, "CronJobs", "nodeinfo");
// Clear cache entries // Clear cache entries
Worker::add(PRIORITY_LOW, "CronJobs", "clear_cache"); Worker::add(PRIORITY_LOW, "CronJobs", "clear_cache");
@ -79,20 +76,23 @@ Class Cron {
Worker::add(PRIORITY_LOW, "CronJobs", "update_contact_birthdays"); Worker::add(PRIORITY_LOW, "CronJobs", "update_contact_birthdays");
Worker::add(PRIORITY_LOW, "CronJobs", "update_photo_albums");
// update nodeinfo data
Worker::add(PRIORITY_LOW, "CronJobs", "nodeinfo");
Worker::add(PRIORITY_LOW, "DiscoverPoCo", "update_server"); Worker::add(PRIORITY_LOW, "DiscoverPoCo", "update_server");
Worker::add(PRIORITY_LOW, "DiscoverPoCo", "suggestions"); Worker::add(PRIORITY_LOW, "DiscoverPoCo", "suggestions");
Config::set('system', 'last_expire_day', $d2);
Worker::add(PRIORITY_LOW, 'Expire'); Worker::add(PRIORITY_LOW, 'Expire');
Worker::add(PRIORITY_MEDIUM, 'DBClean'); Worker::add(PRIORITY_MEDIUM, 'DBClean');
Worker::add(PRIORITY_LOW, "CronJobs", "update_photo_albums");
// check upstream version? // check upstream version?
Worker::add(PRIORITY_LOW, 'CheckVersion'); Worker::add(PRIORITY_LOW, 'CheckVersion');
Config::set('system', 'last_expire_day', $d2);
} }
// Hourly cron calls // Hourly cron calls

View file

@ -26,8 +26,6 @@ class CronJobs
{ {
global $a; global $a;
require_once 'mod/nodeinfo.php';
// No parameter set? So return // No parameter set? So return
if ($command == '') { if ($command == '') {
return; return;