From: Brion Vibber <brion@users.mediawiki.org>
Date: Wed, 7 Jan 2009 01:49:32 +0000 (+0000)
Subject: Reverting r45363 "(bug 15320) Show login page if not logged in (for watchlist)"
X-Git-Tag: 1.31.0-rc.0~43536
X-Git-Url: https://git.cyclocoop.org/%7B%24www_url%7Dadmin/membres/message.php?a=commitdiff_plain;h=af1abceb0efc9416c51a2156ab984bb7d7ff0da3;p=lhc%2Fweb%2Fwiklou.git

Reverting r45363 "(bug 15320) Show login page if not logged in (for watchlist)"

1) Behavior should be consistent; we don't do this anywhere else.

2) We want to be able to consolidate the login form to a secure location, which this would prevent.

Reverting and resolving WONTFIX.
---

diff --git a/includes/specials/SpecialWatchlist.php b/includes/specials/SpecialWatchlist.php
index 9c9c1620e7..61dd6b3eea 100644
--- a/includes/specials/SpecialWatchlist.php
+++ b/includes/specials/SpecialWatchlist.php
@@ -20,14 +20,10 @@ function wfSpecialWatchlist( $par ) {
 
 	# Anons don't get a watchlist
 	if( $wgUser->isAnon() ) {
+		$wgOut->setPageTitle( wfMsg( 'watchnologin' ) );
 		$llink = $skin->makeKnownLinkObj( SpecialPage::getTitleFor( 'Userlogin' ), 
 			wfMsgHtml( 'loginreqlink' ), 'returnto=' . $specialTitle->getPrefixedUrl() );
 		$wgOut->addHTML( wfMsgWikiHtml( 'watchlistanontext', $llink ) );
-		# Add login form for convenience
-		$form = new LoginForm( $wgRequest );
-		$form->execute();
-		# Set page title (also LoginForm sets it too, but we prefer this one)
-		$wgOut->setPageTitle( wfMsg( 'watchnologin' ) );
 		return;
 	}