The overloaded DB_DataObject function staticGet is now called getKV

[quix0rs-gnu-social.git] / plugins / OStatus / scripts / fixup-shadow.php

diff --git a/plugins/OStatus/scripts/fixup-shadow.php b/plugins/OStatus/scripts/fixup-shadow.php
index 3e2c18e02fb314780a4c382136de4fd90ee7fa0e..45e2d8c0b0cff3e80c97ec4e52b0967460e63773 100644 (file)
--- a/plugins/OStatus/scripts/fixup-shadow.php
+++ b/plugins/OStatus/scripts/fixup-shadow.php
@@ -53,7 +53,7 @@ while ($user->fetch()) {
         echo " - skipping\n";
     } else {
         echo " - removing bogus ostatus_profile entry...";
-        $evil = Ostatus_profile::staticGet('uri', $uri);
+        $evil = Ostatus_profile::getKV('uri', $uri);
         $evil->delete();
         echo "  ok\n";
     }
@@ -77,13 +77,57 @@ while ($group->fetch()) {
         echo " - skipping\n";
     } else {
         echo " - removing bogus ostatus_profile entry...";
-        $evil = Ostatus_profile::staticGet('uri', $uri);
+        $evil = Ostatus_profile::getKV('uri', $uri);
         $evil->delete();
         echo "  ok\n";
     }
 }
 echo "\n";
 
+// And there may be user_group entries remaining where we've already killed
+// the ostatus_profile. These were "harmless" until our lookup started actually
+// using the uri field, at which point we can clearly see it breaks stuff.
+echo "Checking for leftover bogus user_group.uri entries obscuring local_group entries...\n";
+
+$group = new User_group();
+$group->joinAdd(array('id', 'local_group:group_id'), 'LEFT');
+$group->whereAdd('group_id IS NULL');
+
+
+$marker = mt_rand(31337, 31337000);
+$groupTemplate = common_local_url('groupbyid', array('id' => $marker));
+$encGroup = $group->escape($groupTemplate, true);
+$encGroup = str_replace($marker, '%', $encGroup);
+echo "  LIKE '$encGroup'\n";
+$group->whereAdd("uri LIKE '$encGroup'");
+
+$group->find();
+$count = $group->N;
+echo "Found $count...\n";
+
+while ($group->fetch()) {
+    $uri = $group->uri;
+    if (preg_match('!/group/(\d+)/id!', $uri, $matches)) {
+        $id = intval($matches[1]);
+        $local = Local_group::getKV('group_id', $id);
+        if ($local) {
+            $nick = $local->nickname;
+        } else {
+            $nick = '<deleted>';
+        }
+        echo "local group $id ($local->nickname) hidden by $uri (bogus group id $group->id)";
+        if ($dry) {
+            echo " - skipping\n";
+        } else {
+            echo " - removing bogus user_group entry...";
+            $evil = User_group::getKV('id', $group->id);
+            $evil->delete();
+            echo "  ok\n";
+        }
+    }
+}
+echo "\n";
+
 
 // Fallback?
 echo "Checking for bogus profiles blocking local users/groups by URI pattern match...\n";
@@ -111,7 +155,7 @@ while ($oprofile->fetch()) {
     $uri = $oprofile->uri;
     if (preg_match('!/group/(\d+)/id!', $oprofile->uri, $matches)) {
         $id = intval($matches[1]);
-        $group = Local_group::staticGet('group_id', $id);
+        $group = Local_group::getKV('group_id', $id);
         if ($group) {
             $nick = $group->nickname;
         } else {
@@ -120,7 +164,7 @@ while ($oprofile->fetch()) {
         echo "group $id ($nick) hidden by $uri";
     } else if (preg_match('!/user/(\d+)!', $uri, $matches)) {
         $id = intval($matches[1]);
-        $user = User::staticGet('id', $id);
+        $user = User::getKV('id', $id);
         if ($user) {
             $nick = $user->nickname;
         } else {