# Breaking changes
- `VillagerPlantableRegistry` replaced with `ItemTags.VILLAGER_PLANTABLE_SEEDS`
- `FabricItemGroup.builder()` no longer takes an `Identifier`
- `FabricItemGroup.build()` no longer registers the ItemGroup, this now needs to go in the vanilla registry.
- `ItemGroupEvents.modifyEntriesEvent` now takes a `RegistryKey<ItemGroup>` in place of an `Identifier`
- `FabricLanguageProvider` now takes a `RegistryKey<ItemGroup>` in place of an `ItemGroup`
- `IdentifiableItemGroup` removed, replaced with vanilla registries.
- `FabricMaterialBuilder` removed, no replacement.
- `HudRenderCallback.onHudRender` now passed a `DrawableHelper` in place of `MatrixStack`
- `ScreenEvents.beforeRender` now passed a `DrawableHelper` in place of `MatrixStack`
- `ScreenEvents.afterRender` now passed a `DrawableHelper` in place of `MatrixStack`
- `Screens.getItemRenderer()` removed. Replace with `MinecraftClient.getItemRenderer()`
`DrawableHelper` is likely to be renamed soon, see: https://github.com/FabricMC/yarn/pull/3548/
* API for brewing recipes which use Ingredients instead of Items
* Improve comments and JavaDoc
* Update mixin compatibility level
* Replace accessor mixin with access widener
* Fix double space
Co-authored-by: Juuz <6596629+Juuxel@users.noreply.github.com>
Co-authored-by: Juuz <6596629+Juuxel@users.noreply.github.com>
(cherry picked from commit 7c6cd14d84)
* Added LandPathNodeTypesRegistry.
* Added test for LandPathNodeTypesRegistry.
* Added block state to PathNodeTypeProvider.getPathNodeType.
* Made LandPathNodeTypesRegistry final, removed unnecessary NotNull.
* Changed putIfAbsent with put to align with other registries.
* Cleanup.
Co-authored-by: Technici4n <13494793+Technici4n@users.noreply.github.com>
* Removed unused import.
* Added possibility to specify the node type of block if the block is found in a neighbor position, improved documentation.
* Merged mixins cleanup.
* Changed CAPTURE_FAILEXCEPTION into CAPTURE_FAILHARD
Co-authored-by: Juuxel <6596629+Juuxel@users.noreply.github.com>
* Specified some nullables, changed 2 parameter names.
* Added missing content registries tests, changed NODE_TYPES map to IdentityHashMap, improved docs.
* Moved PathNodeTypeProvider inside LandPathNodeTypesRegistry.
* Registry revised to make it possible to distinguish between blocks with dynamic and static node types, reduced LandPathNodeMakerMixin priority, applied suggested docs improvements.
* Changed INVOKE with shift, with INVOKE_ASSIGN.
* Merged all LandPathNodeTypesRegistry utility methods used to check the registration type into a single one.
* Inverted if sequence in LandPathNodeTypesRegistry.getPathNodeType, improved docs.
* Sealed PathNodeTypeProvider and made public, cleaned and simplified LandPathNodeTypesRegistry by removing useless mechanics.
* Improved docs.
* Improved docs (2).
Co-authored-by: Technici4n <13494793+Technici4n@users.noreply.github.com>
Co-authored-by: Juuxel <6596629+Juuxel@users.noreply.github.com>
* Added line to Doc to prevent misconception
Added line because people probably want to use their GameEvent for the Warden too.
* Update SculkSensorFrequencyRegistry.java
* Added LandPathNodeTypesRegistry.
* Added test for LandPathNodeTypesRegistry.
* Added block state to PathNodeTypeProvider.getPathNodeType.
* Made LandPathNodeTypesRegistry final, removed unnecessary NotNull.
* Changed putIfAbsent with put to align with other registries.
* Cleanup.
Co-authored-by: Technici4n <13494793+Technici4n@users.noreply.github.com>
* Removed unused import.
* Added possibility to specify the node type of block if the block is found in a neighbor position, improved documentation.
* Merged mixins cleanup.
* Changed CAPTURE_FAILEXCEPTION into CAPTURE_FAILHARD
Co-authored-by: Juuxel <6596629+Juuxel@users.noreply.github.com>
* Specified some nullables, changed 2 parameter names.
* Added missing content registries tests, changed NODE_TYPES map to IdentityHashMap, improved docs.
* Moved PathNodeTypeProvider inside LandPathNodeTypesRegistry.
Co-authored-by: Technici4n <13494793+Technici4n@users.noreply.github.com>
Co-authored-by: Juuxel <6596629+Juuxel@users.noreply.github.com>
The following classes have been made final and unconstructible:
- All convention tags classes
- `FluidVariantAttributes`
- `FluidVariantRendering`
The following classes have been made unconstructible:
- `BiomeModifications`
- `ClientEntityEvents`
- `ClientTickEvents`
- `LootTableEvents`
- `FabricDefaultAttributeRegistry`
- `MinecartComparatorLogicRegistry`
- `StorageUtil`
The following classes have been explicitly marked as final. Note that actually extending such class has always been impossible due to missing public constructor:
- `VillagerInteractionRegistries`
- `VillagerPlantableRegistry`
- `ModelHelper`
- `StoragePreconditions`
While the first two are technically breaking changes, there is no actual or observed usage for any of those.
* Sculk Sensor Frequency Registry
- Mixin'd to `SculkSensorBlock` to make its `FREQUENCIES` map modifiable.
- Created `SculkSensorFrequencyRegistry` with a dedicated method for registering these frequencies, with all necessary checks to avoid issues, and javadoc to assist in understanding.
- Added tests to the test mod, as well as a block that emits a test event, to ensure it all works.
Unfortunately, the mixin currently uses a Redirect. If this is undesired, feel free to suggest other ways of achieving the end result of being able to modify the frequencies map.
* Update SculkSensorFrequencyRegistry.java
* cleaner javadoc
* Update fabric-content-registries-v0/src/main/resources/fabric-content-registries-v0.mixins.json
Co-authored-by: Juuxel <6596629+Juuxel@users.noreply.github.com>
* Final touches
...get it?
(oh and removed that Dynamic because tech didn't like it)
* Whoops
Forgot to remove the Dynamic import
Also restored the actual contents of the dynamic, as javadoc.
Co-authored-by: Juuxel <6596629+Juuxel@users.noreply.github.com>
* Add VillagerFoodRegistry
* Fix style
* Add VillagerCompostingRegistry and separate CollectablesRegistry from FoodRegistry
* Add VillagerPlantableRegistry
* Add warning when registering a non-compostable item as a villager compostable
* Rename some registries, use block placement sound
* Add VillagerHeroGiftRegistry
* Add javadoc
* Combined all registries into one API class
* Remove now redundant class
* Change registries to a static method
* Combine into VillagerInteractionRegistries class
* Fix typo
* Move ImmutableCollectionUtils to impl
* Add isEmpty check to plantables, prefix mixin methods with fabric_
* VillagerPlantableRegistry Rework
* Remove dangling comment
* notNulls, included vanilla items in plantable registry, checkstyle fix
(cherry picked from commit 6f01bfd847)
* Add SyncDataPackContents and TagsLoaded lifecycle events
* tagsLoaded -> onTagsLoaded, isClient -> client
* Compile error in testmod
* Make SyncDataPackContents fire for every player
* syncDataPackContents -> onSyncDataPackContents
(cherry picked from commit cc71601c72)
A common source of crashes on modded Minecraft servers comes from modders accidently calling client only code from the client, this PR is another large step towards elimitating that.
This PR has been months in the making and years in the planning, requiring major changes to Loom & Loader. In recent Minecraft versions Mojang has made it easier than ever to cleanly split the jar, going against the status-quo of merging the client and server into one jar.
From the start we have designed Fabric to have a very clear split between client and common (client & server) code. Fabric has always encoraged keeping client only code seprate from the server, this can be seen at a fundamental level with the entrypoints in Loader. Fabric API's have all been designed with this mind.
This PR provides a compile safety net around Fabric API using client only code on the server. Even though there are almost 400 changed files, minimal changes beyond moving the files were required to achieve this in Fabric API, thanks to the effort of all contributors in the past.
These changes should not affect modders or players in anyway, a single "universal" jar is still produced. Im happy to awnswer any questions.
This snapshot is possibly one of the most impactful for API we have ever had. This PR is an inital port to support 22w06a, stuff will be missing and broken.
# Removed modules:
- fabric-mining-levels-v0 - Previously deprecated
- fabric-object-builders-v0 - Previously deprecated
- fabric-tag-extensions-v0
- fabric-tool-attribute-api-v1
# Modules with API breaking changes:
- fabric-biome-api-v1
- fabric-content-registries-v0
- fabric-data-generation-api-v1
- fabric-mining-level-api-v1
- fabric-object-builder-api-v1
- fabric-resource-conditions-api-v1
- fabric-structure-api-v1
# Impactful API changes:
### fabric-object-builder-api-v1
- Removed - FabricBlockSettings.breakByHand
- Removed - FabricBlockSettings.breakByTool - Previously deprecated
# Notable changes
- fabric-registry-sync-v0 moves vanilla's new registry freezing to a later point in time, allowing mods to add to the registry during init.
# Known issues:
- ServerBugfixMixin used to fix https://bugs.mojang.com/browse/MC-195468 has not yet been ported.
* Oxidizable Blocks registry
* Allow waxable blocks to be registered outside of OxidizableFamilies
* Refactor
Separated OxidizableBlocksRegistry and WaxableBlocksRegistry
Separated OxidizableFamily and WaxableBlockPair
Added null check to WaxableBlockPair
Added several methods
* whoops lol
* Apply suggestions from code review
Co-authored-by: haykam821 <24855774+haykam821@users.noreply.github.com>
* Update ContentRegistryTest.java
* Refactor again lol
Now uses mixins to change the vanilla ImmutableBiMaps to HashBiMaps and adds to them instead of keeping separate maps and having methods of their own. Much cleaner!
* gutted everything lol
removed all the charm and character- uh, i mean, superfluous stuff and reduced the api to the bare basics
oh and also one-way methods which is neat i guess if you're into that sort of thing
* 🦀one way methods are gone🦀
all my homies hate one way methods
maps are linked again
* re-merged oxidizable and waxable registry classes
* Update ContentRegistryTest.java
* implement suggestions by @Juuxel
* Oxidization -> Oxidation
as per https://github.com/FabricMC/yarn/pull/2837
* Improve Dynamics
Co-authored-by: haykam821 <24855774+haykam821@users.noreply.github.com>
* Add flattening, stripping and tilling registries
* Where'd the spaces come from?
* Clarify docs
* Better overloads for TillableBlockRegistry.register
* Mutablise AxeItem.STRIPPED_BLOCKS if needed
* Remove simplest TillableBlockRegistry.register overload because the default predicate is for farmland
* Add test mod
* Update fabric-content-registries-v0/src/main/java/net/fabricmc/fabric/api/registry/TillableBlockRegistry.java
Co-authored-by: haykam821 <24855774+haykam821@users.noreply.github.com>
Co-authored-by: haykam821 <24855774+haykam821@users.noreply.github.com>
* Add custom value denoting module lifecycles.
* Make the module validation work.
My hand has been forced - we must use buildSrc since JsonSlurper is not available in main buildscript.
* Apply task to each project and dont cross projects
* A horrible hack
* Wait what
* It works now.
* Not needed
* Drop unneeded maven repo, cache map lookup
(cherry picked from commit daa38b3d82)
* Cache the fuel time map in the fuel registry
* Use a faster map for the fuel times
* Rebuild the fuel times map on the client side too
* Use more compatible mixins
* Replace a function overwrite by a TAIL inject
* Mixin formatting
* Indentation
* Make cache thread-safe, switch to boxed collection (IHM) since most
uses yield boxed results, refactor cache clearing
* fix import formatting
Co-authored-by: Player <player@player.to>
