r917 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r916‎ | r917 | r918 >
Date:07:17, 18 November 2002
Author:vibber
Status:old
Tags:
Comment:
More helpful explanatory text in page move
Modified paths:
  • /trunk/phpwiki/newcodebase/Language.php (modified) (history)
  • /trunk/phpwiki/newcodebase/LanguageEo.php (modified) (history)
  • /trunk/phpwiki/newcodebase/SpecialMovepage.php (modified) (history)

Diff [purge]

Index: trunk/phpwiki/newcodebase/LanguageEo.php
@@ -745,7 +745,7 @@
746746 "rclsub" => "(al paĝoj ligitaj de \"$1\")",
747747 "debug" => "Kontraŭcima",
748748 "newpages" => "Novaj paĝoj",
749 -"movethispage" => "Translokigi la paĝon",
 749+"movethispage" => "Movu la paĝon",
750750 "unusedimagestext" => "<p>Notu, ke aliaj TTT-ejoj, ekzemple
751751 la alilingvaj Vikipedioj, povas rekte ligi al dosier per URL.
752752 Tio ne estus enkalkutita en la jena listo.",
@@ -935,11 +935,28 @@
936936 "movepagetext" => "Per la jena formulo vi povas ŝanĝi la nomon de iu paĝo, kunportante
937937 ĝian historion de redaktoj je la nova nomo.
938938 La antaŭa titolo fariĝos alidirektilo al la nova titolo.
939 -Ligiloj al la antaŭa titolo <i>ne</i> estos ŝanĝitaj.
 939+Ligiloj al la antaŭa titolo <i>ne</i> estos ŝanĝitaj; uzu
 940+la [[Speciala:Maintenance|riparilojn kaj zorgilojn]] por certigi,
 941+ke ne restos duoblaj aŭ fuŝitaj alidirektiloj.
 942+Kiel movanto, vi respondecas pri ĝustigado de fuŝitaj ligiloj.
 943+
 944+Notu, ke la paĝo '''ne''' estos movita se jam ekzistas paĝo
 945+ĉe la nova titolo, krom se ĝi estas malplena aŭ alidirektilo
 946+al ĉi tiu paĝo, kaj sen antaŭa redaktohistorio. Pro tio, vi ja
 947+povos removi la paĝon je la antaŭa titolo se vi mistajpus, kaj
 948+neeblas ke vi neintence forviŝus ekzistantan paĝon per movo.
 949+
940950 <b>AVERTO!</b>
941951 Tio povas esti drasta kaj neatendita ŝanĝo por populara paĝo;
942952 bonvolu certigi vin, ke vi komprenas ties konsekvencojn antaŭ
943953 ol vi antaŭeniru.",
 954+
 955+"movepagetalktext" => "La movo aŭtomate kunportos la diskuto-paĝon, se tia ekzistas, '''krom se:'''
 956+*Vi movas la paĝon tra sekcioj (ekz de ''Nomo'' je ''Vikipediisto:Nomo''),
 957+*Ne malplena diskuto-paĝo jam ekzistas je la nova nomo, aŭ
 958+*Vi malelektas la suban ŝaltilon.
 959+
 960+Tiujokaze, vi nepre permane kunigu la diskuto-paĝojn se vi tion deziras.",
944961 "movearticle" => "Movu paĝon",
945962 "movenologin" => "Ne salutinta",
946963 "movenologintext" => "Vi nepre estu registrita uzulo kaj <a href=\"" .
@@ -952,6 +969,9 @@
953970 "articleexists" => "Aŭ tiel titolita paĝo jam ekzistas
954971 aŭ la de vi elektita titolo malvalidas.
955972 Bonvolu elekti alian nomon.",
 973+"talkexists" => "Oni ja sukcesis movi la paĝon mem, sed
 974+ne movis la diskuto-paĝon ĉar jam ekzistas tia ĉe la nova titolo.
 975+Bonvolu permane kunigi ilin.",
956976 "movedto" => "movis al",
957977 "movetalk" => "Movu ankaŭ la \"diskuto\"-paĝon, se tia ekzistas.",
958978 "talkpagemoved" => "La diskuto-paĝo estas ankaŭ movita.",
Index: trunk/phpwiki/newcodebase/Language.php
@@ -945,12 +945,27 @@
946946 "movepagetext" => "Using the form below will rename a page, moving all
947947 of its history to the new name.
948948 The old title will become a redirect page to the new title.
949 -Links to the old page title will not be changed, and the talk
950 -page, if any, will not be moved.
 949+Links to the old page title will not be changed; be sure to
 950+[[Special:Maintenance|check]] for double or broken redirects.
 951+You are responsible for making sure that links continue to
 952+point where they are supposed to go.
 953+
 954+Note that the page will '''not''' be moved if there is already
 955+a page at the new title, unless it is empty or a redirect and has no
 956+past edit history. This means that you can rename a page back to where
 957+it was just renamed from if you make a mistake, and you cannot overwrite
 958+an existing page.
 959+
951960 <b>WARNING!</b>
952961 This can be a drastic and unexpected change for a popular page;
953962 please be sure you understand the consequences of this before
954963 proceeding.",
 964+"movepagetalktext" => "The associated talk page, if any, will be automatically moved along with it '''unless:'''
 965+*You are moving the page across namespaces,
 966+*A non-empty talk page already exists under the new name, or
 967+*You uncheck the box below.
 968+
 969+In those cases, you will have to move or merge the page manually if desired.",
955970 "movearticle" => "Move page",
956971 "movenologin" => "Not logged in",
957972 "movenologintext" => "You must be a registered user and <a href=\"" .
@@ -963,6 +978,9 @@
964979 "articleexists" => "A page of that name already exists, or the
965980 name you have chosen is not valid.
966981 Please choose another name.",
 982+"talkexists" => "The page itself was moved successfully, but the
 983+talk page could not be moved because one already exists at the new
 984+title. Please merge them manually.",
967985 "movedto" => "moved to",
968986 "movetalk" => "Move \"talk\" page as well, if applicable.",
969987 "talkpagemoved" => "The corresponding talk page was also moved.",
Index: trunk/phpwiki/newcodebase/SpecialMovepage.php
@@ -34,7 +34,6 @@
3535 global $wpNewTitle, $wpOldTitle, $wpMovetalk, $target;
3636
3737 $wgOut->setPagetitle( wfMsg( "movepage" ) );
38 - $wgOut->addWikiText( wfMsg( "movepagetext" ) );
3938
4039 if ( ! $wpOldTitle ) {
4140 $target = wfCleanQueryVar( $target );
@@ -47,6 +46,10 @@
4847 $ot = Title::newFromURL( $wpOldTitle );
4948 $ott = $ot->getPrefixedText();
5049
 50+ $wgOut->addWikiText( wfMsg( "movepagetext" ) );
 51+ if ( ! Namespace::isTalk( $ot->getNamespace() ) )
 52+ $wgOut->addWikiText( "\n\n" . wfMsg( "movepagetalktext" ) );
 53+
5154 $ma = wfMsg( "movearticle" );
5255 $newt = wfMsg( "newtitle" );
5356 $mpb = wfMsg( "movepagebtn" );
@@ -160,12 +163,12 @@
161164
162165 if ( 0 != $this->oldid ) {
163166 if ( 0 != $this->newid ) {
164 - if ( ! $this->isValidTarget() ) {
165 - $this->showForm( wfMsg( "articleexists" ) );
166 - return;
 167+ if ( $this->isValidTarget() ) {
 168+ $this->moveOverExistingRedirect();
 169+ $this->talkmoved = 1;
 170+ } else {
 171+ $this->talkmoved = 'invalid';
167172 }
168 - $this->moveOverExistingRedirect();
169 - $this->talkmoved = 1;
170173 } else {
171174 $this->moveToNewTitle();
172175 $this->talkmoved = 1;
@@ -196,8 +199,13 @@
197200
198201 if ( 1 == $talkmoved ) {
199202 $wgOut->addHTML( "\n<p>" . wfMsg( "talkpagemoved" ) );
 203+ } elseif( 'invalid' == $talkmoved ) {
 204+ $wgOut->addHTML( "\n<p><strong>" . wfMsg( "talkexists" ) . "</strong>" );
200205 } else {
201 - $wgOut->addHTML( "\n<p>" . wfMsg( "talkpagenotmoved" ) );
 206+ $ot = Title::newFromURL( $oldtitle );
 207+ if ( ! Namespace::isTalk( $ot->getNamespace() ) ) {
 208+ $wgOut->addHTML( "\n<p>" . wfMsg( "talkpagenotmoved" ) );
 209+ }
202210 }
203211 }
204212

Status & tagging log