Fix greeting window malfunction (#7847)

* Fixes

* Log the change

* Remove unused var

code/modules/mob/abstract/new_player/login.dm

@@ -25,8 +25,6 @@
 	var/client/my_client // Need to keep track of this ourselves, since by the time Logout() is called the client has already been nulled
 
 /mob/abstract/new_player/Login()
-	client.InitClient()
-	client.InitPrefs()
 	update_Login_details()	//handles setting lastKnownIP and computer_id for use by the ban systems as well as checking for multikeying
 
 	to_chat(src, "<div class='info'>Game ID: <div class='danger'>[game_id]</div></div>")

code/modules/mob/abstract/unauthed/login.dm

@@ -51,11 +51,14 @@
 	if(newkey)
 		client.key = newkey // Try seeting ckey
 	directory[c.ckey] = c
-	// If mob exists for that ckey, then BYOND will transfer client to it.
+	// Init the client and give it a new_player mob.
+	// Note that modifying the key variable does not invoke client/New() or client/Login() again.
+	c.InitClient()
+	c.InitPrefs()
+
 	if(istype(c.mob, /mob/abstract/unauthed))
-		c.mob = new /mob/abstract/new_player() // Else we just treat them as new player
-	c.InitClient() // And now we shall continue client initilization (permissions and stuff)
-	c.InitPrefs() // We init prefs just in case mob transfer didn't
+		c.mob = new /mob/abstract/new_player()
+
 	unauthed -= token
 
 /mob/abstract/unauthed/Topic(href, href_list)

code/modules/mob/login.dm

@@ -24,8 +24,6 @@
 						log_access("Notice: [key_name(src)] has the same [matches] as [key_name(M)] (no longer logged in).",ckey=key_name(src))
 
 /mob/Login()
-	client.InitClient()
-	client.InitPrefs() // Init perfs in case they wasn't initilized
 	player_list |= src
 	update_Login_details()
 	SSfeedback.update_status()