Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update to 1.19.3 #50

Merged
merged 9 commits into from
Dec 10, 2022
Prev Previous commit
Next Next commit
Update to RegistrateAdvancementProvider constructor to match similar …
…pattern of other DataProvider constructors
  • Loading branch information
ApexModder committed Dec 9, 2022
commit c00cc4a89fdaa3a4d6a90f122a6e1679b9f6a4fa
Original file line number Diff line number Diff line change
Expand Up @@ -31,13 +31,13 @@ public class RegistrateAdvancementProvider implements RegistrateProvider, Consum

private final AbstractRegistrate<?> owner;
private final PackOutput packOutput;
private final CompletableFuture<HolderLookup.Provider> registries;
private final CompletableFuture<HolderLookup.Provider> registriesLookup;
private final List<CompletableFuture<?>> advancementsToSave = Lists.newArrayList();

public RegistrateAdvancementProvider(AbstractRegistrate<?> owner, PackOutput packOutput, CompletableFuture<HolderLookup.Provider> registriesLookup) {
public RegistrateAdvancementProvider(AbstractRegistrate<?> owner, PackOutput packOutputIn, CompletableFuture<HolderLookup.Provider> registriesLookupIn) {
this.owner = owner;
this.packOutput = packOutput;
registries = registriesLookup;
this.packOutput = packOutputIn;
this.registriesLookup = registriesLookupIn;
}

@Override
Expand All @@ -58,7 +58,7 @@ public MutableComponent desc(String category, String name, String desc) {

@Override
public CompletableFuture<?> run(CachedOutput cache) {
return registries.thenCompose(lookup -> {
return registriesLookup.thenCompose(lookup -> {
advancementsToSave.clear();

try {
Expand Down