!whereis can be misused by players on Survival / PvP servers about irc HOT 9 OPEN

I've been thinking about removing that command. I'll accept a PR.

from irc.

In my opinion, !whereis is still useful for server staff... Shall I try to restrict that command to chan OPs as mentioned in my previous comment?

from irc.

Sure.

from irc.

definitely worth doing

from irc.

Figured out how to get it to work, but I'm facing a (minor?) issue:

• irc.conn.channels[irc.config.channel] contains a table of users in the channel, called users.
• Each user in the table is indexed by their nick (user.nick), and stores an access table, with boolean flags for user-mode +o and +v.
• The issue here is that the user's access table users[user.nick].access is empty. Am I missing something? Do I have to invoke some sort of an update function before accessing the table?

from irc.

From https://github.com/JakobOvrum/LuaIRC/blob/master/doc/irc.luadoc :

Apart from nick , fields may be missing. To fill them in, enable user tracking

And

Turn user information tracking on or off. User tracking is enabled by default.
param b Boolean whether or not to track user information.
function irc:trackUsers(b)

Either that, or track modes yourself with the OnUserMode callback.

That's what I get from a quick look at the docs and code, but you should probably ask upstream.

from irc.

User tracking is enabled by default.

If tracking is set to true by default (and it really is enabled, I checked), then this most likely is an upstream bug.

I initially did plan on using OnUserMode callback. I'll see if that works...

from irc.

How about removing this command from irc altogether? Commands like these should ideally be a part of MT itself.

minetest/minetest#7677 :)

from irc.

I've abandoned this, and the aforementioned PR in MT has been rejected. Anyone else interested is welcome to take this up.

from irc.