MLG-Fortress / ExtraHardMode

The now-official repository for the ExtraHardMode Bukkit Plugin.
http://dev.bukkit.org/bukkit-plugins/fun-hard-mode/
Other
25 stars 21 forks source link

Torch mechanics testing #131

Closed SlimeDog closed 6 years ago

SlimeDog commented 6 years ago

Spigot 1.13.1 latest EHM 1.13-alpha

The following torch mechanics are enabled on my servers:

  1. Torches cannot be placed on dirt or grass.
  2. Torches cannot be placed below Y=30.
  3. Rain will extinguish unprotected torches.

Test results:

  1. PARTIAL FAIL: Torches cannot be placed on top of dirt or grass (or sand). Torches can be placed on the side of any of those blocks.
  2. FAIL: Torches can be placed below Y=30.
  3. FAIL: Rain does not extinguish exposed torches.
RoboMWM commented 6 years ago

Exposed torches seem to be gradually removed over time while it is raining. It's only triggered when weather change changes to raining (i.e. if it's already raining in world on plugin load, it won't trigger).

Torch y level check also checks if the world environment is normal (overworld, vs. nether and the end).

For both cases, it seems there's a new Torch material - WALL_TORCH.

SlimeDog commented 6 years ago

That probably explains the torches placed on the sides of soft blocks: sand, dirt, etc.

SlimeDog commented 6 years ago

Spigot 1.13.1 latest EHM 1.13-alpha-2

The following torch mechanics are configured on my servers:

    Torches:
      No Placement Under Y: 30
      No Placement On Soft Materials: true
      Rain Breaks Torches: true.

"Soft Materials" are not defined in the configuration, but (IMHO) should include: coarse_dirt, dirt, grass_block, mycelium, podzol, sand, and soul_sand.

Test results in overworld:

Test results in nether:

NOTE: All results refer to survival mode. Torches can be placed on top or sides of any block in creative mode, due to:

Creative Mode Bypasses: true
RoboMWM commented 6 years ago

Yea, soft materials are hardcoded. Probably can create a separate issue to make it configurable so someone can PR that.

RoboMWM commented 6 years ago

FAIL[2]: Torches cannot be placed on sides of any blocks.

What's the message that prints for this? Don't see any reason why it'd deny for all blocks.

Also, curious how this could succeed in one world but not the other:

Succeed: Torches cannot be placed on sides of coarse_dirt, mycelium, podzol, or soul_sand. FAIL[4]: Torches can be placed on top or sides of coarse_dirt, mycelium, podzol, or soul_sand.

SlimeDog commented 6 years ago

Spigot 1.13.2 latest EHM 1.13-alpha3

In survival mode, I am unable to place a torch on any surface of a stone block or podzol block at Y=19.

(In creative mode, I am allowed to place torches, of course.)

RoboMWM commented 6 years ago

Should be fixed in alpha 4

SlimeDog commented 6 years ago

Spigot 1.13.2 latest EHM 1.13-alpha3

In survival mode, I am able to place a torch on any surface of a stone block at Y=33. I am able to place a torch on any wall surface of a podzol block, but not the top.

(In creative mode, I am allowed to place torches, of course.)

SlimeDog commented 6 years ago

Spigot 1.13.2 latest EHM 1.13-alpha4

Confirmed:

RoboMWM commented 6 years ago

Yes, this behavior is expected. The only other option is to wait a tick and make the torch pop off if it's attached to a "soft" material. See message in https://github.com/MLG-Fortress/ExtraHardMode/commit/31d56de3090066780782e59011b8dbb32c0084f0

In short, feature was added to mitigate "littering landscape with torches." So I'd expect these cases to be rare.

RoboMWM commented 6 years ago

This of course also allows mobs to spawn in these vanilla-generated landscapes since there's no light from easily-obtainable torches

SlimeDog commented 6 years ago

Note that the torches were attached to soft blocks, just not to the explicit target block. It seems to me that the torches attached to sand in the images above are not consistent with EHM. I agree that the instances are rare -- how often to you see farmland with side-adjacent air? I had to manufacture those test scenarios -- but it should not be allowed.

SlimeDog commented 6 years ago

This of course also allows mobs to spawn in these vanilla-generated landscapes since there's no light from easily-obtainable torches

Yes. According to EHM documentation, one has to place glowstone or fire for light at Y<30. This is just one more hardening feature.