]> git.mxchange.org Git - friendica.git/blobdiff - include/dbclean.php
dba functions in poller / avoiding SQL errors in conversation / dbclean is better
[friendica.git] / include / dbclean.php
index bff4ff2a24946a9246513306add33c8fdedafe72..64185b39d9ccdf6fa71740a2ae82dd858f98790f 100644 (file)
@@ -4,7 +4,7 @@
  * @brief The script is called from time to time to clean the database entries and remove orphaned data.
  */
 
-use \Friendica\Core\Config;
+use Friendica\Core\Config;
 
 function dbclean_run(&$argv, &$argc) {
        if (!Config::get('system', 'dbclean', false)) {
@@ -18,13 +18,11 @@ function dbclean_run(&$argv, &$argc) {
        }
 
        if ($stage == 0) {
-               proc_run(PRIORITY_LOW, 'include/dbclean.php', 1);
-               proc_run(PRIORITY_LOW, 'include/dbclean.php', 2);
-               proc_run(PRIORITY_LOW, 'include/dbclean.php', 3);
-               proc_run(PRIORITY_LOW, 'include/dbclean.php', 4);
-               proc_run(PRIORITY_LOW, 'include/dbclean.php', 5);
-               proc_run(PRIORITY_LOW, 'include/dbclean.php', 6);
-               proc_run(PRIORITY_LOW, 'include/dbclean.php', 7);
+               for ($i = 1; $i <= 7; $i++) {
+                       if (!Config::get('system', 'finished-dbclean-'.$i)) {
+                               proc_run(PRIORITY_LOW, 'include/dbclean.php', $i);
+                       }
+               }
        } else {
                remove_orphans($stage);
        }
@@ -41,99 +39,116 @@ function remove_orphans($stage = 0) {
        // We split the deletion in many small tasks
        $limit = 1000;
 
-       if (($stage == 1) OR ($stage == 0)) {
+       if ($stage == 1) {
                logger("Deleting old global item entries from item table without user copy");
-               if ($db->q("SELECT `id` FROM `item` WHERE `uid` = 0
+               $r = dba::p("SELECT `id` FROM `item` WHERE `uid` = 0
                                AND NOT EXISTS (SELECT `guid` FROM `item` AS `i` WHERE `item`.`guid` = `i`.`guid` AND `i`.`uid` != 0)
-                               AND `received` < UTC_TIMESTAMP() - INTERVAL 90 DAY LIMIT ".intval($limit), true)) {
-                       $count = $db->num_rows();
+                               AND `received` < UTC_TIMESTAMP() - INTERVAL 90 DAY LIMIT ".intval($limit));
+               $count = dba::num_rows($r);
+               if ($count > 0) {
                        logger("found global item orphans: ".$count);
-                       while ($orphan = $db->qfetch()) {
-                               q("DELETE FROM `item` WHERE `id` = %d", intval($orphan["id"]));
+                       while ($orphan = dba::fetch($r)) {
+                               dba::delete('item', array('id' => $orphan["id"]));
                        }
-               }
-               $db->qclose();
-               logger("Done deleting old global item entries from item table without user copy");
-       }
+               } else {
+                       logger("No global item orphans found");
 
-       if (($stage == 2) OR ($stage == 0)) {
+                       // We will eventually set this value when we found a good way to delete these items in another way.
+                       // Config::set('system', 'finished-dbclean-1', true);
+               }
+               dba::close($r);
+               logger("Done deleting ".$count." old global item entries from item table without user copy");
+       } elseif ($stage == 2) {
                logger("Deleting items without parents");
-               if ($db->q("SELECT `id` FROM `item` WHERE NOT EXISTS (SELECT `id` FROM `item` AS `i` WHERE `item`.`parent` = `i`.`id`) LIMIT ".intval($limit), true)) {
-                       $count = $db->num_rows();
+               $r = dba::p("SELECT `id` FROM `item` WHERE NOT EXISTS (SELECT `id` FROM `item` AS `i` WHERE `item`.`parent` = `i`.`id`) LIMIT ".intval($limit));
+               $count = dba::num_rows($r);
+               if ($count > 0) {
                        logger("found item orphans without parents: ".$count);
-                       while ($orphan = $db->qfetch()) {
-                               q("DELETE FROM `item` WHERE `id` = %d", intval($orphan["id"]));
+                       while ($orphan = dba::fetch($r)) {
+                               dba::delete('item', array('id' => $orphan["id"]));
                        }
+               } else {
+                       logger("No item orphans without parents found");
+                       Config::set('system', 'finished-dbclean-2', true);
                }
-               $db->qclose();
-               logger("Done deleting items without parents");
-       }
-
-       if (($stage == 3) OR ($stage == 0)) {
+               dba::close($r);
+               logger("Done deleting ".$count." items without parents");
+       } elseif ($stage == 3) {
                logger("Deleting orphaned data from thread table");
-               if ($db->q("SELECT `iid` FROM `thread` WHERE NOT EXISTS (SELECT `id` FROM `item` WHERE `item`.`parent` = `thread`.`iid`) LIMIT ".intval($limit), true)) {
-                       $count = $db->num_rows();
+               $r = dba::p("SELECT `iid` FROM `thread` WHERE NOT EXISTS (SELECT `id` FROM `item` WHERE `item`.`parent` = `thread`.`iid`) LIMIT ".intval($limit));
+               $count = dba::num_rows($r);
+               if ($count > 0) {
                        logger("found thread orphans: ".$count);
-                       while ($orphan = $db->qfetch()) {
-                               q("DELETE FROM `thread` WHERE `iid` = %d", intval($orphan["iid"]));
+                       while ($orphan = dba::fetch($r)) {
+                               dba::delete('thread', array('iid' => $orphan["iid"]));
                        }
+               } else {
+                       logger("No thread orphans found");
+                       Config::set('system', 'finished-dbclean-3', true);
                }
-               $db->qclose();
-               logger("Done deleting orphaned data from thread table");
-       }
 
-       if (($stage == 4) OR ($stage == 0)) {
+               dba::close($r);
+               logger("Done deleting ".$count." orphaned data from thread table");
+       } elseif ($stage == 4) {
                logger("Deleting orphaned data from notify table");
-               if ($db->q("SELECT `iid` FROM `notify` WHERE NOT EXISTS (SELECT `id` FROM `item` WHERE `item`.`id` = `notify`.`iid`) LIMIT ".intval($limit), true)) {
-                       $count = $db->num_rows();
+               $r = dba::p("SELECT `iid` FROM `notify` WHERE NOT EXISTS (SELECT `id` FROM `item` WHERE `item`.`id` = `notify`.`iid`) LIMIT ".intval($limit));
+               $count = dba::num_rows($r);
+               if ($count > 0) {
                        logger("found notify orphans: ".$count);
-                       while ($orphan = $db->qfetch()) {
-                               q("DELETE FROM `notify` WHERE `iid` = %d", intval($orphan["iid"]));
+                       while ($orphan = dba::fetch($r)) {
+                               dba::delete('notify', array('iid' => $orphan["iid"]));
                        }
+               } else {
+                       logger("No notify orphans found");
+                       Config::set('system', 'finished-dbclean-4', true);
                }
-               $db->qclose();
-               logger("Done deleting orphaned data from notify table");
-       }
-
-       if (($stage == 5) OR ($stage == 0)) {
+               dba::close($r);
+               logger("Done deleting ".$count." orphaned data from notify table");
+       } elseif ($stage == 5) {
                logger("Deleting orphaned data from notify-threads table");
-               if ($db->q("SELECT `id` FROM `notify-threads` WHERE NOT EXISTS (SELECT `id` FROM `item` WHERE `item`.`parent` = `notify-threads`.`master-parent-item`) LIMIT ".intval($limit), true)) {
-                       $count = $db->num_rows();
+               $r = dba::p("SELECT `id` FROM `notify-threads` WHERE NOT EXISTS (SELECT `id` FROM `item` WHERE `item`.`parent` = `notify-threads`.`master-parent-item`) LIMIT ".intval($limit));
+               $count = dba::num_rows($r);
+               if ($count > 0) {
                        logger("found notify-threads orphans: ".$count);
-                       while ($orphan = $db->qfetch()) {
-                               q("DELETE FROM `notify-threads` WHERE `id` = %d", intval($orphan["id"]));
+                       while ($orphan = dba::fetch($r)) {
+                               dba::delete('notify-threads', array('id' => $orphan["id"]));
                        }
+               } else {
+                       logger("No notify-threads orphans found");
+                       Config::set('system', 'finished-dbclean-5', true);
                }
-               $db->qclose();
-               logger("Done deleting orphaned data from notify-threads table");
-       }
-
-
-       if (($stage == 6) OR ($stage == 0)) {
+               dba::close($r);
+               logger("Done deleting ".$count." orphaned data from notify-threads table");
+       } elseif ($stage == 6) {
                logger("Deleting orphaned data from sign table");
-               if ($db->q("SELECT `iid` FROM `sign` WHERE NOT EXISTS (SELECT `id` FROM `item` WHERE `item`.`id` = `sign`.`iid`) LIMIT ".intval($limit), true)) {
-                       $count = $db->num_rows();
+               $r = dba::p("SELECT `iid` FROM `sign` WHERE NOT EXISTS (SELECT `id` FROM `item` WHERE `item`.`id` = `sign`.`iid`) LIMIT ".intval($limit));
+               $count = dba::num_rows($r);
+               if ($count > 0) {
                        logger("found sign orphans: ".$count);
-                       while ($orphan = $db->qfetch()) {
-                               q("DELETE FROM `sign` WHERE `iid` = %d", intval($orphan["iid"]));
+                       while ($orphan = dba::fetch($r)) {
+                               dba::delete('sign', array('iid' => $orphan["iid"]));
                        }
+               } else {
+                       logger("No sign orphans found");
+                       Config::set('system', 'finished-dbclean-6', true);
                }
-               $db->qclose();
-               logger("Done deleting orphaned data from sign table");
-       }
-
-
-       if (($stage == 7) OR ($stage == 0)) {
+               dba::close($r);
+               logger("Done deleting ".$count." orphaned data from sign table");
+       } elseif ($stage == 7) {
                logger("Deleting orphaned data from term table");
-               if ($db->q("SELECT `oid` FROM `term` WHERE NOT EXISTS (SELECT `id` FROM `item` WHERE `item`.`id` = `term`.`oid`) LIMIT ".intval($limit), true)) {
-                       $count = $db->num_rows();
+               $r = dba::p("SELECT `oid` FROM `term` WHERE NOT EXISTS (SELECT `id` FROM `item` WHERE `item`.`id` = `term`.`oid`) LIMIT ".intval($limit));
+               $count = dba::num_rows($r);
+               if ($count > 0) {
                        logger("found term orphans: ".$count);
-                       while ($orphan = $db->qfetch()) {
-                               q("DELETE FROM `term` WHERE `oid` = %d", intval($orphan["oid"]));
+                       while ($orphan = dba::fetch($r)) {
+                               dba::delete('term', array('oid' => $orphan["oid"]));
                        }
+               } else {
+                       logger("No term orphans found");
+                       Config::set('system', 'finished-dbclean-7', true);
                }
-               $db->qclose();
-               logger("Done deleting orphaned data from term table");
+               dba::close($r);
+               logger("Done deleting ".$count." orphaned data from term table");
        }
 
        // Call it again if not all entries were purged
@@ -142,4 +157,3 @@ function remove_orphans($stage = 0) {
        }
 
 }
-?>