FabricMC/fabric
1
0
Fork 1
mirror of https://github.com/FabricMC/fabric.git synced 2025-04-21 11:20:55 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions7

Fix log spam with multiple calls to combinedItemApiProvider ()

Browse source 
(cherry picked from commit d6f4a34e15)
(cherry picked from commit bb1ee60efb)
This commit is contained in:
Technici4n 2023-11-02 11:35:06 +01:00 • committed by modmuss50
parent ae9f657a88
commit 1bc31aeb60
1 changed files with 6 additions and 2 deletions

8
fabric-transfer-api-v1/src/main/java/net/fabricmc/fabric/impl/transfer/fluid/CombinedProvidersImpl.java
View file

@ -82,10 +82,14 @@ public class CombinedProvidersImpl {
}
public static Event<FluidStorage.CombinedItemApiProvider> getOrCreateItemEvent(Item item) {
// register here is thread-safe, so the query below will return a valid provider (possibly one registered before or from another thread).
FluidStorage.ITEM.registerForItems(new Provider(), item);
ItemApiLookup.ItemApiProvider<Storage<FluidVariant>, ContainerItemContext> existingProvider = FluidStorage.ITEM.getProvider(item);
if (existingProvider == null) {
FluidStorage.ITEM.registerForItems(new Provider(), item);
// The provider might not be new Provider() if a concurrent registration happened, re-query.
existingProvider = FluidStorage.ITEM.getProvider(item);
}
if (existingProvider instanceof Provider registeredProvider) {
return registeredProvider.event;
} else {