Change Details

https://quarry.wmcloud.org/query/68437 Namespaces 0 (main), 2 (user), and 118 (draft) are expected to be in this table as these are namespaces PageTriage patrols. However there are a bunch of other namespaces in this table. Is this a bug? If so, let's track it down and fix it. {F35662310} * Fix the root cause of the bugs # [ ] Hooks.php -> onPageMoveComplete, the last conditional should check if page is being moved out of a namespace PageTriage patrols. If so, run `$pageTriage->deleteFromPageTriage()` # [ ] includes/Maintenance/RemoveOldRows.php should delete from all namespaces after 30 days but doesn't. The bug is on line 72, where the `$secondaryNamespaces` variable is declared. Despite a comment stating otherwise, it contains an incomplete list of namespaces. * [ ] Keep an eye on this / query this after the two patches are deployed and after 30 days elapse (giving the cron job time to delete the old entries). If it starts swelling again, track down the offending code and fix it. Rinse and repeat.

https://quarry.wmcloud.org/query/68437 Namespaces 0 (main), 2 (user), and 118 (draft) are expected to be in this table as these are namespaces PageTriage patrols. However there are a bunch of other namespaces in this table. Is this a bug? If so, let's track it down and fix it. {F35662310} * Fix the root cause of the bugs # [ ] Hooks.php -> onPageMoveComplete, the last conditional should check if page is being moved out of a namespace PageTriage patrols. If so, run `$pageTriage->deleteFromPageTriage()` # [ ] includes/Maintenance/RemoveOldRows.php should delete from all namespaces after 30 days but doesn't. The bug is on line 72, where the `$secondaryNamespaces` variable is declared. Despite a comment stating otherwise, it contains an incomplete list of namespaces. * [ ] Re-run [[ https://quarry.wmcloud.org/query/68541 | this Quarry query ]] after the two patches are deployed and after 30 days elapse (giving the cron job time to delete the old entries). If the counts start swelling again, track down the offending code and fix it. Rinse and repeat.

https://quarry.wmcloud.org/query/68437 Namespaces 0 (main), 2 (user), and 118 (draft) are expected to be in this table as these are namespaces PageTriage patrols. However there are a bunch of other namespaces in this table. Is this a bug? If so, let's track it down and fix it. {F35662310} * Fix the root cause of the bugs # [ ] Hooks.php -> onPageMoveComplete, the last conditional should check if page is being moved out of a namespace PageTriage patrols. If so, run `$pageTriage->deleteFromPageTriage()` # [ ] includes/Maintenance/RemoveOldRows.php should delete from all namespaces after 30 days but doesn't. The bug is on line 72, where the `$secondaryNamespaces` variable is declared. Despite a comment stating otherwise, it contains an incomplete list of namespaces. * [ ] Keep an eye onRe-run [[ https://quarry.wmcloud.org/query/68541 | this /Quarry query this]] after the two patches are deployed and after 30 days elapse (giving the cron job time to delete the old entries). If itIf the counts starts swelling again, track down the offending code and fix it. Rinse and repeat.