Fix command bugs, and people joining on second attempt

commands.lua

@@ -17,8 +17,9 @@ minetest.register_chatcommand("nn_unblock", {
 	func = function(caller, argstring)
 		local tokens = tokenize(argstring)
 	
-		for playername in tokens do
-			nonew:unblock( playername )
+		for i = 1, #tokens do
+			minetest.debug("NoNew: "..caller.." unblocks "..tokens[i])
+			nonew:unblock( tokens[i] )
 		end
 	end,
 })
@@ -30,8 +31,9 @@ minetest.register_chatcommand("nn_block", {
 	func = function(caller, argstring)
 		local tokens = tokenize(argstring)
 	
-		for playername in tokens do
-			nonew:block( playername )
+		for i = 1, #tokens do
+			minetest.debug("NoNew: "..caller.." blocks "..tokens[i])
+			nonew:block( tokens[i] )
 		end
 	end,
 })
@@ -44,12 +46,9 @@ local function setstate(caller, argstring, option)
 		nonew[option] = true
 	elseif value == 'off' then
 		nonew[option] = false
-	else
-		minetest.chat_send_player(caller, "Invalid option for "..option.." - please choose on or off.")
-		return
 	end
 
-	minetest.chat_send_player(caller, "NoNew "..option..": "..nonew[option])
+	minetest.chat_send_player(caller, "NoNew "..option..": "..tostring(nonew[option]) )
 end
 
 minetest.register_chatcommand("nn_state",{

init.lua

@@ -11,10 +11,13 @@ end
 function nonew:action(playername)
 	if not nonew.state then return end
 
+	disconnect_string = "No new players are being accepted at the moment."
+
 	if nonew.players[playername] == 1 then
 		minetest.after(0, function()
-			minetest.kick_player(playername, "No new players are being accepted at the moment.")
+			minetest.kick_player(playername, disconnect_string)
 		end)
+		return disconnect_string
 	end
 end
 
@@ -70,7 +73,7 @@ end)
 
 minetest.register_on_prejoinplayer(function(playername, ip)
 	-- If they are already in the block list, just get them here
-	nonew:action(playername)
+	return nonew:action(playername)
 end)
 
 read_blocked_players()