baby steps towards editing structured data

[lhc/web/wiklou.git] / includes / ContentHandler.php

diff --git a/includes/ContentHandler.php b/includes/ContentHandler.php
index 67bc812..496d7c9 100644 (file)
--- a/includes/ContentHandler.php
+++ b/includes/ContentHandler.php
@@ -304,84 +304,91 @@ abstract class ContentHandler {
     public function getAutoDeleteReason( Title $title, &$hasHistory ) {
         global $wgContLang;
 
-        $dbw = wfGetDB( DB_MASTER );
+        try {
+            $dbw = wfGetDB( DB_MASTER );
 
-        // Get the last revision
-        $rev = Revision::newFromTitle( $title );
+            // Get the last revision
+            $rev = Revision::newFromTitle( $title );
 
-        if ( is_null( $rev ) ) {
-            return false;
-        }
+            if ( is_null( $rev ) ) {
+                return false;
+            }
 
-        // Get the article's contents
-        $content = $rev->getContent();
-        $blank = false;
+            // Get the article's contents
+            $content = $rev->getContent();
+            $blank = false;
 
-        // If the page is blank, use the text from the previous revision,
-        // which can only be blank if there's a move/import/protect dummy revision involved
-        if ( $content->getSize() == 0 ) {
-            $prev = $rev->getPrevious();
+            // If the page is blank, use the text from the previous revision,
+            // which can only be blank if there's a move/import/protect dummy revision involved
+            if ( $content->getSize() == 0 ) {
+                $prev = $rev->getPrevious();
 
-            if ( $prev )       {
-                $content = $rev->getContent();
-                $blank = true;
+                if ( $prev )   {
+                    $content = $rev->getContent();
+                    $blank = true;
+                }
             }
-        }
 
-        // Find out if there was only one contributor
-        // Only scan the last 20 revisions
-        $res = $dbw->select( 'revision', 'rev_user_text',
-            array( 'rev_page' => $title->getArticleID(), $dbw->bitAnd( 'rev_deleted', Revision::DELETED_USER ) . ' = 0' ),
-            __METHOD__,
-            array( 'LIMIT' => 20 )
-        );
-
-        if ( $res === false ) {
-            // This page has no revisions, which is very weird
-            return false;
-        }
-
-        $hasHistory = ( $res->numRows() > 1 );
-        $row = $dbw->fetchObject( $res );
+            // Find out if there was only one contributor
+            // Only scan the last 20 revisions
+            $res = $dbw->select( 'revision', 'rev_user_text',
+                array( 'rev_page' => $title->getArticleID(), $dbw->bitAnd( 'rev_deleted', Revision::DELETED_USER ) . ' = 0' ),
+                __METHOD__,
+                array( 'LIMIT' => 20 )
+            );
+
+            if ( $res === false ) {
+                // This page has no revisions, which is very weird
+                return false;
+            }
 
-        if ( $row ) { // $row is false if the only contributor is hidden
-            $onlyAuthor = $row->rev_user_text;
-            // Try to find a second contributor
-            foreach ( $res as $row ) {
-                if ( $row->rev_user_text != $onlyAuthor ) { // Bug 22999
-                    $onlyAuthor = false;
-                    break;
+            $hasHistory = ( $res->numRows() > 1 );
+            $row = $dbw->fetchObject( $res );
+
+            if ( $row ) { // $row is false if the only contributor is hidden
+                $onlyAuthor = $row->rev_user_text;
+                // Try to find a second contributor
+                foreach ( $res as $row ) {
+                    if ( $row->rev_user_text != $onlyAuthor ) { // Bug 22999
+                        $onlyAuthor = false;
+                        break;
+                    }
                 }
+            } else {
+                $onlyAuthor = false;
             }
-        } else {
-            $onlyAuthor = false;
-        }
 
-        // Generate the summary with a '$1' placeholder
-        if ( $blank ) {
-            // The current revision is blank and the one before is also
-            // blank. It's just not our lucky day
-            $reason = wfMsgForContent( 'exbeforeblank', '$1' );
-        } else {
-            if ( $onlyAuthor ) {
-                $reason = wfMsgForContent( 'excontentauthor', '$1', $onlyAuthor );
+            // Generate the summary with a '$1' placeholder
+            if ( $blank ) {
+                // The current revision is blank and the one before is also
+                // blank. It's just not our lucky day
+                $reason = wfMsgForContent( 'exbeforeblank', '$1' );
             } else {
-                $reason = wfMsgForContent( 'excontent', '$1' );
+                if ( $onlyAuthor ) {
+                    $reason = wfMsgForContent( 'excontentauthor', '$1', $onlyAuthor );
+                } else {
+                    $reason = wfMsgForContent( 'excontent', '$1' );
+                }
             }
-        }
 
-        if ( $reason == '-' ) {
-            // Allow these UI messages to be blanked out cleanly
-            return '';
-        }
+            if ( $reason == '-' ) {
+                // Allow these UI messages to be blanked out cleanly
+                return '';
+            }
+
+            // Max content length = max comment length - length of the comment (excl. $1)
+            $text = $content->getTextForSummary( 255 - ( strlen( $reason ) - 2 ) );
 
-        // Max content length = max comment length - length of the comment (excl. $1)
-        $text = $content->getTextForSummary( 255 - ( strlen( $reason ) - 2 ) );
+            // Now replace the '$1' placeholder
+            $reason = str_replace( '$1', $text, $reason );
 
-        // Now replace the '$1' placeholder
-        $reason = str_replace( '$1', $text, $reason );
+            return $reason;
+        } catch (MWException $e) {
+            # if a page is horribly broken, we still want to be able to delete it. so be lenient about errors here.
+            wfDebug("Error while building auto delete summary: $e");
+        }
 
-        return $reason;
+        return '';
     }
 
     /**