(bug 12120) Unescaped quote in YAML output

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

diff --git a/includes/FileDeleteForm.php b/includes/FileDeleteForm.php
index 7c97d46..13d35f4 100644 (file)
--- a/includes/FileDeleteForm.php
+++ b/includes/FileDeleteForm.php
@@ -72,12 +72,14 @@ class FileDeleteForm {
                        } elseif ( $reason == 'other' ) {
                                $reason = $this->DeleteReason;
                        }
+                               
+                       $article = null;
                        if( $this->oldimage ) {
                                $status = $this->file->deleteOld( $this->oldimage, $reason );
                                if( $status->ok ) {
                                        // Need to do a log item
                                        $log = new LogPage( 'delete' );
-                                       $logComment = wfMsg( 'deletedrevision', $this->oldimage );
+                                       $logComment = wfMsgForContent( 'deletedrevision', $this->oldimage );
                                        if( trim( $reason ) != '' )
                                                $logComment .= ": {$reason}";
                                        $log->addEntry( 'delete', $this->title, $logComment );
@@ -90,6 +92,9 @@ class FileDeleteForm {
                                        $article->doDeleteArticle( $reason );
                                }
                        }
+                       if( $status->isGood() ) wfRunHooks('FileDeleteComplete', array( 
+                               &$this->file, &$this->oldimage, &$article, &$wgUser, &$reason));
+                       
                        if( !$status->isGood() )
                                $wgOut->addWikiText( $status->getWikiText( 'filedeleteerror-short', 'filedeleteerror-long' ) );
                        if( $status->ok ) {
@@ -110,29 +115,48 @@ class FileDeleteForm {
         * Show the confirmation form
         */
        private function showForm() {
-               global $wgOut, $wgUser, $wgRequest;
-               
-               $deletereasonother = Xml::label( wfMsg( 'filedelete-otherreason' ), 'wpReason' );
-               $delcom = Xml::label( wfMsg( 'filedelete-comment' ), 'wpDeleteReasonList' );
-               $reasonDropDown = Xml::listDropDown( 'wpDeleteReasonList',
-                       wfMsgForContent( 'filedelete-reason-dropdown' ), 
-                       wfMsgForContent( 'filedelete-reason-otherlist' ), '', 'wpReasonDropDown', 1 );
+               global $wgOut, $wgUser, $wgRequest, $wgContLang;
+               $align = $wgContLang->isRtl() ? 'left' : 'right';
+
+               $form = Xml::openElement( 'form', array( 'method' => 'post', 'action' => $this->getAction() ) ) .
+                       Xml::openElement( 'fieldset' ) .
+                       Xml::element( 'legend', null, wfMsg( 'filedelete-legend' ) ) .
+                       Xml::hidden( 'wpEditToken', $wgUser->editToken( $this->oldimage ) ) .
+                       $this->prepareMessage( 'filedelete-intro' ) .
+                       Xml::openElement( 'table' ) .
+                       "<tr>
+                               <td align='$align'>" .
+                                       Xml::label( wfMsg( 'filedelete-comment' ), 'wpDeleteReasonList' ) .
+                               "</td>
+                               <td>" .
+                                       Xml::listDropDown( 'wpDeleteReasonList',
+                                               wfMsgForContent( 'filedelete-reason-dropdown' ), 
+                                               wfMsgForContent( 'filedelete-reason-otherlist' ), '', 'wpReasonDropDown', 1 ) .
+                               "</td>
+                       </tr>
+                       <tr>
+                               <td align='$align'>" .
+                                       Xml::label( wfMsg( 'filedelete-otherreason' ), 'wpReason' ) .
+                               "</td>
+                               <td>" .
+                                       Xml::input( 'wpReason', 60, $wgRequest->getText( 'wpReason' ), array( 'type' => 'text', 'maxlength' => '255', 'tabindex' => '2', 'id' => 'wpReason' ) ) .
+                               "</td>
+                       </tr>
+                       <tr>
+                               <td></td>
+                               <td>" .
+                                       Xml::submitButton( wfMsg( 'filedelete-submit' ), array( 'name' => 'mw-filedelete-submit', 'id' => 'mw-filedelete-submit', 'tabindex' => '3' ) ) .
+                               "</td>
+                       </tr>" .
+                       Xml::closeElement( 'table' ) .
+                       Xml::closeElement( 'fieldset' ) .
+                       Xml::closeElement( 'form' );
 
-               $form  = Xml::openElement( 'form', array( 'method' => 'post', 'action' => $this->getAction() ) );
-               $form .= '<fieldset><legend>' . wfMsgHtml( 'filedelete-legend' ) . '</legend>';
-               $form .= Xml::hidden( 'wpEditToken', $wgUser->editToken( $this->oldimage ) );
-               $form .= '<table><tr><td colspan="2">';
-               $form .= $this->prepareMessage( 'filedelete-intro' );
-               $form .= "</td></tr><tr><td align=\"right\"> $delcom </td><td align=\"left\">";
-               $form .= $reasonDropDown;
-               $form .= "</td></tr><tr><td align=\"right\"> $deletereasonother </td><td align=\"left\">";
-               $form .= "<input type='text' maxlength='255' size='60' name='wpReason' id='wpReason' ";
-               $form .= "value=\"". htmlspecialchars( $wgRequest->getText( 'wpReason' ) ) ."\" tabindex=\"1\" />";
-               $form .= '</td></tr><tr><td colspan="2">';
-               $form .= '<p>' . Xml::submitButton( wfMsg( 'filedelete-submit' ), array( 'name' => 'mw-filedelete-submit', 'id' => 'mw-filedelete-submit' ) ) . '</p>';
-               $form .= '</td></tr></table>';
-               $form .= '</fieldset>';
-               $form .= '</form>';
+                       if ( $wgUser->isAllowed( 'editinterface' ) ) {
+                               $skin = $wgUser->getSkin();
+                               $link = $skin->makeLink ( 'MediaWiki:Filedelete-reason-dropdown', wfMsgHtml( 'filedelete-edit-reasonlist' ) );
+                               $form .= '<p class="mw-filedelete-editreasons">' . $link . '</p>';
+                       }
 
                $wgOut->addHtml( $form );
        }
@@ -165,16 +189,16 @@ class FileDeleteForm {
         * @return string
         */
        private function prepareMessage( $message ) {
-               global $wgLang, $wgServer;
+               global $wgLang;
                if( $this->oldimage ) {
+                       $url = $this->file->getArchiveUrl( $this->oldimage );
                        return wfMsgExt(
-                               "{$message}-old",
+                               "{$message}-old", # To ensure grep will find them: 'filedelete-intro-old', 'filedelete-nofile-old', 'filedelete-success-old'
                                'parse',
                                $this->title->getText(),
                                $wgLang->date( $this->getTimestamp(), true ),
                                $wgLang->time( $this->getTimestamp(), true ),
-                               $wgServer . $this->file->getArchiveUrl( $this->oldimage )
-                       );
+                               wfExpandUrl( $this->file->getArchiveUrl( $this->oldimage ) ) );
                } else {
                        return wfMsgExt(
                                $message,