From: Brion Vibber <brion@users.mediawiki.org>
Date: Thu, 14 Oct 2004 05:53:30 +0000 (+0000)
Subject: Fix XHTML tags
X-Git-Tag: 1.5.0alpha1~1550
X-Git-Url: http://git.cyclocoop.org/%7B%24www_url%7Dadmin/password.php?a=commitdiff_plain;h=f2f29a11e35d14241fd03808de1f18c55cb4ef8f;p=lhc%2Fweb%2Fwiklou.git

Fix XHTML tags
---

diff --git a/includes/SpecialLockdb.php b/includes/SpecialLockdb.php
index 363ac9141a..bbc8e7e0be 100644
--- a/includes/SpecialLockdb.php
+++ b/includes/SpecialLockdb.php
@@ -16,7 +16,7 @@ function wfSpecialLockdb()
 		$wgOut->developerRequired();
 		return;
 	}
-	$action = $wgRequest->getText( 'action' );
+	$action = $wgRequest->getVal( 'action' );
 	$f = new DBLockForm();
 
 	if ( "success" == $action ) { $f->showSuccess(); }
@@ -54,21 +54,27 @@ class DBLockForm {
 		$titleObj = Title::makeTitle( NS_SPECIAL, "Lockdb" );
 		$action = $titleObj->escapeLocalURL( "action=submit" );
 
-		$wgOut->addHTML( "<p>
-<form id=\"lockdb\" method=\"post\" action=\"{$action}\">
+		$wgOut->addHTML( <<<END
+<form id="lockdb" method="post" action="{$action}">
 {$elr}:
-<textarea name=\"wpLockReason\" rows=10 cols=60 wrap=virtual>
-</textarea>
-<table border=0><tr>
-<td align=right>
-<input type=checkbox name=\"wpLockConfirm\">
-</td>
-<td align=left>{$lc}<td>
-</tr><tr>
-<td>&nbsp;</td><td align=left>
-<input type=submit name=\"wpLock\" value=\"{$lb}\">
-</td></tr></table>
-</form>\n" );
+<textarea name="wpLockReason" rows="10" cols="60" wrap="virtual"></textarea>
+<table border="0">
+	<tr>
+		<td align="right">
+			<input type="checkbox" name="wpLockConfirm" />
+		</td>
+		<td align="left">{$lc}</td>
+	</tr>
+	<tr>
+		<td>&nbsp;</td>
+		<td align="left">
+			<input type="submit" name="wpLock" value="{$lb}" />
+		</td>
+	</tr>
+</table>
+</form>
+END
+);
 
 	}
 
diff --git a/includes/SpecialUnlockdb.php b/includes/SpecialUnlockdb.php
index 80ce71bd06..f2d056e081 100644
--- a/includes/SpecialUnlockdb.php
+++ b/includes/SpecialUnlockdb.php
@@ -15,11 +15,11 @@ function wfSpecialUnlockdb() {
 		$wgOut->developerRequired();
 		return;
 	}
-	$action = $wgRequest->getText( 'action' );
+	$action = $wgRequest->getVal( 'action' );
 	$f = new DBUnlockForm();
 
 	if ( "success" == $action ) { $f->showSuccess(); }
-	else if ( "submit" == $action ) { $f->doSubmit(); }
+	else if ( "submit" == $action && $wgRequest->wasPosted() ) { $f->doSubmit(); }
 	else { $f->showForm( "" ); }
 }
 
@@ -45,18 +45,26 @@ class DBUnlockForm {
 		$titleObj = Title::makeTitle( NS_SPECIAL, "Unlockdb" );
 		$action = $titleObj->escapeLocalURL( "action=submit" );
 
-		$wgOut->addHTML( "<p>
-<form id=\"unlockdb\" method=\"post\" action=\"{$action}\">
-<table border=0><tr>
-<td align=right>
-<input type=checkbox name=\"wpLockConfirm\">
-</td>
-<td align=\"left\">{$lc}<td>
-</tr><tr>
-<td>&nbsp;</td><td align=left>
-<input type=submit name=\"wpLock\" value=\"{$lb}\">
-</td></tr></table>
-</form>\n" );
+		$wgOut->addHTML( <<<END
+
+<form id="unlockdb" method="post" action="{$action}">
+<table border="0">
+	<tr>
+		<td align="right">
+			<input type="checkbox" name="wpLockConfirm" />
+		</td>
+		<td align="left">{$lc}</td>
+	</tr>
+	<tr>
+		<td>&nbsp;</td>
+		<td align="left">
+			<input type="submit" name="wpLock" value="{$lb}" />
+		</td>
+	</tr>
+</table>
+</form>
+END
+);
 
 	}
 
@@ -69,7 +77,7 @@ class DBUnlockForm {
 			$this->showForm( wfMsg( "locknoconfirm" ) );
 			return;
 		}
-		if ( ! unlink( $wgReadOnlyFile ) ) {
+		if ( @! unlink( $wgReadOnlyFile ) ) {
 			$wgOut->fileDeleteError( $wgReadOnlyFile );
 			return;
 		}