Reduce debug log level for autosaving debug msgs

init.lua

@@ -98,30 +98,32 @@ function playereffects.register_effect_type(effect_type_id, description, icon, g
 	minetest.log("action", "[playereffects] Effect type "..effect_type_id.." registered!")
 end
 
-function playereffects.apply_effect_type(effect_type_id, duration, entity, repeat_interval_time_left)
+function playereffects.apply_effect_type(effect_type_id, duration, player, repeat_interval_time_left)
 	local start_time = os.time()
 	local is_player = false
-	if(type(entity)=="userdata") then
+	if(type(player)=="userdata") then
-		if(entity.is_player ~= nil) then
+		if(player.is_player ~= nil) then
-			if(entity:is_player() == true) then
+			if(player:is_player() == true) then
 				is_player = true
 			end
 		end
 	end
+	if(is_player == false) then
+		minetest.log("error", "[playereffects] Attempted to apply effect type "..effect_type_id.." to a non-player!")
+		return false
+	end
 
-	local playername = entity:get_player_name()
+	local playername = player:get_player_name()
 	local groups = playereffects.effect_types[effect_type_id].groups
 	for k,v in pairs(groups) do
-		playereffects.cancel_effect_group(v, entity)
+		playereffects.cancel_effect_group(v, playername)
 	end
 
 	local metadata
 	if(playereffects.effect_types[effect_type_id].repeat_interval == nil) then
-		local status = playereffects.effect_types[effect_type_id].apply(entity)
+		local status = playereffects.effect_types[effect_type_id].apply(player)
 		if(status == false) then
-			if is_player then
+			minetest.log("action", "[playereffects] Attempt to apply effect type "..effect_type_id.." to player "..playername.." failed!")
-				minetest.log("action", "[playereffects] Attempt to apply effect type "..effect_type_id.." to player "..playername.." failed!")
-			end
 			return false
 		else
 			metadata = status
@@ -130,8 +132,32 @@ function playereffects.apply_effect_type(effect_type_id, duration, entity, repea
 
 
 	local effect_id = playereffects.next_effect_id()
+	local smallest_hudpos
+	local biggest_hudpos = -1
+	local free_hudpos
+	if(playereffects.hudinfos[playername] == nil) then
+		playereffects.hudinfos[playername] = {}
+	end
+	local hudinfos = playereffects.hudinfos[playername]
+	for effect_id, hudinfo in pairs(hudinfos) do
+		local hudpos = hudinfo.pos
+		if(hudpos > biggest_hudpos) then
+			biggest_hudpos = hudpos
+		end
+		if(smallest_hudpos == nil) then
+			smallest_hudpos = hudpos
+		elseif(hudpos < smallest_hudpos) then
+			smallest_hudpos = hudpos
+		end
+	end
+	if(smallest_hudpos == nil) then
+		free_hudpos = 0
+	elseif(smallest_hudpos >= 0) then
+		free_hudpos = smallest_hudpos - 1
+	else
+		free_hudpos = biggest_hudpos + 1
+	end
 
-	-- repeat stuff
 	local repeat_interval = playereffects.effect_types[effect_type_id].repeat_interval
 	if(repeat_interval ~= nil) then
 		if(repeat_interval_time_left == nil) then
@@ -139,47 +165,18 @@ function playereffects.apply_effect_type(effect_type_id, duration, entity, repea
 		end
 	end
 
-	-- Handle HUD
+	--[[ show no more than 20 effects on the screen, so that hud_update does not need to be called so often ]]
-	if is_player then
+	local text_id, icon_id
-		local smallest_hudpos
+	if(free_hudpos <= 20) then
-		local biggest_hudpos = -1
+		text_id, icon_id = playereffects.hud_effect(effect_type_id, player, free_hudpos, duration, repeat_interval_time_left)
-		local free_hudpos
+		local hudinfo = {
-		if(playereffects.hudinfos[playername] == nil) then
+				text_id = text_id,
-			playereffects.hudinfos[playername] = {}
+				icon_id = icon_id,
-		end
+				pos = free_hudpos,
-		local hudinfos = playereffects.hudinfos[playername]
+		}
-		for effect_id, hudinfo in pairs(hudinfos) do
+		playereffects.hudinfos[playername][effect_id] = hudinfo
-			local hudpos = hudinfo.pos
+	else
-			if(hudpos > biggest_hudpos) then
+		text_id, icon_id = nil, nil
-				biggest_hudpos = hudpos
-			end
-			if(smallest_hudpos == nil) then
-				smallest_hudpos = hudpos
-			elseif(hudpos < smallest_hudpos) then
-				smallest_hudpos = hudpos
-			end
-		end
-		if(smallest_hudpos == nil) then
-			free_hudpos = 0
-		elseif(smallest_hudpos >= 0) then
-			free_hudpos = smallest_hudpos - 1
-		else
-			free_hudpos = biggest_hudpos + 1
-		end
-
-		--[[ show no more than 20 effects on the screen, so that hud_update does not need to be called so often ]]
-		local text_id, icon_id
-		if(free_hudpos <= 20) then
-			text_id, icon_id = playereffects.hud_effect(effect_type_id, entity, free_hudpos, duration, repeat_interval_time_left)
-			local hudinfo = {
-					text_id = text_id,
-					icon_id = icon_id,
-					pos = free_hudpos,
-			}
-			playereffects.hudinfos[playername][effect_id] = hudinfo
-		else
-			text_id, icon_id = nil, nil
-		end
 	end
 
 	local effect = {
@@ -196,7 +193,7 @@ function playereffects.apply_effect_type(effect_type_id, duration, entity, repea
 	playereffects.effects[effect_id] = effect
 
 	if(repeat_interval ~= nil) then
-		minetest.after(repeat_interval_time_left, playereffects.repeater, effect_id, duration, entity, playereffects.effect_types[effect_type_id].apply)
+		minetest.after(repeat_interval_time_left, playereffects.repeater, effect_id, duration, player, playereffects.effect_types[effect_type_id].apply)
 	else
 		minetest.after(duration, function(effect_id) playereffects.cancel_effect(effect_id) end, effect_id)
 	end
@@ -204,11 +201,11 @@ function playereffects.apply_effect_type(effect_type_id, duration, entity, repea
 	return effect_id
 end
 
-function playereffects.repeater(effect_id, repetitions, entity, apply)
+function playereffects.repeater(effect_id, repetitions, player, apply)
 	local effect = playereffects.effects[effect_id]
-	if(effect ~= nil and entity ~= nil and entity:get_luaentity() ~= nil) then
+	if(effect ~= nil) then
 		local repetitions = effect.time_left
-		apply(entity)
+		apply(player)
 		repetitions = repetitions - 1
 		effect.time_left = repetitions
 		if(repetitions <= 0) then
@@ -222,15 +219,15 @@ function playereffects.repeater(effect_id, repetitions, entity, apply)
 				playereffects.repeater,
 				effect_id,
 				repetitions,
-				entity,
+				player,
 				apply
 			)
 		end
 	end
 end
 
-function playereffects.cancel_effect_type(effect_type_id, cancel_all, entity_or_playername)
+function playereffects.cancel_effect_type(effect_type_id, cancel_all, playername)
-	local effects = playereffects.get_player_effects(entity_or_playername)
+	local effects = playereffects.get_player_effects(playername)
 	if(cancel_all==nil) then cancel_all = false end
 	for e=1, #effects do
 		if(effects[e].effect_type_id == effect_type_id) then
@@ -242,8 +239,8 @@ function playereffects.cancel_effect_type(effect_type_id, cancel_all, entity_or_
 	end
 end
 
-function playereffects.cancel_effect_group(groupname, entity_or_playername)
+function playereffects.cancel_effect_group(groupname, playername)
-	local effects = playereffects.get_player_effects(entity_or_playername)
+	local effects = playereffects.get_player_effects(playername)
 	for e=1,#effects do
 		local effect = effects[e]
 		local thesegroups = playereffects.effect_types[effect.effect_type_id].groups
@@ -271,35 +268,22 @@ function playereffects.cancel_effect(effect_id)
 	local effect = playereffects.effects[effect_id]
 	if(effect ~= nil) then
 		local player = minetest.get_player_by_name(effect.playername)
-		if player then
+		local hudinfo = playereffects.hudinfos[effect.playername][effect_id]
-			local hudinfo = playereffects.hudinfos[effect.playername][effect_id]
+		if(hudinfo ~= nil) then
-			if(hudinfo ~= nil) then
+			if(hudinfo.text_id~=nil) then
-				if(hudinfo.text_id~=nil) then
+				player:hud_remove(hudinfo.text_id)
-					player:hud_remove(hudinfo.text_id)
-				end
-				if(hudinfo.icon_id~=nil) then
-					player:hud_remove(hudinfo.icon_id)
-				end
-				playereffects.hudinfos[effect.playername][effect_id] = nil
 			end
-
+			if(hudinfo.icon_id~=nil) then
-			-- TODO: Implement cancellation for non-players
+				player:hud_remove(hudinfo.icon_id)
-			playereffects.effect_types[effect.effect_type_id].cancel(effect, player)
+			end
+			playereffects.hudinfos[effect.playername][effect_id] = nil
 		end
+		playereffects.effect_types[effect.effect_type_id].cancel(effect, player)
 		playereffects.effects[effect_id] = nil
 	end
 end
 
-function playereffects.get_player_effects(entity_or_playername)
+function playereffects.get_player_effects(playername)
-	-- TODO: support entity
-	local playername = ""
-	if type(entity_or_playername) == "string" then
-		playername = entity_or_playername
-	elseif type(entity_or_playername) == "userdata" and entity_or_playername:is_player() then
-		playername = entity_or_playername:get_player_name()
-	else
-		return {}
-	end
 	if(minetest.get_player_by_name(playername) ~= nil) then
 		local effects = {}
 		for k,v in pairs(playereffects.effects) do
@@ -313,8 +297,8 @@ function playereffects.get_player_effects(entity_or_playername)
 	end
 end
 
-function playereffects.has_effect_type(entity_or_playername, effect_type_id)
+function playereffects.has_effect_type(playername, effect_type_id)
-	local pe = playereffects.get_player_effects(entity_or_playername)
+	local pe = playereffects.get_player_effects(playername)
 	for i=1,#pe do
 		if pe[i].effect_type_id == effect_type_id then
 			return true
@@ -330,13 +314,11 @@ function playereffects.save_to_file()
 	local inactive_effects = {}
 	for id,effecttable in pairs(playereffects.inactive_effects) do
 		local playername = id
-		if playername ~= "" then
+		if(inactive_effects[playername] == nil) then
-			if(inactive_effects[playername] == nil) then
+			inactive_effects[playername] = {}
-				inactive_effects[playername] = {}
+		end
-			end
+		for i=1,#effecttable do
-			for i=1,#effecttable do
+			table.insert(inactive_effects[playername], effecttable[i])
-				table.insert(inactive_effects[playername], effecttable[i])
-			end
 		end
 	end
 	for id,effect in pairs(playereffects.effects) do
@@ -360,9 +342,7 @@ function playereffects.save_to_file()
 		if(inactive_effects[effect.playername] == nil) then
 			inactive_effects[effect.playername] = {}
 		end
-		if effect.playername ~= "" then
+		table.insert(inactive_effects[effect.playername], new_effect)
-			table.insert(inactive_effects[effect.playername], new_effect)
-		end
 	end
 
 	savetable.inactive_effects = inactive_effects
@@ -375,7 +355,7 @@ function playereffects.save_to_file()
 	if file then
 		file:write(savestring)
 		io.close(file)
-		minetest.log("action", "[playereffects] Wrote playereffects data into "..filepath..".")
+		minetest.log("info", "[playereffects] Wrote playereffects data into "..filepath..".")
 	else
 		minetest.log("error", "[playereffects] Failed to write playereffects data into "..filepath..".")
 	end
@@ -413,7 +393,7 @@ minetest.register_on_leaveplayer(function(player)
 end)
 
 minetest.register_on_shutdown(function()
-	minetest.log("action", "[playereffects] Server shuts down. Rescuing data into playereffects.mt")
+	minetest.log("info", "[playereffects] Server shuts down. Rescuing data into playereffects.mt")
 	playereffects.save_to_file()
 end)
 
@@ -447,7 +427,7 @@ minetest.register_globalstep(function(dtime)
 	-- Autosave into file
 	if(playereffects.use_autosave == true and playereffects.autosave_timer >= playereffects.autosave_time) then
 		playereffects.autosave_timer = 0
-		minetest.log("action", "[playereffects] Autosaving mod data to playereffects.mt ...")
+		minetest.log("info", "[playereffects] Autosaving mod data to playereffects.mt ...")
 		playereffects.save_to_file()
 	end
 end)