Draco894

Affects versions 1.20.31.1; 1.20.40.22

" Reed Cartwright added a comment - 20/Apr/22 7:00 PM - edited

I figured out what changed in 1.18.30 that broke creeper farms.

Code Analysis

When spiders (and most mobs) spawn, the game will cancel the spawning attempt if the mob collides with any block that has an AABB (axis-aligned bounding box). The function used for this is the virtual function BlockLegacy::getAABB() or its overrides. A refactoring occurred between 1.17 and 1.18 that added a boolean parameter to this function. I'll call it param_5. When param_5 is set to false, many blocks return an empty AABB. This is likely used to distinguish different types of AABBs, such as collision boxes or hit boxes. Notably param_5 is set to false when the game checks if a mob collides with any block.

In 1.18.30, ButtonBlock::getAABB() has been added to the game. (Buttons used BlockLegacy::getAABB() in earlier versions.) The dev who wrote this function included logic that returns an empty AABB if param_5 is false. This returns an empty bounding box during mob spawning, causing buttons to stop blocking spider spawns. This isn't a minor inconvenience. Even a small creeper farm can have thousands of buttons that would need to be replaced by an alternative block (if one even exists).

The Fix

Remove the logic in ButtonBlock::getAABB() that returns an empty bounding box if param_5 is false. ButtonBlock::getAABB() should always return an accurate bounding box regardless of the value of param_5. Given how quickly changes to pathfinding on coral fans was fixed, I hope that this bug can be quickly fixed as well."

I have a feeling this might be the fix/what broke it this time again.

@GoldenHelmet Moss carpet in Bedrock are treated like wool carpet, can confirm from my SMP base on the previous season that ended 9/1/23.

As for the spiders spawning where they shouldn't because something broke with buttons I present the following screenshots. Ghost overlay is courtesy of MaddHatter and Bud's Technical Resource pack.

My point being Pico, what has been reported here is not a bug. These mobs are intended to be minecartable as I previously indicated. As I also stated these same mobs, save for Iron Golems and only Horses (Donkeys and Mules ride carts), work on Java. There are other bugs, eg random fatal damage while flying, that the devs could be focusing on rather than this frivolous ticket.

So in all actuality for parity sake Java needs horses and Iron Golems added to the cartable mobs, instead of taking this away from Bedrock. Just my honest thoughts.

Yes Pico that would be a Java bug report (MC = Java; MCPE = Bedrock). As I said, if the Dev team so chooses to update horses and other equestrian mobs to not ride minecarts or boats, I am all for parity. However the spirit of this bug report is disparity. By that I mean most of the mobs listed are able to ride a minecart in Java, see uploaded photos.

If we are going for Vanilla parity then we should only look at Horses, MC-96212 , otherwise leave it as "Working as Intended" since there are ways to break boats and minecarts without player direct action (eg minecart ran into a cactus, activator rails). We use mobs in minecarts/boats as Mob Switches (permanent mobs that block others from spawning), Ghast Block Blast Chambers (especially given that TNT duping does not exist on Bedrock like Java), and general transportation (eg in game Zoos). @Picomos plays @EVGENSYPERPRO

@Mega_Spud can you help clarify this ticket

This might be related or a separate issue. Skulk can generate hanging unattached and grow on all sides of sensor when spreading.

This ticket needs to be reopened under a similar issue to glow lichen in strongholds. Seed: 6160187550968571116 Precise coordinates: 325 -30 928

Addendum: This is in the 1.19.024 Beta.

Tested in Beta 1.18.10.26 once water is removed spawner functions as intended.

Potentially related to MCPE-142285

Nhi Vo I completed a similar process as GoldenHelmet did. Any fish that are found in deep biomes spawn at a nearby valid space Y=>50. I never noticed spawning according to the second condition GoldenHelmet describes.

In my digging in the .json files for spawn rules the only variation between cod, salmon, and squid is nested in the Vanilla, 1.18.0, and 1.15 folders was the height filter on cod and salmon in 1.18.0.

Package paths:
WindowsApp folder \data\behavior_packs\vanilla\spawn_rules

WindowsApp folder \data\behavior_packs\vanilla_1.15\spawn_rules

WindowsApp folder \data\behavior_packs\vanilla_1.18.0\spawn_rules

 cod.json example:
Vanilla

{
  "format_version": "1.8.0",
  "minecraft:spawn_rules": {
    "description": {
      "identifier": "minecraft:cod",
      "population_control": "water_animal"
    },
    "conditions": [
      {
        "minecraft:spawns_on_surface": {},
        "minecraft:spawns_underwater": {},

        "minecraft:distance_filter": {
          "min": 12,
          "max": 32
        },
        "minecraft:weight": {
          "default": 75
        },
        "minecraft:density_limit": {
          "surface":20
        },
        "minecraft:herd": {
          "min_size":4,
          "max_size":7
        },

        "minecraft:biome_filter": [
          {"test": "has_biome_tag", "operator":"==", "value": "ocean"},
          {"test": "has_biome_tag", "operator":"!=", "value": "warm"}
        ]
      }
    ]
  }
}

1.18

{
  "format_version": "1.8.0",
  "minecraft:spawn_rules": {
    "description": {
      "identifier": "minecraft:cod",
      "population_control": "water_animal"
    },
    "conditions": [
      {
        "minecraft:spawns_on_surface": {},
        "minecraft:spawns_underwater": {},

        "minecraft:distance_filter": {
          "min": 12,
          "max": 32
        },
        "minecraft:weight": {
          "default": 75
        },
        "minecraft:density_limit": {
          "surface":20
        },
        "minecraft:height_filter": {
          "min": 50,
          "max": 64
        },
        "minecraft:herd": {
          "min_size":4,
          "max_size":7
        },

        "minecraft:biome_filter": [
          \{"test": "has_biome_tag", "operator":"==", "value": "ocean"},
          \{"test": "has_biome_tag", "operator":"!=", "value": "warm"}
        ]
      }
    ]
  }
}

This is a difficult one as water caves can come above Y = 50. Perhaps updating the spawn rule to match Drowned minus light level. The only bug in copying over code I can foresee is where Drowned can spawn within cave systems, where we don't want salmon and cod spawning.

Second thought is matching them to vanilla squid spawn rules with a biome discriminator for the correct biomes, as squid spawn in warm oceans if I am not mistaken.

Okay is that a bug, or is it a feature request? The two loot tables don't need to be implicit of one another.

Can you provide more details. Is this on a Realm, Server, Local world? Are you using add-ons? Are you using Experimental?

Is still an issue in 1.17.2 why is a disability accommodation not being resolved after a year?

ItRichHeart not sure why getting rid of Sim distance 4 would benefit anything.

As for lowering the Mob cap Mojang @helper what is the rationale for that? Yes, it can be a pain sometimes but is it the number of mobs or the particular type that is the issue. I know for a fact... need to report this one...Skeletons standing on/in a block that impairs movement removes the random chance of them missing (yes I know coding perspective the chance isn't really gone, but their slowed movement makes shots that much more accurate). If we are going to implement actual parity fixes let us start with Agro distance of Zombie Piglins (increase from 20 effectively 10 blocks to 88 effectively 44 blocks), Magma Cubes attacking Golems of either type.

This is one of those unwelcome changes as it is rendering Nether Side Gold Farms useless for a start. Will make a video response to better clarify these points.

Apologies, did not realize my texture packs worked in the Beta

And it is only when that one player connects, no others have other it

Hi <Owner>! I recently had a case with the exact same symptom your describing, and we were able to fix it by removing all their Bedrock Tweaks addons. Are you using Bedrock Tweaks as well? I think there's a bug in their code, and their site also says "Note: Updating to 1.16.100-1.16.200 right now they may work, or may not". If you'd like we can remove the addons to see if that helps! 10:57 pm February 9

any particular one of the addons... or all of them? Stuart profile I'm not sure, I'm sorry! My best guess was the Multiplayer Sleep feature, since it seems to be the only one that could relate to day cycle, but I can't say for sure. You could try reaching out to the developer to see if they have any suggestions or updates coming soon! will do. I appreciate all the hard work the team does to provide servers Stuart profile Of course, thank you! Let me know if we can help any further! 4w ago.