Unconf Command

[sli]

Handy for unsetting lockMap, which I find myself needing to do from time to time, or any other config variable, of course. I also cleaned up what configModify says (try unconf'ing then conf'ing a variable to see what I mean).

Code: Select all

Index: src/Commands.pm
===================================================================
--- src/Commands.pm	(revision 6342)
+++ src/Commands.pm	(working copy)
@@ -76,6 +76,7 @@
 	cl                 => \&cmdChatLogClear,
 	closeshop          => \&cmdCloseShop,
 	conf               => \&cmdConf,
+	unconf             => \&cmdUnconf,
 	damage             => \&cmdDamage,
 	deal               => \&cmdDeal,
 	debug              => \&cmdDebug,
@@ -1063,6 +1064,22 @@
 	main::closeShop();
 }
 
+sub cmdUnconf {
+	my (undef, $args) = @_;
+	my ($arg1) = $args =~ /^(\w*\.*\w+)/;
+
+	if (exists $config{$arg1}) {
+		Plugins::callHook('Commands::cmdConf', {
+			key => $arg1,
+			val => ''
+		});
+		configModify($arg1, '');
+		Log::initLogFiles();
+	} else {
+		error TF("Config variable %s doesn't exist\n", $arg1);
+	}
+}
+
 sub cmdConf {
 	my (undef, $args) = @_;
 	my ($arg1) = $args =~ /^(\w*\.*\w+)/;
Index: src/Misc.pm
===================================================================
--- src/Misc.pm	(revision 6342)
+++ src/Misc.pm	(working copy)
@@ -289,7 +289,13 @@
 
 	if (!$args{silent} && $key !~ /password/i) {
 		my $oldval = $config{$key};
-		message TF("Config '%s' set to %s (was %s)\n", $key, $val, $oldval), "info";
+		if ($val ne '' && $oldval ne '') {
+			message TF("Config '%s' set to %s (was %s)\n", $key, $val, $oldval), "info";
+		} elsif ($oldval eq '') {
+			message TF("Config '%s' set to %s\n", $key, $val), "info";
+		} else {
+			message TF("Config '%s' unset (was %s)\n", $key, $oldval), "info";
+		}
 	}
 	if ($args{autoCreate} && !exists $config{$key}) {
 		my $f;

[Kissa2k]

You can do this with conf command too. For example:

conf lockMap none

[sli]

I like typing less than that. Plus, there's my whole... aversion towards using arbitrary strings to represent internal stuff (like "none" representing a null value instead of... just using a null value).

[darkfate]

I like typing less than that.

how lazy can a human actually become?

[sli]

I like typing less than that.

how lazy can a human actually become?

Pretty damn lazy. Some of us are college students, you know.

[Cozzie]

isnt that the whole point of computers? to make lives easier? lol

[Kissa2k]

I remember that quote from Futurama:

Bender: You humans think that robots are just machines made to make people's lives easier.
Fry: Well, aren't they?
Bender: I've never made anyone's life easier and you know it!

[sli]

Yeah, but that's Bender. He hit a busted light bulb and it scrambled his AI, causing him to not benefit humans like he should. But then again, Hedonismbot doesn't really benefit anyone, either. But I think he was just made that way.

[Kissa2k]

lol

[isieo]

it could even get shorter if you use alias =P