1
0
mirror of https://github.com/Suiranoil/SkinRestorer.git synced 2026-01-16 04:42:12 +00:00

Compare commits

...

13 Commits

Author SHA1 Message Date
8fba8a997e Merge branch '1.21.9-multiloader' into 1.21.6-multiloader
# Conflicts:
#	common/src/main/java/net/lionarius/skinrestorer/mixin/ServerLoginPacketListenerImplMixin.java
#	common/src/main/resources/skinrestorer.mixins.json
#	gradle.properties
2026-01-14 23:46:53 +03:00
a0d8f3d518 another backport to 1.21.9 2026-01-14 23:45:13 +03:00
aa3cb3df09 Merge branch '1.21.11-multiloader' into 1.21.9-multiloader 2026-01-14 23:41:52 +03:00
0524b5dab8 the worst bug I encountered in gson 2.10.1 2026-01-14 23:39:53 +03:00
301428147d update CHANGELOG 2026-01-14 21:53:09 +03:00
2e655ce1f3 bump version 2026-01-14 19:06:02 +03:00
7dcd1d4e81 reorder mixins 2026-01-14 18:33:21 +03:00
1aa606038e add forceFirstJoinSkinFetch config 2026-01-14 18:33:18 +03:00
e832e02fd4 add collection provider 2026-01-14 18:32:58 +03:00
ffd2a0dba7 update mineskin 2026-01-12 23:28:32 +03:00
76e801c944 use parchment 2026-01-10 14:55:31 +03:00
da754aa830 backport to 1.21.9
This reverts commit 69a561f679.
2025-12-15 23:31:54 +03:00
b5bed1c441 backport to 1.21.6 2025-10-04 22:59:06 +03:00
43 changed files with 522 additions and 144 deletions

View File

@@ -4,6 +4,11 @@ All notable changes to this project will be documented in this file.
The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/),
and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
## [2.5.0] - 2026-01-14
### Added
- Added collection skin provider (allows assigning random skins from a predefined set)
- Added `forceFirstJoinSkinFetch` config option to force skin fetch on first join even if player already has a skin
## [2.4.3] - 2025-07-25
### Fixed
- Fixed crash on client when loading player head skin (fixes [#63](https://github.com/Suiranoil/SkinRestorer/issues/63) and [#64](https://github.com/Suiranoil/SkinRestorer/issues/64))

View File

@@ -1,3 +1,3 @@
### Fixed
- Fixed crash on client when loading player head skin (fixes [#63](https://github.com/Suiranoil/SkinRestorer/issues/63) and [#64](https://github.com/Suiranoil/SkinRestorer/issues/64))
- Fixed server freeze when loading player head skin
### Added
- Added collection skin provider (allows assigning random skins from a predefined set) (see [wiki](https://github.com/Suiranoil/SkinRestorer/wiki/Configuration#providerscollection))
- Added `forceFirstJoinSkinFetch` config option to force skin fetch on first join even if player already has a skin (see [wiki](https://github.com/Suiranoil/SkinRestorer/wiki/Configuration#forcefirstjoinskinfetch))

View File

@@ -1,6 +1,6 @@
plugins {
// see https://fabricmc.net/develop/ for new versions
id 'fabric-loom' version '1.13-SNAPSHOT' apply false
id 'fabric-loom' version '1.10-SNAPSHOT' apply false
// see https://projects.neoforged.net/neoforged/moddevgradle for new versions
id 'net.neoforged.moddev' version '2.0.+' apply false
// see https://files.minecraftforge.net/net/minecraftforge/gradle/ForgeGradle/ for new versions

View File

@@ -8,7 +8,7 @@ dependencies {
minecraft "com.mojang:minecraft:${minecraft_version}"
mappings loom.layered {
officialMojangMappings()
// parchment("org.parchmentmc.data:parchment-${parchment_minecraft}:${parchment_version}@zip")
parchment("org.parchmentmc.data:parchment-${parchment_minecraft}:${parchment_version}@zip")
}
compileOnly group: 'org.spongepowered', name: 'mixin', version: '0.8.5'

View File

@@ -1,12 +1,12 @@
package net.lionarius.skinrestorer;
import com.google.common.base.Throwables;
import com.mojang.authlib.GameProfile;
import com.mojang.brigadier.CommandDispatcher;
import net.lionarius.skinrestorer.command.SkinCommand;
import net.lionarius.skinrestorer.config.Config;
import net.lionarius.skinrestorer.config.provider.BuiltInProviderConfig;
import net.lionarius.skinrestorer.exception.TransparentException;
import net.lionarius.skinrestorer.mixin.PlayerAccessor;
import net.lionarius.skinrestorer.platform.Services;
import net.lionarius.skinrestorer.skin.SkinIO;
import net.lionarius.skinrestorer.skin.SkinStorage;
@@ -15,7 +15,7 @@ import net.lionarius.skinrestorer.skin.provider.*;
import net.lionarius.skinrestorer.translation.Translation;
import net.lionarius.skinrestorer.util.*;
import net.minecraft.commands.CommandSourceStack;
import net.minecraft.resources.Identifier;
import net.minecraft.resources.ResourceLocation;
import net.minecraft.server.MinecraftServer;
import net.minecraft.server.level.ServerPlayer;
import net.minecraft.world.level.storage.LevelResource;
@@ -70,8 +70,8 @@ public final class SkinRestorer {
return Optional.ofNullable(SkinRestorer.providersRegistry.get(name));
}
public static Identifier resourceLocation(String name) {
return Identifier.fromNamespaceAndPath(SkinRestorer.MOD_ID, name);
public static ResourceLocation resourceLocation(String name) {
return ResourceLocation.fromNamespaceAndPath(SkinRestorer.MOD_ID, name);
}
public static String assetPath(String name) {
@@ -88,6 +88,7 @@ public final class SkinRestorer {
SkinRestorer.registerDefaultSkinProvider(MojangSkinProvider.PROVIDER_NAME, SkinProvider.MOJANG, SkinRestorer.getConfig().providersConfig().mojang());
SkinRestorer.registerDefaultSkinProvider(ElyBySkinProvider.PROVIDER_NAME, SkinProvider.ELY_BY, SkinRestorer.getConfig().providersConfig().ely_by());
SkinRestorer.registerDefaultSkinProvider(MineskinSkinProvider.PROVIDER_NAME, SkinProvider.MINESKIN, SkinRestorer.getConfig().providersConfig().mineskin());
SkinRestorer.registerDefaultSkinProvider(CollectionSkinProvider.PROVIDER_NAME, SkinProvider.COLLECTION, SkinRestorer.getConfig().providersConfig().collection());
}
private static void registerDefaultSkinProvider(String defaultName, SkinProvider provider, BuiltInProviderConfig config) {
@@ -106,28 +107,26 @@ public final class SkinRestorer {
MojangSkinProvider.reload();
ElyBySkinProvider.reload();
MineskinSkinProvider.reload();
CollectionSkinProvider.reload();
}
public static Collection<ServerPlayer> applySkin(MinecraftServer server, Iterable<ServerPlayer> targets, SkinValue value, boolean save) {
public static Collection<ServerPlayer> applySkin(MinecraftServer server, Iterable<GameProfile> targets, SkinValue value, boolean save) {
var acceptedPlayers = new HashSet<ServerPlayer>();
for (var player : targets) {
var profile = player.getGameProfile();
var skin = PlayerUtils.getPlayerSkin(profile);
for (var profile : targets) {
if (!SkinRestorer.getSkinStorage().hasSavedSkin(profile.getId()))
value = value.setOriginalValue(PlayerUtils.getPlayerSkin(profile));
if (!SkinRestorer.getSkinStorage().hasSavedSkin(profile.id()))
value = value.setOriginalValue(skin);
if (PlayerUtils.areSkinPropertiesEquals(value.value(), skin))
if (PlayerUtils.areSkinPropertiesEquals(value.value(), PlayerUtils.getPlayerSkin(profile)))
continue;
if (save)
SkinRestorer.getSkinStorage().setSkin(profile.id(), value);
SkinRestorer.getSkinStorage().setSkin(profile.getId(), value);
var newProfile = PlayerUtils.applyRestoredSkin(profile, value.value());
((PlayerAccessor) player).setGameProfile(newProfile);
PlayerUtils.applyRestoredSkin(profile, value.value());
if (player.connection == null)
var player = server.getPlayerList().getPlayer(profile.getId());
if (player == null)
continue;
PlayerUtils.refreshPlayer(player);
@@ -139,13 +138,13 @@ public final class SkinRestorer {
return acceptedPlayers;
}
public static Collection<ServerPlayer> applySkin(MinecraftServer server, Iterable<ServerPlayer> targets, SkinValue value) {
public static Collection<ServerPlayer> applySkin(MinecraftServer server, Iterable<GameProfile> targets, SkinValue value) {
return SkinRestorer.applySkin(server, targets, value, true);
}
public static CompletableFuture<Result<Collection<ServerPlayer>, String>> setSkinAsync(
MinecraftServer server,
Collection<ServerPlayer> targets,
Collection<GameProfile> targets,
SkinProviderContext context,
boolean save
) {

View File

@@ -1,5 +1,6 @@
package net.lionarius.skinrestorer.command;
import com.mojang.authlib.GameProfile;
import com.mojang.brigadier.CommandDispatcher;
import com.mojang.brigadier.arguments.StringArgumentType;
import com.mojang.brigadier.builder.ArgumentBuilder;
@@ -15,15 +16,12 @@ import net.lionarius.skinrestorer.skin.provider.SkinProviderContext;
import net.lionarius.skinrestorer.translation.Translation;
import net.lionarius.skinrestorer.util.PlayerUtils;
import net.minecraft.commands.CommandSourceStack;
import net.minecraft.commands.Commands;
import net.minecraft.commands.arguments.GameProfileArgument;
import net.minecraft.server.level.ServerPlayer;
import net.minecraft.server.players.NameAndId;
import java.util.Collection;
import java.util.Collections;
import java.util.HashSet;
import java.util.Objects;
import java.util.function.BiFunction;
import java.util.function.Function;
import java.util.function.Supplier;
@@ -56,7 +54,7 @@ public final class SkinCommand {
base.then(
literal("config")
.requires(commandSourceStack -> Commands.LEVEL_OWNERS.check(commandSourceStack.permissions()))
.requires(commandSourceStack -> commandSourceStack.hasPermission(4))
.then(literal("reload").executes(SkinCommand::configReloadSubcommand))
);
@@ -74,41 +72,37 @@ public final class SkinCommand {
SkinProviderContext context = null;
var save = true;
if (!SkinRestorer.getSkinStorage().hasSavedSkin(profile.id())) {
if (profile.properties().containsKey(PlayerUtils.TEXTURES_KEY)) {
if (!SkinRestorer.getSkinStorage().hasSavedSkin(profile.getId())) {
if (profile.getProperties().containsKey(PlayerUtils.TEXTURES_KEY)) {
save = false;
context = MojangSkinProvider.skinProviderContextFromProfile(profile);
}
} else {
context = SkinRestorer.getSkinStorage().getSkin(profile.id()).toProviderContext();
context = SkinRestorer.getSkinStorage().getSkin(profile.getId()).toProviderContext();
}
if (context == null)
return 0;
return SkinCommand.setSubcommand(src, Collections.singleton(new NameAndId(profile)), context, save, false);
return SkinCommand.setSubcommand(src, Collections.singleton(profile), context, save, false);
}
private static int resetSubcommand(
CommandSourceStack src,
Collection<NameAndId> targets,
Collection<GameProfile> targets,
boolean setByOperator
) {
var updatedPlayers = new HashSet<ServerPlayer>();
for (var nameAndId : targets) {
for (var profile : targets) {
SkinValue skin = null;
if (SkinRestorer.getSkinStorage().hasSavedSkin(nameAndId.id()))
skin = SkinRestorer.getSkinStorage().getSkin(nameAndId.id()).replaceValueWithOriginal();
if (SkinRestorer.getSkinStorage().hasSavedSkin(profile.getId()))
skin = SkinRestorer.getSkinStorage().getSkin(profile.getId()).replaceValueWithOriginal();
if (skin == null)
continue;
var player = src.getServer().getPlayerList().getPlayer(nameAndId.id());
if (player == null)
continue;
var updatedPlayer = SkinRestorer.applySkin(src.getServer(), Collections.singleton(player), skin, false);
SkinRestorer.getSkinStorage().deleteSkin(nameAndId.id());
var updatedPlayer = SkinRestorer.applySkin(src.getServer(), Collections.singleton(profile), skin, false);
SkinRestorer.getSkinStorage().deleteSkin(profile.getId());
updatedPlayers.addAll(updatedPlayer);
}
@@ -124,24 +118,19 @@ public final class SkinCommand {
if (src.getPlayer() == null)
return 0;
return resetSubcommand(src, Collections.singleton(src.getPlayer().nameAndId()), false);
return resetSubcommand(src, Collections.singleton(src.getPlayer().getGameProfile()), false);
}
private static int setSubcommand(
CommandSourceStack src,
Collection<NameAndId> targets,
Collection<GameProfile> targets,
SkinProviderContext context,
boolean save,
boolean setByOperator
) {
src.sendSystemMessage(Translation.translatableWithFallback(Translation.COMMAND_SKIN_LOADING_KEY));
var profileTargets = targets.stream()
.map(nameAndId -> src.getServer().getPlayerList().getPlayer(nameAndId.id()))
.filter(Objects::nonNull)
.toList();
SkinRestorer.setSkinAsync(src.getServer(), profileTargets, context, save).thenAccept(result -> {
SkinRestorer.setSkinAsync(src.getServer(), targets, context, save).thenAccept(result -> {
if (result.isError()) {
src.sendFailure(Translation.translatableWithFallback(
Translation.COMMAND_SKIN_FAILED_KEY,
@@ -160,7 +149,7 @@ public final class SkinCommand {
private static int setSubcommand(
CommandSourceStack src,
Collection<NameAndId> targets,
Collection<GameProfile> targets,
SkinProviderContext context,
boolean setByOperator
) {
@@ -174,7 +163,7 @@ public final class SkinCommand {
if (src.getPlayer() == null)
return 0;
return setSubcommand(src, Collections.singleton(src.getPlayer().nameAndId()), context, false);
return setSubcommand(src, Collections.singleton(src.getPlayer().getGameProfile()), context, false);
}
private static int configReloadSubcommand(CommandContext<CommandSourceStack> context) {
@@ -267,10 +256,10 @@ public final class SkinCommand {
}
private static RequiredArgumentBuilder<CommandSourceStack, GameProfileArgument.Result> makeTargetsArgument(
BiFunction<CommandContext<CommandSourceStack>, Collection<NameAndId>, Integer> consumer
BiFunction<CommandContext<CommandSourceStack>, Collection<GameProfile>, Integer> consumer
) {
return argument("targets", GameProfileArgument.gameProfile())
.requires(source -> Commands.LEVEL_GAMEMASTERS.check(source.permissions()))
.requires(source -> source.hasPermission(2))
.executes(context -> consumer.apply(context, GameProfileArgument.getGameProfiles(context, "targets")));
}
}

View File

@@ -5,7 +5,7 @@ import net.lionarius.skinrestorer.platform.Services;
import net.lionarius.skinrestorer.skin.SkinValue;
import net.lionarius.skinrestorer.skin.provider.SkinShuffleSkinProvider;
import net.lionarius.skinrestorer.util.PlayerUtils;
import net.minecraft.resources.Identifier;
import net.minecraft.resources.ResourceLocation;
import net.minecraft.server.MinecraftServer;
import net.minecraft.server.level.ServerPlayer;
@@ -23,8 +23,8 @@ public class SkinShuffleCompatibility {
return SkinShuffleCompatibility.SHOULD_APPLY;
}
public static Identifier resourceLocation(String name) {
return Identifier.fromNamespaceAndPath(SkinShuffleCompatibility.MOD_ID, name);
public static ResourceLocation resourceLocation(String name) {
return ResourceLocation.fromNamespaceAndPath(SkinShuffleCompatibility.MOD_ID, name);
}
public static void onPlayerJoin(ServerPlayer player) {
@@ -43,7 +43,7 @@ public class SkinShuffleCompatibility {
server.execute(() -> {
SkinRestorer.applySkin(
server,
Collections.singleton(player),
Collections.singleton(player.getGameProfile()),
new SkinValue(SkinShuffleSkinProvider.PROVIDER_NAME, null, null, property),
!server.usesAuthentication()
);

View File

@@ -22,6 +22,8 @@ public final class Config implements GsonPostProcessable {
private boolean fetchSkinOnFirstJoin = true;
private boolean forceFirstJoinSkinFetch = false;
private FirstJoinSkinProvider firstJoinSkinProvider = FirstJoinSkinProvider.MOJANG;
private String proxy = "";
@@ -47,6 +49,10 @@ public final class Config implements GsonPostProcessable {
return this.fetchSkinOnFirstJoin;
}
public boolean forceFirstJoinSkinFetch() {
return this.forceFirstJoinSkinFetch;
}
public FirstJoinSkinProvider firstJoinSkinProvider() {
return this.firstJoinSkinProvider;
}

View File

@@ -1,6 +1,7 @@
package net.lionarius.skinrestorer.config;
import com.google.gson.annotations.SerializedName;
import net.lionarius.skinrestorer.skin.provider.CollectionSkinProvider;
import net.lionarius.skinrestorer.skin.provider.ElyBySkinProvider;
import net.lionarius.skinrestorer.skin.provider.MojangSkinProvider;
@@ -8,7 +9,9 @@ public enum FirstJoinSkinProvider {
@SerializedName(value = "MOJANG", alternate = {"mojang"})
MOJANG(MojangSkinProvider.PROVIDER_NAME),
@SerializedName(value = "ELY.BY", alternate = {"ely.by", "ELY_BY", "ely_by"})
ELY_BY(ElyBySkinProvider.PROVIDER_NAME);
ELY_BY(ElyBySkinProvider.PROVIDER_NAME),
@SerializedName(value = "COLLECTION", alternate = {"collection"})
COLLECTION(CollectionSkinProvider.PROVIDER_NAME);
private final String name;

View File

@@ -8,8 +8,13 @@ public abstract class BuiltInProviderConfig implements GsonPostProcessable {
protected String name;
protected CacheConfig cache;
public BuiltInProviderConfig(String name, CacheConfig cache) {
this.enabled = true;
this(name, cache, true);
}
public BuiltInProviderConfig(String name, CacheConfig cache, boolean enabled) {
this.enabled = enabled;
this.name = name;
this.cache = cache;
}

View File

@@ -1,23 +1,27 @@
package net.lionarius.skinrestorer.config.provider;
import net.lionarius.skinrestorer.SkinRestorer;
import net.lionarius.skinrestorer.config.provider.collection.CollectionProviderConfig;
import net.lionarius.skinrestorer.util.gson.GsonPostProcessable;
public final class ProvidersConfig implements GsonPostProcessable {
public static final ProvidersConfig DEFAULT = new ProvidersConfig(
new MojangProviderConfig(),
new ElyByProviderConfig(),
new MineskinProviderConfig()
new MineskinProviderConfig(),
new CollectionProviderConfig()
);
private MojangProviderConfig mojang;
private ElyByProviderConfig ely_by;
private MineskinProviderConfig mineskin;
private CollectionProviderConfig collection;
public ProvidersConfig(MojangProviderConfig mojang, ElyByProviderConfig ely_by, MineskinProviderConfig mineskin) {
public ProvidersConfig(MojangProviderConfig mojang, ElyByProviderConfig ely_by, MineskinProviderConfig mineskin, CollectionProviderConfig collection) {
this.mojang = mojang;
this.ely_by = ely_by;
this.mineskin = mineskin;
this.collection = collection;
}
public MojangProviderConfig mojang() {
@@ -32,6 +36,10 @@ public final class ProvidersConfig implements GsonPostProcessable {
return this.mineskin;
}
public CollectionProviderConfig collection() {
return this.collection;
}
@Override
public void gsonPostProcess() {
if (this.mojang == null) {
@@ -48,5 +56,10 @@ public final class ProvidersConfig implements GsonPostProcessable {
SkinRestorer.LOGGER.warn("Mineskin provider config is null, using default");
this.mineskin = ProvidersConfig.DEFAULT.mineskin();
}
if (this.collection == null) {
SkinRestorer.LOGGER.warn("Collection provider config is null, using default");
this.collection = ProvidersConfig.DEFAULT.collection();
}
}
}

View File

@@ -0,0 +1,32 @@
package net.lionarius.skinrestorer.config.provider.collection;
import com.google.gson.annotations.JsonAdapter;
import net.lionarius.skinrestorer.config.provider.BuiltInProviderConfig;
import net.lionarius.skinrestorer.config.provider.CacheConfig;
import net.lionarius.skinrestorer.skin.provider.CollectionSkinProvider;
import net.lionarius.skinrestorer.util.gson.GsonPostProcessable;
import java.util.ArrayList;
import java.util.List;
public final class CollectionProviderConfig extends BuiltInProviderConfig implements GsonPostProcessable {
private static final CacheConfig DEFAULT_CACHE_VALUE = new CacheConfig(true, 604800);
@JsonAdapter(CollectionSkinSourceListDeserializer.class)
private List<CollectionSkinSource> sources = new ArrayList<>();
public CollectionProviderConfig() {
super(CollectionSkinProvider.PROVIDER_NAME, DEFAULT_CACHE_VALUE, false);
}
public List<CollectionSkinSource> sources() {
return this.sources;
}
@Override
public void gsonPostProcess() {
if (this.sources == null) {
this.sources = new ArrayList<>();
}
}
}

View File

@@ -0,0 +1,60 @@
package net.lionarius.skinrestorer.config.provider.collection;
import net.lionarius.skinrestorer.SkinRestorer;
import net.lionarius.skinrestorer.skin.SkinVariant;
import net.lionarius.skinrestorer.util.gson.GsonPostProcessable;
import org.jetbrains.annotations.Nullable;
import java.net.URI;
import java.nio.file.Files;
import java.nio.file.Path;
public final class CollectionSkinFile implements CollectionSkinSource, GsonPostProcessable {
private String path = "";
private SkinVariant variant = SkinVariant.CLASSIC;
@Override
public @Nullable URI uri() {
if (this.path.isEmpty())
return null;
try {
var filePath = SkinRestorer.getConfigDir().resolve(this.path);
if (!Files.exists(filePath)) {
SkinRestorer.LOGGER.warn("Skin file does not exist: {}", this.path);
return null;
}
if (!Files.isRegularFile(filePath)) {
SkinRestorer.LOGGER.warn("Skin path is not a file: {}", this.path);
return null;
}
if (!this.path.toLowerCase().endsWith(".png")) {
SkinRestorer.LOGGER.warn("Skin file is not a PNG file: {}", this.path);
return null;
}
return filePath.toUri();
} catch (Exception e) {
SkinRestorer.LOGGER.warn("Invalid file path: {}", this.path, e);
return null;
}
}
@Override
public SkinVariant variant() {
return this.variant;
}
@Override
public void gsonPostProcess() {
if (this.path == null) {
this.path = "";
}
if (this.variant == null) {
this.variant = SkinVariant.CLASSIC;
}
}
}

View File

@@ -0,0 +1,12 @@
package net.lionarius.skinrestorer.config.provider.collection;
import net.lionarius.skinrestorer.skin.SkinVariant;
import org.jetbrains.annotations.Nullable;
import java.net.URI;
public interface CollectionSkinSource {
@Nullable URI uri();
SkinVariant variant();
}

View File

@@ -0,0 +1,35 @@
package net.lionarius.skinrestorer.config.provider.collection;
import com.google.gson.*;
import java.lang.reflect.Type;
import java.util.ArrayList;
import java.util.List;
public class CollectionSkinSourceListDeserializer implements JsonSerializer<List<CollectionSkinSource>>, JsonDeserializer<List<CollectionSkinSource>> {
@Override
public JsonElement serialize(List<CollectionSkinSource> src, Type typeOfT, JsonSerializationContext context) {
return context.serialize(src, List.class);
}
@Override
public List<CollectionSkinSource> deserialize(JsonElement json, Type typeOfT, JsonDeserializationContext context)
throws JsonParseException {
List<CollectionSkinSource> sources = new ArrayList<>();
if (json.isJsonArray()) {
for (JsonElement element : json.getAsJsonArray()) {
if (element.isJsonObject()) {
JsonObject obj = element.getAsJsonObject();
if (obj.has("url")) {
sources.add(context.deserialize(obj, CollectionSkinUrl.class));
} else if (obj.has("path")) {
sources.add(context.deserialize(obj, CollectionSkinFile.class));
}
}
}
}
return sources;
}
}

View File

@@ -0,0 +1,41 @@
package net.lionarius.skinrestorer.config.provider.collection;
import net.lionarius.skinrestorer.SkinRestorer;
import net.lionarius.skinrestorer.skin.SkinVariant;
import net.lionarius.skinrestorer.util.gson.GsonPostProcessable;
import org.jetbrains.annotations.Nullable;
import java.net.URI;
public final class CollectionSkinUrl implements CollectionSkinSource, GsonPostProcessable {
private String url = "";
private SkinVariant variant = SkinVariant.CLASSIC;
@Override
public @Nullable URI uri() {
try {
if (this.url.isEmpty())
return null;
return new URI(this.url);
} catch (Exception e) {
SkinRestorer.LOGGER.warn("Invalid URI: {}", this.url, e);
return null;
}
}
@Override
public SkinVariant variant() {
return this.variant;
}
@Override
public void gsonPostProcess() {
if (this.url == null) {
this.url = "";
}
if (this.variant == null) {
this.variant = SkinVariant.CLASSIC;
}
}
}

View File

@@ -0,0 +1,14 @@
package net.lionarius.skinrestorer.mixin;
import net.minecraft.server.players.GameProfileCache;
import org.spongepowered.asm.mixin.Mixin;
import org.spongepowered.asm.mixin.gen.Accessor;
import java.util.Map;
@Mixin(GameProfileCache.class)
public interface GameProfileCacheAccessor {
@Accessor
Map<String, GameProfileCache.GameProfileInfo> getProfilesByName();
}

View File

@@ -1,15 +0,0 @@
package net.lionarius.skinrestorer.mixin;
import com.mojang.authlib.GameProfile;
import net.minecraft.world.entity.player.Player;
import org.spongepowered.asm.mixin.Mixin;
import org.spongepowered.asm.mixin.Mutable;
import org.spongepowered.asm.mixin.gen.Accessor;
@Mixin(Player.class)
public interface PlayerAccessor {
@Accessor("gameProfile")
@Mutable
void setGameProfile(GameProfile properties);
}

View File

@@ -57,6 +57,6 @@ public abstract class PlayerListMixin {
@Unique
private static void skinrestorer$tryApplySkin(MinecraftServer server, ServerPlayer player) {
if (SkinRestorer.getSkinStorage().hasSavedSkin(player.getUUID()))
SkinRestorer.applySkin(server, Collections.singleton(player), SkinRestorer.getSkinStorage().getSkin(player.getUUID()));
SkinRestorer.applySkin(server, Collections.singleton(player.getGameProfile()), SkinRestorer.getSkinStorage().getSkin(player.getUUID()));
}
}

View File

@@ -2,6 +2,7 @@ package net.lionarius.skinrestorer.mixin;
import com.mojang.authlib.GameProfile;
import net.lionarius.skinrestorer.SkinRestorer;
import net.lionarius.skinrestorer.config.FirstJoinSkinProvider;
import net.lionarius.skinrestorer.skin.SkinValue;
import net.lionarius.skinrestorer.skin.provider.SkinProviderContext;
import net.lionarius.skinrestorer.util.PlayerUtils;
@@ -27,7 +28,7 @@ public abstract class ServerLoginPacketListenerImplMixin {
private CompletableFuture<Void> skinrestorer$pendingSkin;
@Inject(method = "verifyLoginAndFinishConnectionSetup", at = @At(value = "INVOKE",
target = "Lnet/minecraft/server/players/PlayerList;canPlayerLogin(Ljava/net/SocketAddress;Lnet/minecraft/server/players/NameAndId;)Lnet/minecraft/network/chat/Component;"),
target = "Lnet/minecraft/server/players/PlayerList;canPlayerLogin(Ljava/net/SocketAddress;Lcom/mojang/authlib/GameProfile;)Lnet/minecraft/network/chat/Component;"),
cancellable = true)
public void waitForSkin(CallbackInfo ci) {
if (skinrestorer$pendingSkin == null) {
@@ -37,14 +38,14 @@ public abstract class ServerLoginPacketListenerImplMixin {
assert profile != null;
var originalSkin = PlayerUtils.getPlayerSkin(profile);
if (SkinRestorer.getSkinStorage().hasSavedSkin(profile.id())) {
if (SkinRestorer.getSkinStorage().hasSavedSkin(profile.getId())) {
if (originalSkin != null) { // update to the latest official skin
var value = SkinRestorer.getSkinStorage().getSkin(profile.id());
SkinRestorer.getSkinStorage().setSkin(profile.id(), value.setOriginalValue(originalSkin));
var value = SkinRestorer.getSkinStorage().getSkin(profile.getId());
SkinRestorer.getSkinStorage().setSkin(profile.getId(), value.setOriginalValue(originalSkin));
}
if (SkinRestorer.getConfig().refreshSkinOnJoin()) {
var currentSkin = SkinRestorer.getSkinStorage().getSkin(profile.id());
var currentSkin = SkinRestorer.getSkinStorage().getSkin(profile.getId());
var context = currentSkin.toProviderContext();
skinrestorer$fetchSkin(profile, context);
@@ -53,10 +54,16 @@ public abstract class ServerLoginPacketListenerImplMixin {
return null;
}
if (originalSkin == null && SkinRestorer.getConfig().fetchSkinOnFirstJoin()) {
var config = SkinRestorer.getConfig();
var provider = config.firstJoinSkinProvider();
var shouldFetch = (originalSkin == null && config.fetchSkinOnFirstJoin()) ||
(originalSkin != null && config.forceFirstJoinSkinFetch() && provider != FirstJoinSkinProvider.MOJANG);
if (shouldFetch) {
var context = new SkinProviderContext(
SkinRestorer.getConfig().firstJoinSkinProvider().getName(),
profile.name(),
provider.getName(),
profile.getName(),
null
);
skinrestorer$fetchSkin(profile, context);
@@ -72,7 +79,7 @@ public abstract class ServerLoginPacketListenerImplMixin {
@Unique
private static void skinrestorer$fetchSkin(GameProfile profile, SkinProviderContext context) {
SkinRestorer.LOGGER.debug("Fetching {}'s skin", profile.name());
SkinRestorer.LOGGER.debug("Fetching {}'s skin", profile.getName());
var result = SkinRestorer.getProvider(context.name()).map(
provider -> provider.fetchSkin(context.argument(), context.variant())
@@ -80,7 +87,7 @@ public abstract class ServerLoginPacketListenerImplMixin {
if (!result.isError()) {
var value = SkinValue.fromProviderContextWithValue(context, result.getSuccessValue().orElse(null));
SkinRestorer.getSkinStorage().setSkin(profile.id(), value);
SkinRestorer.getSkinStorage().setSkin(profile.getId(), value);
} else {
SkinRestorer.LOGGER.warn("Failed to fetch skin '{}:{}'", context.name(), context.argument(), result.getErrorValue());
}

View File

@@ -0,0 +1,69 @@
package net.lionarius.skinrestorer.mixin;
import com.mojang.authlib.GameProfile;
import net.lionarius.skinrestorer.SkinRestorer;
import net.lionarius.skinrestorer.util.PlayerUtils;
import net.minecraft.Util;
import net.minecraft.server.Services;
import net.minecraft.world.level.block.entity.SkullBlockEntity;
import org.spongepowered.asm.mixin.Mixin;
import org.spongepowered.asm.mixin.Unique;
import org.spongepowered.asm.mixin.injection.At;
import org.spongepowered.asm.mixin.injection.Inject;
import org.spongepowered.asm.mixin.injection.callback.CallbackInfoReturnable;
import java.util.Locale;
import java.util.Optional;
import java.util.UUID;
import java.util.concurrent.CompletableFuture;
import java.util.function.BooleanSupplier;
@Mixin(SkullBlockEntity.class)
public abstract class SkullBlockEntityMixin {
@Inject(method = "fetchProfileByName", at = @At("HEAD"),
cancellable = true)
private static void fetchProfileByName(String name, Services services, CallbackInfoReturnable<CompletableFuture<Optional<GameProfile>>> cir) {
if (name == null)
return;
var profileOpt = Optional.<GameProfile>empty();
var gameProfileInfo = ((GameProfileCacheAccessor) services.profileCache()).getProfilesByName().get(name.toLowerCase(Locale.ROOT));
if (gameProfileInfo != null)
profileOpt = Optional.of(gameProfileInfo.getProfile());
skinrestorer$replaceSkin(profileOpt, cir);
}
@Inject(method = "fetchProfileById", at = @At("HEAD"),
cancellable = true)
private static void fetchProfileById(UUID id, Services services, BooleanSupplier cacheUninitialized, CallbackInfoReturnable<CompletableFuture<Optional<GameProfile>>> cir) {
if (id == null)
return;
var profileOpt = services.profileCache().get(id);
skinrestorer$replaceSkin(profileOpt, cir);
}
@Unique
private static void skinrestorer$replaceSkin(Optional<GameProfile> profileOpt, CallbackInfoReturnable<CompletableFuture<Optional<GameProfile>>> cir) {
if (SkinRestorer.getMinecraftServer() == null)
return;
if (profileOpt.isEmpty())
return;
var profile = PlayerUtils.cloneGameProfile(profileOpt.get());
if (SkinRestorer.getSkinStorage().hasSavedSkin(profile.getId())) {
cir.setReturnValue(CompletableFuture.supplyAsync(() -> {
var skin = SkinRestorer.getSkinStorage().getSkin(profile.getId(), false);
PlayerUtils.applyRestoredSkin(profile, skin.value());
return Optional.of(profile);
}, Util.backgroundExecutor().forName("getProfile")));
}
}
}

View File

@@ -1,8 +1,11 @@
package net.lionarius.skinrestorer.skin;
import com.google.gson.annotations.SerializedName;
public enum SkinVariant {
@SerializedName(value = "classic", alternate = {"CLASSIC"})
CLASSIC("classic"),
@SerializedName(value = "slim", alternate = {"SLIM"})
SLIM("slim");
private final String name;

View File

@@ -0,0 +1,91 @@
package net.lionarius.skinrestorer.skin.provider;
import com.google.common.cache.CacheBuilder;
import com.google.common.cache.CacheLoader;
import com.google.common.cache.LoadingCache;
import com.google.common.util.concurrent.UncheckedExecutionException;
import com.mojang.authlib.properties.Property;
import it.unimi.dsi.fastutil.Pair;
import net.lionarius.skinrestorer.SkinRestorer;
import net.lionarius.skinrestorer.config.provider.collection.CollectionSkinSource;
import net.lionarius.skinrestorer.skin.SkinVariant;
import net.lionarius.skinrestorer.util.Result;
import org.jetbrains.annotations.NotNull;
import java.net.URI;
import java.util.ArrayList;
import java.util.List;
import java.util.Optional;
import java.util.concurrent.TimeUnit;
public final class CollectionSkinProvider implements SkinProvider {
public static final String PROVIDER_NAME = "collection";
private static LoadingCache<Integer, Optional<Property>> SKIN_CACHE;
private static List<Pair<URI, SkinVariant>> COLLECTION_SKINS;
public static void reload() {
COLLECTION_SKINS = loadCollectionSkins();
createCache();
}
private static List<Pair<URI, SkinVariant>> loadCollectionSkins() {
List<Pair<URI, SkinVariant>> skins = new ArrayList<>();
var config = SkinRestorer.getConfig().providersConfig().collection();
for (CollectionSkinSource source : config.sources()) {
var uri = source.uri();
if (uri != null) {
skins.add(Pair.of(uri, source.variant()));
}
}
return skins;
}
private static void createCache() {
var config = SkinRestorer.getConfig().providersConfig().collection();
var time = config.cache().enabled() ? config.cache().duration() : 0;
SKIN_CACHE = CacheBuilder.newBuilder()
.expireAfterWrite(time, TimeUnit.SECONDS)
.build(new CacheLoader<>() {
@Override
public @NotNull Optional<Property> load(@NotNull Integer key) throws Exception {
var skinEntry = COLLECTION_SKINS.get(key);
return MineskinSkinProvider.loadSkin(skinEntry.first(), skinEntry.second());
}
});
}
@Override
public String getArgumentName() {
return "seed";
}
@Override
public boolean hasVariantSupport() {
return false;
}
@Override
public Result<Optional<Property>, Exception> fetchSkin(String argument, SkinVariant variant) {
if (COLLECTION_SKINS.isEmpty()) {
return Result.error(new IllegalStateException("No collection skins configured"));
}
var skinIndex = Math.abs(argument.hashCode()) % COLLECTION_SKINS.size();
try {
return Result.success(SKIN_CACHE.get(skinIndex));
} catch (UncheckedExecutionException e) {
return Result.error((Exception) e.getCause());
} catch (Exception e) {
return Result.error(e);
}
}
}

View File

@@ -106,6 +106,6 @@ public final class ElyBySkinProvider implements SkinProvider {
if (response.statusCode() != 200)
throw new IllegalArgumentException("no profile with name " + username);
return JsonUtils.fromJson(response.body(), MinecraftProfilePropertiesResponse.class).profile();
return JsonUtils.fromJson(response.body(), MinecraftProfilePropertiesResponse.class).toProfile();
}
}

View File

@@ -22,6 +22,8 @@ import org.mineskin.response.QueueResponse;
import java.net.InetSocketAddress;
import java.net.URI;
import java.nio.file.Files;
import java.nio.file.Path;
import java.time.Duration;
import java.util.Optional;
import java.util.concurrent.TimeUnit;
@@ -94,13 +96,18 @@ public final class MineskinSkinProvider implements SkinProvider {
}
}
private static Optional<Property> loadSkin(URI uri, SkinVariant variant) throws Exception {
static Optional<Property> loadSkin(URI uri, SkinVariant variant) throws Exception {
var mineskinVariant = switch (variant) {
case CLASSIC -> Variant.CLASSIC;
case SLIM -> Variant.SLIM;
};
var request = GenerateRequest.url(uri)
var request = "file".equals(uri.getScheme())
? GenerateRequest.upload(Files.newInputStream(Path.of(uri)))
.variant(mineskinVariant)
.name("skinrestorer-skin")
.visibility(Visibility.UNLISTED)
: GenerateRequest.url(uri)
.variant(mineskinVariant)
.name("skinrestorer-skin")
.visibility(Visibility.UNLISTED);

View File

@@ -8,7 +8,6 @@ import com.mojang.authlib.*;
import com.mojang.authlib.properties.Property;
import com.mojang.authlib.yggdrasil.YggdrasilEnvironment;
import com.mojang.authlib.yggdrasil.response.MinecraftProfilePropertiesResponse;
import com.mojang.authlib.yggdrasil.response.NameAndId;
import com.mojang.util.UndashedUuid;
import net.lionarius.skinrestorer.SkinRestorer;
import net.lionarius.skinrestorer.exception.TransparentException;
@@ -17,7 +16,7 @@ import net.lionarius.skinrestorer.util.JsonUtils;
import net.lionarius.skinrestorer.util.PlayerUtils;
import net.lionarius.skinrestorer.util.Result;
import net.lionarius.skinrestorer.util.WebUtils;
import net.minecraft.server.players.CachedUserNameToIdResolver;
import net.minecraft.server.players.GameProfileCache;
import net.minecraft.util.StringUtil;
import org.jetbrains.annotations.NotNull;
@@ -32,11 +31,13 @@ import java.util.concurrent.TimeUnit;
public final class MojangSkinProvider implements SkinProvider {
public static final String PROVIDER_NAME = "mojang";
public static final String PROFILE_CACHE_FILENAME = "mojang_profile_cache.json";
private static final Environment ENVIRONMENT;
private static final URI SERVICES_SERVER_URI;
private static final URI SESSION_SERVER_URI;
private static final CachedUserNameToIdResolver PROFILE_CACHE;
public static final String PROFILE_CACHE_FILENAME = "mojang_profile_cache.json";
private static final GameProfileCache PROFILE_CACHE;
private static LoadingCache<UUID, Optional<Property>> SKIN_CACHE;
@@ -50,13 +51,13 @@ public final class MojangSkinProvider implements SkinProvider {
throw new IllegalArgumentException(e);
}
PROFILE_CACHE = new CachedUserNameToIdResolver(new GameProfileRepository() {
PROFILE_CACHE = new GameProfileCache(new GameProfileRepository() {
@Override
public void findProfilesByNames(String[] names, ProfileLookupCallback callback) {
for (var name : names) {
try {
var profile = MojangSkinProvider.getProfile(name);
callback.onProfileLookupSucceeded(profile.name(), profile.id());
callback.onProfileLookupSucceeded(profile);
} catch (IOException e) {
throw new TransparentException(e);
}
@@ -64,10 +65,10 @@ public final class MojangSkinProvider implements SkinProvider {
}
@Override
public Optional<NameAndId> findProfileByName(String name) {
public Optional<GameProfile> findProfileByName(String name) {
try {
var profile = MojangSkinProvider.getProfile(name);
return Optional.of(new NameAndId(profile.id(), profile.name()));
return Optional.of(profile);
} catch (IOException e) {
throw new TransparentException(e);
}
@@ -94,7 +95,7 @@ public final class MojangSkinProvider implements SkinProvider {
}
public static SkinProviderContext skinProviderContextFromProfile(GameProfile gameProfile) {
return new SkinProviderContext(MojangSkinProvider.PROVIDER_NAME, gameProfile.name(), null);
return new SkinProviderContext(MojangSkinProvider.PROVIDER_NAME, gameProfile.getName(), null);
}
@Override
@@ -117,7 +118,7 @@ public final class MojangSkinProvider implements SkinProvider {
if (cachedProfile.isEmpty())
throw new IllegalArgumentException("no profile found for " + username);
return Result.success(SKIN_CACHE.get(cachedProfile.get().id()));
return Result.success(SKIN_CACHE.get(cachedProfile.get().getId()));
} catch (UncheckedExecutionException e) {
return Result.error((Exception) e.getCause());
} catch (Exception e) {
@@ -132,7 +133,7 @@ public final class MojangSkinProvider implements SkinProvider {
return Optional.ofNullable(textures);
}
private static NameAndId getProfile(final String name) throws IOException {
private static GameProfile getProfile(final String name) throws IOException {
var request = HttpRequest.newBuilder()
.uri(MojangSkinProvider.SERVICES_SERVER_URI
.resolve("/minecraft/profile/lookup/name/")
@@ -147,7 +148,7 @@ public final class MojangSkinProvider implements SkinProvider {
if (response.statusCode() != 200)
throw new IllegalArgumentException("no profile with name " + name);
return JsonUtils.fromJson(response.body(), NameAndId.class);
return JsonUtils.fromJson(response.body(), GameProfile.class);
}
private static GameProfile getProfileWithProperties(UUID uuid) throws IOException {
@@ -165,6 +166,6 @@ public final class MojangSkinProvider implements SkinProvider {
if (response.statusCode() != 200)
throw new IllegalArgumentException("no profile with uuid " + uuid);
return JsonUtils.fromJson(response.body(), MinecraftProfilePropertiesResponse.class).profile();
return JsonUtils.fromJson(response.body(), MinecraftProfilePropertiesResponse.class).toProfile();
}
}

View File

@@ -13,6 +13,7 @@ public interface SkinProvider {
MojangSkinProvider MOJANG = new MojangSkinProvider();
ElyBySkinProvider ELY_BY = new ElyBySkinProvider();
MineskinSkinProvider MINESKIN = new MineskinSkinProvider();
CollectionSkinProvider COLLECTION = new CollectionSkinProvider();
SkinShuffleSkinProvider SKIN_SHUFFLE = new SkinShuffleSkinProvider();
Set<String> BUILTIN_PROVIDER_NAMES = ImmutableSet.of(
@@ -20,6 +21,7 @@ public interface SkinProvider {
MojangSkinProvider.PROVIDER_NAME,
ElyBySkinProvider.PROVIDER_NAME,
MineskinSkinProvider.PROVIDER_NAME,
CollectionSkinProvider.PROVIDER_NAME,
SkinShuffleSkinProvider.PROVIDER_NAME
);

View File

@@ -4,6 +4,7 @@ import com.google.gson.Gson;
import com.google.gson.GsonBuilder;
import com.google.gson.JsonElement;
import com.google.gson.JsonObject;
import com.mojang.authlib.GameProfile;
import com.mojang.authlib.properties.Property;
import com.mojang.authlib.properties.PropertyMap;
import com.mojang.util.UUIDTypeAdapter;
@@ -21,6 +22,7 @@ public final class JsonUtils {
.registerTypeAdapterFactory(new PostProcessingEnabler())
.registerTypeAdapter(UUID.class, new UUIDTypeAdapter())
.registerTypeAdapter(PropertyMap.class, new PropertyMap.Serializer())
.registerTypeAdapter(GameProfile.class, new GameProfile.Serializer())
.setPrettyPrinting()
.create();

View File

@@ -1,11 +1,9 @@
package net.lionarius.skinrestorer.util;
import com.google.common.collect.Iterables;
import com.google.common.collect.LinkedHashMultimap;
import com.google.gson.JsonObject;
import com.mojang.authlib.GameProfile;
import com.mojang.authlib.properties.Property;
import com.mojang.authlib.properties.PropertyMap;
import com.mojang.authlib.yggdrasil.response.MinecraftProfilePropertiesResponse;
import net.lionarius.skinrestorer.mixin.ChunkMapAccessor;
import net.minecraft.network.chat.Component;
@@ -98,26 +96,21 @@ public final class PlayerUtils {
}
public static GameProfile cloneGameProfile(GameProfile profile) {
var newProfile = new GameProfile(profile.id(), profile.name());
newProfile.properties().putAll(profile.properties());
var newProfile = new GameProfile(profile.getId(), profile.getName());
newProfile.getProperties().putAll(profile.getProperties());
return newProfile;
}
public static Property getPlayerSkin(GameProfile profile) {
return Iterables.getFirst(profile.properties().get(TEXTURES_KEY), null);
return Iterables.getFirst(profile.getProperties().get(TEXTURES_KEY), null);
}
public static GameProfile applyRestoredSkin(GameProfile profile, Property skin) {
var propertiesMap = profile.properties();
public static void applyRestoredSkin(GameProfile profile, Property skin) {
profile.getProperties().removeAll(TEXTURES_KEY);
var newProperties = LinkedHashMultimap.create(propertiesMap);
newProperties.removeAll(TEXTURES_KEY);
if (skin != null) {
newProperties.put(TEXTURES_KEY, skin);
}
return new GameProfile(profile.id(), profile.name(), new PropertyMap(newProperties));
if (skin != null)
profile.getProperties().put(TEXTURES_KEY, skin);
}
public static boolean areSkinPropertiesEquals(Property x, Property y) {
@@ -141,7 +134,7 @@ public final class PlayerUtils {
public static GameProfile toProfile(MinecraftProfilePropertiesResponse response) {
final GameProfile profile = new GameProfile(response.id(), response.name());
profile.properties().putAll(response.properties());
profile.getProperties().putAll(response.properties());
return profile;
}
}

View File

@@ -0,0 +1 @@
public net.minecraft.server.players.GameProfileCache$GameProfileInfo

View File

@@ -1 +1,3 @@
accessWidener v2 named
accessible class net/minecraft/server/players/GameProfileCache$GameProfileInfo

View File

@@ -10,7 +10,8 @@
"PlayerListMixin",
"ServerLoginPacketListenerImplMixin",
"TrackedEntityAccessorInvoker",
"PlayerAccessor"
"SkullBlockEntityMixin",
"GameProfileCacheAccessor"
],
"injectors": {
"defaultRequire": 1

View File

@@ -10,7 +10,7 @@ dependencies {
minecraft "com.mojang:minecraft:${minecraft_version}"
mappings loom.layered {
officialMojangMappings()
// parchment("org.parchmentmc.data:parchment-${parchment_minecraft}:${parchment_version}@zip")
parchment("org.parchmentmc.data:parchment-${parchment_minecraft}:${parchment_version}@zip")
}
modImplementation "net.fabricmc:fabric-loader:${fabric_loader_version}"

View File

@@ -1,6 +1,6 @@
# Fabric, see https://fabricmc.net/develop/ for new versions
fabric_loader_version=0.15.0
fabric_api_version=0.139.4+1.21.11
fabric_api_version=0.127.0+1.21.6
optional_dependencies=fabric-api
additional_modloaders=quilt

View File

@@ -32,6 +32,6 @@ public final class SkinShuffleCompatibility {
}
private static void handleSkinRefreshPacket(SkinShuffleSkinRefreshPayload payload, ServerPlayNetworking.Context context) {
net.lionarius.skinrestorer.compat.skinshuffle.SkinShuffleCompatibility.handleSkinRefresh(SkinRestorer.getMinecraftServer(), context.player(), payload);
net.lionarius.skinrestorer.compat.skinshuffle.SkinShuffleCompatibility.handleSkinRefresh(context.player().getServer(), context.player(), payload);
}
}

View File

@@ -11,7 +11,7 @@ import org.spongepowered.asm.mixin.injection.callback.CallbackInfo;
public abstract class MinecraftServerMixin {
@Inject(method = "runServer",
at = @At(value = "INVOKE", target = "Lnet/minecraft/util/Util;getNanos()J", ordinal = 0))
at = @At(value = "INVOKE", target = "Lnet/minecraft/Util;getNanos()J", ordinal = 0))
private void onServerStarted(CallbackInfo ci) {
SkinRestorer.Events.onServerStarted((MinecraftServer) (Object) this);
}

View File

@@ -24,8 +24,7 @@ tasks.named('jarJar') {
jar.finalizedBy('jarJar')
minecraft {
mappings channel: 'official', version: minecraft_version
// mappings channel: 'parchment', version: "${parchment_minecraft}-${parchment_version}-${minecraft_version}"
mappings channel: 'parchment', version: "${parchment_minecraft}-${parchment_version}-${minecraft_version}"
copyIdeResources = true //Calls processResources when in dev

View File

@@ -1,5 +1,5 @@
# Forge, see https://files.minecraftforge.net/net/minecraftforge/forge/ for new versions
forge_version=61.0.0
forge_loader_version_range=[61,)
forge_version=56.0.0
forge_loader_version_range=[56,)
# Forge sometimes skips minor minecraft versions (like 1.20.5)
forge_minecraft_version=1.21.11
forge_minecraft_version=1.21.6

View File

@@ -40,6 +40,6 @@ public class SkinShufflePacketHandler {
if (!context.isServerSide() || sender == null)
return;
SkinShuffleCompatibility.handleSkinRefresh(SkinRestorer.getMinecraftServer(), sender, payload);
SkinShuffleCompatibility.handleSkinRefresh(sender.getServer(), sender, payload);
}
}

View File

@@ -3,12 +3,12 @@ group=net.lionarius
java_version=21
# Common
minecraft_version=1.21.11
minecraft_version_list=1.21.11
minecraft_version_range=[1.21.11, 1.22)
minecraft_version=1.21.6
minecraft_version_list=1.21.6,1.21.7,1.21.8
minecraft_version_range=[1.21.6, 1.21.8]
mod_id=skinrestorer
mod_name=SkinRestorer
mod_version=2.4.3
mod_version=2.5.0
mod_author=Lionarius
mod_homepage=https://modrinth.com/mod/skinrestorer
mod_sources=https://github.com/Suiranoil/SkinRestorer
@@ -18,11 +18,11 @@ credits=
description=A server-side mod for managing skins.
# Dependencies
mineskin_client_version=3.0.6-SNAPSHOT
mineskin_client_version=3.2.1-SNAPSHOT
# ParchmentMC mappings, see https://parchmentmc.org/docs/getting-started#choose-a-version for new versions
parchment_minecraft=1.21.8
parchment_version=2025.09.14
parchment_minecraft=1.21.6
parchment_version=2025.06.29
# Publishing
curseforge_id=443823

View File

@@ -17,10 +17,10 @@ neoForge {
accessTransformers.from(at.absolutePath)
}
// parchment {
// minecraftVersion = parchment_minecraft
// mappingsVersion = parchment_version
// }
parchment {
minecraftVersion = parchment_minecraft
mappingsVersion = parchment_version
}
runs {
configureEach {
@@ -55,4 +55,6 @@ dependencies {
prefer mineskin_client_version
}
}
additionalRuntimeClasspath "org.mineskin:java-client:${mineskin_client_version}"
}

View File

@@ -1,3 +1,3 @@
# NeoForge, see https://projects.neoforged.net/neoforged/neoforge for new versions
neoforge_version=21.11.0-beta
neoforge_version=21.6.0-beta
neoforge_loader_version_range=[4,)

View File

@@ -1,6 +1,5 @@
package net.lionarius.skinrestorer.neoforge.compat.skinshuffle;
import net.lionarius.skinrestorer.SkinRestorer;
import net.lionarius.skinrestorer.compat.skinshuffle.*;
import net.lionarius.skinrestorer.compat.skinshuffle.SkinShuffleCompatibility;
import net.minecraft.server.level.ServerPlayer;
@@ -27,6 +26,6 @@ public final class SkinShuffleModEventHandler {
private static void handleSkinRefreshPacket(SkinShuffleSkinRefreshPayload payload, IPayloadContext context) {
var player = (ServerPlayer) context.player();
SkinShuffleCompatibility.handleSkinRefresh(SkinRestorer.getMinecraftServer(), player, payload);
SkinShuffleCompatibility.handleSkinRefresh(player.getServer(), player, payload);
}
}