minetest-mods / areas

A rewrite of the node_ownership Minetest mod with many new features.
GNU Lesser General Public License v2.1
51 stars 52 forks source link

/areas_cleanup for removing ownerless areas #43

Closed MoNTE48 closed 4 years ago

MoNTE48 commented 4 years ago

Not sure if anyone else needs this. I created my fork of this good mod some time ago.

During this time, more than 10,000 areas appeared on my server. I became worried that after removing a player, all of his areas are saved. Manually checking and clearing 10,000 records is not possible. So I add chatcommand for it. If only I need this, I'm sorry.

Panquesito7 commented 4 years ago

This new command is not listed on README.md, it should be added there.

MoNTE48 commented 4 years ago

Thanks for the reminder, fixed!

* areas_cleanup -- Removes all ownerless areas. Useful for cleaning after user deletion, for example using /remove_player.

MoNTE48 commented 4 years ago

@SmallJoker If anyone other than me is interested in this...

BuckarooBanzay commented 4 years ago

(maybe offtopic)

I became worried that after removing a player, all of his areas are saved

@MoNTE48 out of curiosity: on what basis do you remove players?

MoNTE48 commented 4 years ago

1) The DB of more than 0.5 million players has become too large, many good names are occupied 2) The player has not visited for more than 6 months. Then his datas is deleted, and his area is also freed.

SmallJoker commented 4 years ago

Looks good. Will squash my commit into N°2 (fa11486) and merge this PR in a few days unless there are objections.