From 8737795ba0c14723cbbcc73df8cf89dc12162f74 Mon Sep 17 00:00:00 2001 From: Brion Vibber Date: Wed, 19 Aug 2009 02:33:04 +0000 Subject: [PATCH] Revert r54585 "Fix bug 19261 by saving the username as the signature if it is submitted empty" Looks like the base problem is that empty is being interpreted as "put empty in the sig" instead of as "use default sig" at signature replacement time. We shouldn't be saving the username into preferences if it hasn't been explicitly typed; if the user changes their name, they should automatically pick up the new value. --- includes/Preferences.php | 3 --- 1 file changed, 3 deletions(-) diff --git a/includes/Preferences.php b/includes/Preferences.php index e4da11e205..9553a401ae 100644 --- a/includes/Preferences.php +++ b/includes/Preferences.php @@ -1121,9 +1121,6 @@ class Preferences { // When no fancy sig used, make sure ~{3,5} get removed. $signature = $wgParser->cleanSigInSig( $signature ); } - - global $wgUser; - if (!$signature) $signature = $wgUser->getName(); return $signature; } -- 2.20.1