From 220aab468da04b5097e7d617c41f726bef149aa6 Mon Sep 17 00:00:00 2001
From: psychon <psychon@726aef4b-f618-498e-8847-2d620e286838>
Date: Wed, 10 Jun 2009 14:41:50 +0000
Subject: [PATCH] admin: Require a username argument

This patch makes set, setchan and addserver require a username argument.
(One can just use $me)


git-svn-id: https://znc.svn.sourceforge.net/svnroot/znc/trunk@1541 726aef4b-f618-498e-8847-2d620e286838
---
 modules/admin.cpp | 30 +++++++-----------------------
 1 file changed, 7 insertions(+), 23 deletions(-)

diff --git a/modules/admin.cpp b/modules/admin.cpp
index b1aa586f..c1d260c8 100644
--- a/modules/admin.cpp
+++ b/modules/admin.cpp
@@ -172,13 +172,8 @@ class CAdminMod : public CModule {
 		CString value     = sLine.Token(3, true);
 
 		if (value.empty()) {
-			if (!username.empty()) {
-				value    = username;
-				username = m_pUser->GetUserName();;
-			} else {
-				PutModule("Usage: set <variable> [username] <value>");
-				return;
-			}
+			PutModule("Usage: set <variable> <username> <value>");
+			return;
 		}
 
 		CUser* user = GetUser(username);
@@ -259,7 +254,7 @@ class CAdminMod : public CModule {
 		CString chan = sLine.Token(3, true);
 
 		if (var.empty()) {
-			PutModule("Usage: getchan <variable> [username] [chan]");
+			PutModule("Usage: getchan <variable> [username] <chan>");
 			return;
 		}
 		if (chan.empty()) {
@@ -301,14 +296,8 @@ class CAdminMod : public CModule {
 		CString value     = sLine.Token(4, true);
 
 		if (value.empty()) {
-			if (!username.empty() && !chan.empty()) {
-				value    = chan;
-				chan     = username;
-				username = m_pUser->GetUserName();;
-			} else {
-				PutModule("Usage: set <variable> [username] <value>");
-				return;
-			}
+			PutModule("Usage: set <variable> <username> <value>");
+			return;
 		}
 
 		CUser* user = GetUser(username);
@@ -497,13 +486,8 @@ class CAdminMod : public CModule {
 		CString server   = sLine.Token(2, true);
 
 		if (server.empty()) {
-			if (!username.empty()) {
-				server = username;
-				username = "";
-			} else {
-				PutModule("Usage: addserver [username] <server>");
-				return;
-			}
+			PutModule("Usage: addserver <username> <server>");
+			return;
 		}
 
 		CUser* user = GetUser(username);