This moves review into a separate permission, aside from "server".
The benefit is that we can grant this to more people without the
need to pass out "server" privs, which has a lot more powers than
needeed.
This doesn't grant noclip. This needs to be done separately. Scoring
will be disabled for those with the review priv.
Since we converted these tools into craftitems, to allow them to
stack, they have been missing from the tools tab. Explicitly insert
them so builders can obtain them easily again.
Instead of ANAND's suggestion to put this function in the `player` API,
I feel it belongs in the `boxes` API instead, other than that it's
mostly similar to his MR#18.
This moves the perk code so it can handle offline players
easily. Granted perks are immediate if the player is online
and otherwise granted at login. This avoids any of the issues
with offline players.
We were accidentally shadowing `nodes` which broke the texture
retrieval call, but worse, we never searched for placeholder
nodes in the search filter.
Boxes need to be tested. Untested boxes can not be submitted.
For small edits, re-testing isn't required (<150sec). Lots of
messaging to the builder as well to make sure it doesn't go
unnoticed.
These were previously filtered out. Unfortunately now there's some
awkward emtpy lines in various places, but, I prefer having empty
lines show since that makes typing much simpler.
This really breaks badly. Too many untranslatable characters that
completely break things. These are meant to be technical and postion
strings are not translatable either. Revert.
Using the inspector, I found that the teleport-into-box code
overwrites and deletes the gravity and speed values for each
player. This was never intentional.
This fixes that problem. However, that means that my physics estimates
are completely off and I will need to redo the custom physics
values to match the expectations on the server and to make
sure this doesn't massively nerf those values.
oops :(
