You can lasso wolves now ... #5835

New Issue

Boot · 2023-12-26T15:40:06Z

Boot commented

2023-12-26 15:40:06 +00:00

... without taming them five times with raw meat beforehand.

Boot commented

2023-12-26 15:45:35 +00:00

All other petz too.

rewired_X commented

2023-12-26 15:57:06 +00:00

Are you sure the petz are not yours? Cannot reproduce.

Screenshot (1248).png

1.7 MiB

Boot commented

2023-12-26 16:43:10 +00:00

Just tried it with an alt there. And yes, it doesn't work with that. So I'm guessing it's because of one of the privs. But even that would probably be a bug if it only affects one mod and why should any of the privs have such an effect at all? This should be investigated.

flux commented

2023-12-26 19:31:25 +00:00

Just tried it with an alt there. And yes, it doesn't work with that. So I'm guessing it's because of one of the privs. But even that would probably be a bug if it only affects one mod and why should any of the privs have such an effect at all? This should be investigated.

Privileges of Boot: bailiff, ban, basic_privs, cannoneer, channelmod, command_stream, fly, home, interact, kick, locate, mesemaker, noclip, notify, npc_talk_owner, ocean_builder, place, privs, pvp, server, settime, shout, spawn, staff, teleport, teleport_other

apparently it's the server priv: cad2a65800/petz/api/api_on_rightclick.lua (L61)

imo it should instead be protection_bypass

> Just tried it with an alt there. And yes, it doesn't work with that. So I'm guessing it's because of one of the privs. But even that would probably be a bug if it only affects one mod and why should any of the privs have such an effect at all? This should be investigated. ``` Privileges of Boot: bailiff, ban, basic_privs, cannoneer, channelmod, command_stream, fly, home, interact, kick, locate, mesemaker, noclip, notify, npc_talk_owner, ocean_builder, place, privs, pvp, server, settime, shout, spawn, staff, teleport, teleport_other ``` apparently it's the server priv: https://github.com/runsy/petz/blob/cad2a65800723a7cd0c196087e1a39223a15a909/petz/api/api_on_rightclick.lua#L61 imo it should instead be [protection_bypass](https://github.com/minetest/minetest/blob/335af393f09b3629587f14d41a90ded4a3cbddcd/doc/lua_api.md?plain=1#L5160-L5161)

flux added the

1. kind/bug

3. source/mod upstream

ugh/petz

labels 2023-12-26 19:32:01 +00:00

flux commented

2023-12-26 19:33:04 +00:00

actually, imo it should be configurable, default to protection_bypass, and create a new priv if the priv in the setting doesn't exist, similar to what i do w/ e.g. the smartshop_admin priv.

flux commented

2023-12-26 19:34:02 +00:00

and alias recently granted the flux account server priv at my request, so i'll have to be careful with this until it's resolved.

AliasAlreadyTaken commented

2023-12-26 19:59:24 +00:00

protection_bypass is the right priv to grab other people's petz IMO.

Configurable would be nice.

Probably we need to go over privs in general, some privs are "misaligned" in what they do.

protection_bypass is the right priv to grab other people's petz IMO. Configurable would be nice. Probably we need to go over privs in general, some privs are "misaligned" in what they do.

AliasAlreadyTaken was assigned by Boot

2024-01-15 20:59:34 +00:00

Sign in to join this conversation.