/perimeterinfo can flash entities into view even if it fails with "Failed to spawn test entity"
James103 opened this issue ยท 3 comments
To reproduce:
- Install
fabric-carpet-1.16.5-1.4.28+v210224
. - Launch Minecraft.
- Create a new superflat world with cheats enabled, consisting of a single layer of glass at Y = 0.
- Look straight down.
- Run
/perimeterinfo ~ ~ ~ armor_stand
, optionally replacingarmor_stand
with any non-living entity. Failed to spawn test entity
is printed to chat.- (Actual behavior) The test entity briefly flashes into view.
(Expected behavior) The test entity does not flash into view at all.
That is actually intended, as that is the way it is calculates. Also it is a purely visual artefact, so it isnt rly that big a deal tbh.
it mimics the vanilla way of spawning entites based on the id, like /summon does. Problem is that this way you don't know the entity type until it is created using a factory method.
Could rewrite using registries and getting them from there, but I don't thing it is that necessary. An entity to spawn needs to be a mob, otherwise its little pointless. I should change the error message, cause its not true. Didn't fail to spawn a test entity, just the entity to test was pointless.