From: Raimond Spekking Date: Tue, 29 Jan 2008 16:32:49 +0000 (+0000) Subject: * Prettify input form for RTL wikis X-Git-Tag: 1.31.0-rc.0~49712 X-Git-Url: https://git.cyclocoop.org/%20%27.%28%24debut%20%20%20%24par_page%29.%27?a=commitdiff_plain;h=b6c4e92d1429c0e8404e0aea4bf5cdfc0252c720;p=lhc%2Fweb%2Fwiklou.git * Prettify input form for RTL wikis * Consistent appearance to the file deletion form * Consistent usage of Xml::functions, code cleanup --- diff --git a/includes/Article.php b/includes/Article.php index 5840afa636..626d59c210 100644 --- a/includes/Article.php +++ b/includes/Article.php @@ -1954,8 +1954,8 @@ class Article { return; } - $wgOut->setPagetitle( wfMsg( 'confirmdelete' ) ); - + $wgOut->setPagetitle( wfMsg( 'delete-confirm', $this->mTitle->getPrefixedText() ) ); + # Better double-check that it hasn't been deleted yet! $dbw = wfGetDB( DB_MASTER ); $conds = $this->mTitle->pageCond(); @@ -2078,64 +2078,60 @@ class Article { /** * Output deletion confirmation dialog + * @param $par string FIXME: do we need this parameter? One Call from Article::delete with '' only. + * @param $reason string Prefilled reason */ function confirmDelete( $par, $reason ) { - global $wgOut, $wgUser; + global $wgOut, $wgUser, $wgContLang; + $align = $wgContLang->isRtl() ? 'left' : 'right'; wfDebug( "Article::confirmDelete\n" ); - $sub = htmlspecialchars( $this->mTitle->getPrefixedText() ); - $wgOut->setSubtitle( wfMsg( 'deletesub', $sub ) ); + $wgOut->setSubtitle( wfMsg( 'delete-backlink', $wgUser->getSkin()->makeKnownLinkObj( $this->mTitle ) ) ); $wgOut->setRobotpolicy( 'noindex,nofollow' ); $wgOut->addWikiText( wfMsg( 'confirmdeletetext' ) ); - $formaction = $this->mTitle->escapeLocalURL( 'action=delete' . $par ); - - $confirm = htmlspecialchars( wfMsg( 'deletepage' ) ); - $delcom = Xml::label( wfMsg( 'deletecomment' ), 'wpDeleteReasonList' ); - $token = htmlspecialchars( $wgUser->editToken() ); - $watch = Xml::checkLabel( wfMsg( 'watchthis' ), 'wpWatch', 'wpWatch', $wgUser->getBoolOption( 'watchdeletion' ) || $this->mTitle->userIsWatching(), array( 'tabindex' => '2' ) ); - - $deletereasonother = Xml::label( wfMsg( 'deleteotherreason' ), 'wpReason' ); - $reasonDropDown = Xml::listDropDown( 'wpDeleteReasonList', - wfMsgForContent( 'deletereason-dropdown' ), - wfMsgForContent( 'deletereasonotherlist' ), '', 'wpReasonDropDown', 1 ); - - $wgOut->addHTML( " -
- - - - - - - - - - - - - - - - - -
- $delcom: - - $reasonDropDown -
- $deletereasonother - - -
 $watch
  - -
- -
\n" ); - - $wgOut->returnToMain( false, $this->mTitle ); - + $form = Xml::openElement( 'form', array( 'method' => 'post', 'action' => $this->mTitle->getLocalURL( 'action=delete' . $par ), 'id' => 'deleteconfirm' ) ) . + Xml::openElement( 'fieldset' ) . + Xml::element( 'legend', array(), wfMsg( 'delete-legend' ) ) . + Xml::openElement( 'table' ) . + " + " . + Xml::label( wfMsg( 'deletecomment' ), 'wpDeleteReasonList' ) . + " + " . + Xml::listDropDown( 'wpDeleteReasonList', + wfMsgForContent( 'deletereason-dropdown' ), + wfMsgForContent( 'deletereasonotherlist' ), '', 'wpReasonDropDown', 1 ) . + " + + + " . + Xml::label( wfMsg( 'deleteotherreason' ), 'wpReason' ) . + " + " . + Xml::input( 'wpReason', 60, $reason, array( 'type' => 'text', 'maxlength' => '255', 'tabindex' => '2' ) ) . + " + + + + " . + Xml::checkLabel( wfMsg( 'watchthis' ), 'wpWatch', 'wpWatch', $wgUser->getBoolOption( 'watchdeletion' ) || $this->mTitle->userIsWatching(), array( 'tabindex' => '3' ) ) . + " + + + + " . + Xml::submitButton( wfMsg( 'deletepage' ), array( 'name' => 'wpConfirmB', 'id' => 'wpConfirmB', 'tabindex' => '4' ) ) . + " + " . + Xml::closeElement( 'table' ) . + Xml::closeElement( 'fieldset' ) . + Xml::hidden( 'wpEditToken', $wgUser->editToken() ) . + Xml::closeElement( 'fieldset' ) . + Xml::closeElement( 'form' ); + + $wgOut->addHTML( $form ); $this->showLogExtract( $wgOut ); } diff --git a/languages/messages/MessagesDe.php b/languages/messages/MessagesDe.php index 61d4be4d63..3a5d53f0d7 100644 --- a/languages/messages/MessagesDe.php +++ b/languages/messages/MessagesDe.php @@ -1521,10 +1521,10 @@ Um die Einstellungen deiner Beobachtungsliste anzupassen, besuche: {{fullurl:Spe 'excontentauthor' => "Inhalt war: '$1' (einziger Bearbeiter: [[{{ns:User}}:$2|$2]] – [[{{ns:User talk}}:$2|Diskussion]])", 'exbeforeblank' => "Inhalt vor dem Leeren der Seite: '$1'", 'exblank' => 'Seite war leer', -'confirmdelete' => 'Löschen bestätigen', -'deletesub' => '(Lösche „$1“)', +'delete-confirm' => 'Löschen von „$1“', +'delete-legend' => 'Löschen', 'historywarning' => 'Achtung, die Seite, die du löschen möchtest, hat eine Versionsgeschichte:', -'confirmdeletetext' => 'Du bist dabei, eine Seite oder eine Datei und alle zugehörigen älteren Versionen zu löschen. Bitte bestätige dazu, dass du dir der Konsequenzen bewusst bist, und dass du in Übereinstimmung mit den [[{{MediaWiki:Policy-url}}|Richtlinien]] handelst.', +'confirmdeletetext' => 'Du bist dabei, eine Seite mit allen zugehörigen älteren Versionen zu löschen. Bitte bestätige dazu, dass du dir der Konsequenzen bewusst bist, und dass du in Übereinstimmung mit den [[{{MediaWiki:Policy-url}}|Richtlinien]] handelst.', 'actioncomplete' => 'Aktion beendet', 'deletedtext' => '„$1“ wurde gelöscht. Im $2 findest du eine Liste der letzten Löschungen.', 'deletedarticle' => 'hat „[[$1]]“ gelöscht', @@ -1532,7 +1532,7 @@ Um die Einstellungen deiner Beobachtungsliste anzupassen, besuche: {{fullurl:Spe 'dellogpagetext' => 'Dies ist das Logbuch der gelöschten Seiten und Dateien.', 'deletionlog' => 'Lösch-Logbuch', 'reverted' => 'Auf eine alte Version zurückgesetzt', -'deletecomment' => 'Grund der Löschung', +'deletecomment' => 'Grund der Löschung:', 'deleteotherreason' => 'Anderer/ergänzender Grund:', 'deletereasonotherlist' => 'Anderer Grund', 'deletereason-dropdown' => ' diff --git a/languages/messages/MessagesDe_formal.php b/languages/messages/MessagesDe_formal.php index ae539be296..088dc32198 100644 --- a/languages/messages/MessagesDe_formal.php +++ b/languages/messages/MessagesDe_formal.php @@ -360,7 +360,7 @@ Um die Einstellungen Ihrer Beobachtungsliste anzupassen, besuchen Sie: {{fullurl # Delete/protect/revert 'historywarning' => 'Achtung, die Seite, die Sie löschen möchten, hat eine Versionsgeschichte:', -'confirmdeletetext' => 'Sie sind dabei, eine Seite oder eine Datei und alle zugehörigen älteren Versionen zu löschen. Bitte bestätigen Sie, dass Sie sich der Konsequenzen bewusst sind, und dass Sie in Übereinstimmung mit den [[{{MediaWiki:Policy-url}}|Richtlinien]] handeln.', +'confirmdeletetext' => 'Sie sind dabei, eine Seite mit allen zugehörigen älteren Versionen zu löschen. Bitte bestätigen Sie, dass Sie sich der Konsequenzen bewusst sind, und dass Sie in Übereinstimmung mit den [[{{MediaWiki:Policy-url}}|Richtlinien]] handeln.', 'deletedtext' => '„$1“ wurde gelöscht. Im $2 finden Sie eine Liste der letzten Löschungen.', 'sessionfailure' => 'Es gab ein Problem mit Ihrer Benutzersitzung. Diese Aktion wurde aus Sicherheitsgründen abgebrochen, um eine falsche Zuordnung Ihrer Änderungen zu einem anderen Benutzer zu verhindern. diff --git a/languages/messages/MessagesEn.php b/languages/messages/MessagesEn.php index 98d83519ea..07e0db39b9 100644 --- a/languages/messages/MessagesEn.php +++ b/languages/messages/MessagesEn.php @@ -1906,20 +1906,20 @@ Feedback and further assistance: 'excontentauthor' => "content was: '$1' (and the only contributor was '[[Special:Contributions/$2|$2]]')", 'exbeforeblank' => "content before blanking was: '$1'", 'exblank' => 'page was empty', -'confirmdelete' => 'Confirm delete', -'deletesub' => '(Deleting "$1")', +'delete-confirm' => 'Delete "$1"', +'delete-backlink' => '← $1', # only translate this message to other languages if you have to change it +'delete-legend' => 'Delete', 'historywarning' => 'Warning: The page you are about to delete has a history:', 'confirmdeletetext' => 'You are about to delete a page along with all of its history. Please confirm that you intend to do this, that you understand the consequences, and that you are doing this in accordance with [[{{MediaWiki:Policy-url}}|the policy]].', 'actioncomplete' => 'Action complete', -'deletedtext' => '"$1" has been deleted. -See $2 for a record of recent deletions.', +'deletedtext' => '"$1" has been deleted. See $2 for a record of recent deletions.', 'deletedarticle' => 'deleted "[[$1]]"', 'dellogpage' => 'Deletion log', 'dellogpagetext' => 'Below is a list of the most recent deletions.', 'deletionlog' => 'deletion log', 'reverted' => 'Reverted to earlier revision', -'deletecomment' => 'Reason for deletion', +'deletecomment' => 'Reason for deletion:', 'deleteotherreason' => 'Other/additional reason:', 'deletereasonotherlist' => 'Other reason', 'deletereason-dropdown' => ' diff --git a/maintenance/language/messageTypes.inc b/maintenance/language/messageTypes.inc index ab2ddab2a0..43ca41c2cf 100644 --- a/maintenance/language/messageTypes.inc +++ b/maintenance/language/messageTypes.inc @@ -223,6 +223,7 @@ $wgOptionalMessages = array( 'hours-abbrev', 'filerevert-backlink', 'filedelete-backlink', + 'delete-backlink', 'pagetitle', 'filename-prefix-blacklist', 'edittools', diff --git a/maintenance/language/messages.inc b/maintenance/language/messages.inc index 804fbdfd8f..d06d6715b0 100644 --- a/maintenance/language/messages.inc +++ b/maintenance/language/messages.inc @@ -1241,8 +1241,9 @@ $wgMessageStructure = array( 'excontentauthor', 'exbeforeblank', 'exblank', - 'confirmdelete', - 'deletesub', + 'delete-confirm', + 'delete-backlink', + 'delete-legend', 'historywarning', 'confirmdeletetext', 'actioncomplete',