From 17a7b25a30fb09b8b1cc482b11d74e8c2a4e9173 Mon Sep 17 00:00:00 2001 From: daniel Date: Wed, 18 Apr 2012 15:02:35 +0200 Subject: [PATCH] some todo comments --- includes/ContentHandler.php | 3 +++ includes/WikiPage.php | 2 ++ 2 files changed, 5 insertions(+) diff --git a/includes/ContentHandler.php b/includes/ContentHandler.php index 13ad3505d3..c6daeff5ce 100644 --- a/includes/ContentHandler.php +++ b/includes/ContentHandler.php @@ -605,6 +605,9 @@ abstract class ContentHandler { return $reason; } + #TODO: getSecondaryUpdatesForDeletion( Content ) returns an array of SecondaryDataUpdate objects + #... or do that in the Content class? + /** * Get the Content object that needs to be saved in order to undo all revisions * between $undo and $undoafter. Revisions must belong to the same page, diff --git a/includes/WikiPage.php b/includes/WikiPage.php index 3088131a1a..8f5d9a5250 100644 --- a/includes/WikiPage.php +++ b/includes/WikiPage.php @@ -2271,6 +2271,8 @@ class WikiPage extends Page { $this->updateCategoryCounts( array(), $cats ); + #TODO: move this to an Update object! + # If using cascading deletes, we can skip some explicit deletes if ( !$dbw->cascadingDeletes() ) { $dbw->delete( 'revision', array( 'rev_page' => $id ), __METHOD__ ); -- 2.20.1