Fix code style and obvious issues

[rubenwardy]

• Replace deprecated usage of getpos and minetest.env
• Make function local and follow Lua naming convention
• Use minetest.after instead of globalstep (more efficient)
• Fix indentation

[rubenwardy]

Question: what is player.tag supposed to be? You can't store data in userdata, and there is no tag property. This means that not player.tag will always be true

[stujones11]

👍 LGTM a good start, anyway :)

[Elkien3]

I'm actually not sure what I meant by player.tag. originally I actually had two separate mods, that bit was from the "taghide" one I whipped up basically the day I learned it was possible. anyhow, thanks for the PR