ModernFix

ModernFix

2M Downloads

`modernfix.debugReloader` should affect datapack reload as well

SettingDust opened this issue ยท 1 comments

commented

Bug Description

For now the modernfix.debugReloader only applied in https://github.com/embeddedt/ModernFix/blob/1.16/common/src/main/java/org/embeddedt/modernfix/common/mixin/feature/measure_time/SimpleReloadableResourceManagerMixin.java

But not DataPackContents(ReloadableServerResources in mojmap)
SimpleResourceReload.start(manager, dataPackContents.getContents(), prepareExecutor, applyExecutor, COMPLETED_UNIT, LOGGER.isDebugEnabled())

Modify the boolean in SimpleReloadInstance#create maybe better?

Reproduction Steps

Enable modernfix.debugReloader with JVM flag and join world

Log File

no-op

commented

Implemented the suggested change in 5.19.0.