]> git.mxchange.org Git - friendica.git/blobdiff - src/Worker/CronJobs.php
4) Adding Factories to other entrypoints
[friendica.git] / src / Worker / CronJobs.php
index ca898f2b6285a3fb4dfd8776d1354059c0384a8d..f723cf69f17a32fde4b69214b0708ae422741f1f 100644 (file)
@@ -8,15 +8,18 @@ use Friendica\App;
 use Friendica\BaseObject;
 use Friendica\Core\Cache;
 use Friendica\Core\Config;
+use Friendica\Core\Logger;
+use Friendica\Core\Protocol;
 use Friendica\Database\DBA;
+use Friendica\Database\PostUpdate;
 use Friendica\Model\Contact;
 use Friendica\Model\GContact;
 use Friendica\Model\Photo;
 use Friendica\Model\User;
 use Friendica\Network\Probe;
 use Friendica\Protocol\PortableContact;
+use Friendica\Util\Proxy as ProxyUtils;
 
-require_once 'include/dba.php';
 require_once 'mod/nodeinfo.php';
 
 class CronJobs
@@ -30,13 +33,12 @@ class CronJobs
                        return;
                }
 
-               logger("Starting cronjob " . $command, LOGGER_DEBUG);
+               Logger::log("Starting cronjob " . $command, Logger::DEBUG);
 
                // Call possible post update functions
                // see src/Database/PostUpdate.php for more details
                if ($command == 'post_update') {
-// Post updates will be reenabled (hopefully in a few days) when most item works are done
-//                     PostUpdate::update();
+                       PostUpdate::update();
                        return;
                }
 
@@ -80,7 +82,7 @@ class CronJobs
                        return;
                }
 
-               logger("Xronjob " . $command . " is unknown.", LOGGER_DEBUG);
+               Logger::log("Xronjob " . $command . " is unknown.", Logger::DEBUG);
 
                return;
        }
@@ -106,7 +108,7 @@ class CronJobs
        private static function expireAndRemoveUsers()
        {
                // expire any expired regular accounts. Don't expire forums.
-               $condition = ["NOT `account_expired` AND `account_expires_on` > ? AND `account_expires_on` < UTC_TIMESTAMP() AND `page-flags` = 0", NULL_DATE];
+               $condition = ["NOT `account_expired` AND `account_expires_on` > ? AND `account_expires_on` < UTC_TIMESTAMP() AND `page-flags` = 0", DBA::NULL_DATETIME];
                DBA::update('user', ['account_expired' => true], $condition);
 
                // Remove any freshly expired account
@@ -116,8 +118,14 @@ class CronJobs
                }
 
                // delete user records for recently removed accounts
-               $users = DBA::select('user', ['uid'], ["`account_removed` AND `account_expires_on` < UTC_TIMESTAMP() - INTERVAL 3 DAY"]);
+               $users = DBA::select('user', ['uid'], ["`account_removed` AND `account_expires_on` < UTC_TIMESTAMP() "]);
                while ($user = DBA::fetch($users)) {
+                       // Delete the contacts of this user
+                       $self = DBA::selectFirst('contact', ['nurl'], ['self' => true, 'uid' => $user['uid']]);
+                       if (DBA::isResult($self)) {
+                               DBA::delete('contact', ['nurl' => $self['nurl'], 'self' => false]);
+                       }
+
                        DBA::delete('user', ['uid' => $user['uid']]);
                }
        }
@@ -126,6 +134,7 @@ class CronJobs
         * @brief Clear cache entries
         *
         * @param App $a
+        * @throws \Friendica\Network\HTTPException\InternalServerErrorException
         */
        private static function clearCache(App $a)
        {
@@ -149,21 +158,23 @@ class CronJobs
                clear_cache();
 
                // clear cache for photos
-               clear_cache($a->get_basepath(), $a->get_basepath() . "/photo");
+               clear_cache($a->getBasePath(), $a->getBasePath() . "/photo");
 
                // clear smarty cache
-               clear_cache($a->get_basepath() . "/view/smarty3/compiled", $a->get_basepath() . "/view/smarty3/compiled");
+               clear_cache($a->getBasePath() . "/view/smarty3/compiled", $a->getBasePath() . "/view/smarty3/compiled");
 
                // clear cache for image proxy
                if (!Config::get("system", "proxy_disabled")) {
-                       clear_cache($a->get_basepath(), $a->get_basepath() . "/proxy");
+                       clear_cache($a->getBasePath(), $a->getBasePath() . "/proxy");
 
                        $cachetime = Config::get('system', 'proxy_cache_time');
+
                        if (!$cachetime) {
-                               $cachetime = PROXY_DEFAULT_TIME;
+                               $cachetime = ProxyUtils::DEFAULT_TIME;
                        }
+
                        $condition = ['`uid` = 0 AND `resource-id` LIKE "pic:%" AND `created` < NOW() - INTERVAL ? SECOND', $cachetime];
-                       DBA::delete('photo', $condition);
+                       Photo::delete($condition);
                }
 
                // Delete the cached OEmbed entries that are older than three month
@@ -201,7 +212,7 @@ class CronJobs
                                // Calculate fragmentation
                                $fragmentation = $table["Data_free"] / ($table["Data_length"] + $table["Index_length"]);
 
-                               logger("Table " . $table["Name"] . " - Fragmentation level: " . round($fragmentation * 100, 2), LOGGER_DEBUG);
+                               Logger::log("Table " . $table["Name"] . " - Fragmentation level: " . round($fragmentation * 100, 2), Logger::DEBUG);
 
                                // Don't optimize tables that needn't to be optimized
                                if ($fragmentation < $fragmentation_level) {
@@ -209,7 +220,7 @@ class CronJobs
                                }
 
                                // So optimize it
-                               logger("Optimize Table " . $table["Name"], LOGGER_DEBUG);
+                               Logger::log("Optimize Table " . $table["Name"], Logger::DEBUG);
                                q("OPTIMIZE TABLE `%s`", DBA::escape($table["Name"]));
                        }
                }
@@ -221,6 +232,8 @@ class CronJobs
         * @brief Repair missing values in Diaspora contacts
         *
         * @param App $a
+        * @throws \Friendica\Network\HTTPException\InternalServerErrorException
+        * @throws \ImagickException
         */
        private static function repairDiaspora(App $a)
        {
@@ -228,12 +241,12 @@ class CronJobs
 
                $r = q("SELECT `id`, `url` FROM `contact`
                        WHERE `network` = '%s' AND (`batch` = '' OR `notify` = '' OR `poll` = '' OR pubkey = '')
-                               ORDER BY RAND() LIMIT 50", DBA::escape(NETWORK_DIASPORA));
+                               ORDER BY RAND() LIMIT 50", DBA::escape(Protocol::DIASPORA));
                if (!DBA::isResult($r)) {
                        return;
                }
 
-               foreach ($r AS $contact) {
+               foreach ($r as $contact) {
                        // Quit the loop after 3 minutes
                        if (time() > ($starttime + 180)) {
                                return;
@@ -244,11 +257,11 @@ class CronJobs
                        }
 
                        $data = Probe::uri($contact["url"]);
-                       if ($data["network"] != NETWORK_DIASPORA) {
+                       if ($data["network"] != Protocol::DIASPORA) {
                                continue;
                        }
 
-                       logger("Repair contact " . $contact["id"] . " " . $contact["url"], LOGGER_DEBUG);
+                       Logger::log("Repair contact " . $contact["id"] . " " . $contact["url"], Logger::DEBUG);
                        q("UPDATE `contact` SET `batch` = '%s', `notify` = '%s', `poll` = '%s', pubkey = '%s' WHERE `id` = %d",
                                DBA::escape($data["batch"]), DBA::escape($data["notify"]), DBA::escape($data["poll"]), DBA::escape($data["pubkey"]),
                                intval($contact["id"]));
@@ -266,15 +279,11 @@ class CronJobs
                $r = q("SELECT `uid` FROM `user` WHERE NOT EXISTS (SELECT `uid` FROM `contact` WHERE `contact`.`uid` = `user`.`uid` AND `contact`.`self`)");
                if (DBA::isResult($r)) {
                        foreach ($r AS $user) {
-                               logger('Create missing self contact for user ' . $user['uid']);
+                               Logger::log('Create missing self contact for user ' . $user['uid']);
                                Contact::createSelfFromUserId($user['uid']);
                        }
                }
 
-               // Set the parent if it wasn't set. (Shouldn't happen - but does sometimes)
-               // This call is very "cheap" so we can do it at any time without a problem
-               q("UPDATE `item` INNER JOIN `item` AS `parent` ON `parent`.`uri` = `item`.`parent-uri` AND `parent`.`uid` = `item`.`uid` SET `item`.`parent` = `parent`.`id` WHERE `item`.`parent` = 0");
-
                // There was an issue where the nick vanishes from the contact table
                q("UPDATE `contact` INNER JOIN `user` ON `contact`.`uid` = `user`.`uid` SET `nick` = `nickname` WHERE `self` AND `nick`=''");