From f092690945436d0671e229ab5372150c24a20dad Mon Sep 17 00:00:00 2001
From: Shish <shish@shishnet.org>
Date: Mon, 19 Mar 2012 20:57:50 +0000
Subject: [PATCH] bulk tag revert bettering

---
 contrib/tag_history/main.php  |  3 ++-
 contrib/tag_history/theme.php | 10 +++++-----
 2 files changed, 7 insertions(+), 6 deletions(-)

diff --git a/contrib/tag_history/main.php b/contrib/tag_history/main.php
index 33bd5826..ec7d4846 100644
--- a/contrib/tag_history/main.php
+++ b/contrib/tag_history/main.php
@@ -155,7 +155,7 @@ class Tag_History extends Extension {
 
 	protected function process_bulk_revert_request() {
 		if (isset($_POST['revert_name']) && !empty($_POST['revert_name'])) {
-			$revert_name = $_POST['revert_ip'];
+			$revert_name = $_POST['revert_name'];
 		}
 		else {
 			$revert_name = null;
@@ -241,6 +241,7 @@ class Tag_History extends Extension {
 		if(!is_null($name)) {
 			$duser = User::by_name($name);
 			if(is_null($duser)) {
+				$this->theme->add_status($name, "user not found");
 				return;
 			}
 			else {
diff --git a/contrib/tag_history/theme.php b/contrib/tag_history/theme.php
index 1445ffd2..e88a2b81 100644
--- a/contrib/tag_history/theme.php
+++ b/contrib/tag_history/theme.php
@@ -113,16 +113,16 @@ class Tag_HistoryTheme extends Themelet {
 			<br>(Date format: 2011-10-23)
 			'.$validation_msg.'
 
-			<br><br>'.make_form(make_link("admin/bulk_revert"),'POST',false,'revert_ip_form')."
+			<br><br>'.make_form(make_link("tag_history/bulk_revert"), 'POST')."
 				<table class='form'>
-					<tr><th>Username</th>   <td><input type='text' name='revert_name' size='15'></td></tr>
+					<tr><th>Username</th>        <td><input type='text' name='revert_name' size='15'></td></tr>
 					<tr><th>IP&nbsp;Address</th> <td><input type='text' name='revert_ip' size='15'></td></tr>
 					<tr><th>Date&nbsp;range</th> <td><input type='text' name='revert_date' size='15'></td></tr>
-					<tr><td colspan='2'><input type='submit' value='Revert' onclick='return confirm(\"Revert all edits by this IP?\");'></td></tr>
+					<tr><td colspan='2'><input type='submit' value='Revert'></td></tr>
 				</table>
 			</form>
 		";
-		$page->add_block(new Block("Revert By IP", $html));
+		$page->add_block(new Block("Mass Tag Revert", $html));
 	}
 	
 	/*
@@ -131,7 +131,7 @@ class Tag_HistoryTheme extends Themelet {
 	public function display_revert_ip_results() {
 		global $page;
 		$html = implode($this->messages, "\n");
-		$page->add_block(new Block("Revert by IP", $html));
+		$page->add_block(new Block("Bulk Revert Results", $html));
 	}
 	
 	public function add_status(/*string*/ $title, /*string*/ $body) {