diff --git a/.github/ISSUE_TEMPLATE/bug_report.yml b/.github/ISSUE_TEMPLATE/bug_report.yml new file mode 100644 index 00000000..113b6525 --- /dev/null +++ b/.github/ISSUE_TEMPLATE/bug_report.yml @@ -0,0 +1,54 @@ +name: Bug Report +description: "For reporting bugs and other defects" +body: + - type: markdown + attributes: + value: >- + **Note: This issue tracker is not intended for support requests!** If you need help with crashes or other issues, then + you should [ask on our Discord server](https://discord.gg/rN9Y7caguP) instead. Unless you are certain that you + have found a defect, and you are able to point to where the problem is, you should not open an issue. +

+ Additionally, please make sure you have done the following: + + - **Have you ensured that all of your mods (including ModernFix) are up-to-date?** The latest version of ModernFix + can always be found [on Modrinth](https://modrinth.com/mod/modernfix). + + - **Have you used the [search tool](https://github.com/embeddedt/ModernFix/issues) to check whether your issue + has already been reported?** If it has been, then consider adding more information to the existing issue instead. + + - **Have you determined the minimum set of instructions to reproduce the issue?** If your problem only occurs + with other mods installed, then you should narrow down exactly which mods are causing the issue. Please do not + provide your entire list of mods to us and expect that we will be able to figure out the problem. + - type: textarea + id: description + attributes: + label: Bug Description + description: >- + Use this section to describe the issue you are experiencing in as much depth as possible. The description should + explain what behavior you were expecting, and why you believe the issue to be a bug. If the issue you are reporting + only occurs with specific mods installed, then provide the name and version of each mod. + + **Hint:** If you have any screenshots, videos, or other information that you feel is necessary to + explain the issue, you can attach them here. + - type: textarea + id: description-reproduction-steps + attributes: + label: Reproduction Steps + description: >- + Provide as much information as possible on how to reproduce this bug. Make sure your instructions are as clear and + concise as possible, because other people will need to be able to follow your guide in order to re-create the issue. + + **Hint:** A common way to fill this section out is to write a step-by-step guide. + validations: + required: true + - type: textarea + id: log-file + attributes: + label: Log File + description: >- + **Hint:** You can usually find the log files within the folder `.minecraft/logs`. Most often, you will want the `latest.log` + file, since that file belongs to the last played session of the game. + placeholder: >- + Drag-and-drop the log file here. + validations: + required: true diff --git a/.github/ISSUE_TEMPLATE/config.yml b/.github/ISSUE_TEMPLATE/config.yml new file mode 100644 index 00000000..2354f98d --- /dev/null +++ b/.github/ISSUE_TEMPLATE/config.yml @@ -0,0 +1,6 @@ +blank_issues_enabled: true +contact_links: + - name: For help with other issues, join our Discord community + url: https://discord.gg/rN9Y7caguP + about: This is the best option for getting help with mod installation, performance issues, and any other support inquiries + # Copied from https://github.com/CaffeineMC/sodium-fabric#community diff --git a/fabric/src/main/java/org/embeddedt/modernfix/platform/fabric/ModernFixPlatformHooksImpl.java b/fabric/src/main/java/org/embeddedt/modernfix/platform/fabric/ModernFixPlatformHooksImpl.java index 65380a62..a6a5ee78 100644 --- a/fabric/src/main/java/org/embeddedt/modernfix/platform/fabric/ModernFixPlatformHooksImpl.java +++ b/fabric/src/main/java/org/embeddedt/modernfix/platform/fabric/ModernFixPlatformHooksImpl.java @@ -38,13 +38,11 @@ public class ModernFixPlatformHooksImpl implements ModernFixPlatformHooks { return FabricLoader.getInstance().getEnvironmentType() == EnvType.SERVER; } - private static String verString; + private static final String verString = FabricLoader.getInstance().getModContainer("modernfix") + .map(mfModContainer -> mfModContainer.getMetadata().getVersion().getFriendlyString()) + .orElse("[unknown]"); public String getVersionString() { - if(verString == null) { - ModContainer mfModContainer = FabricLoader.getInstance().getModContainer("modernfix").get(); - verString = mfModContainer.getMetadata().getVersion().getFriendlyString(); - } return verString; } diff --git a/forge/src/main/java/org/embeddedt/modernfix/platform/forge/ModernFixPlatformHooksImpl.java b/forge/src/main/java/org/embeddedt/modernfix/platform/forge/ModernFixPlatformHooksImpl.java index 744dd83f..f5e958bd 100644 --- a/forge/src/main/java/org/embeddedt/modernfix/platform/forge/ModernFixPlatformHooksImpl.java +++ b/forge/src/main/java/org/embeddedt/modernfix/platform/forge/ModernFixPlatformHooksImpl.java @@ -43,7 +43,7 @@ import java.lang.reflect.Constructor; import java.lang.reflect.Field; import java.nio.file.Path; import java.util.Map; -import java.util.Objects; +import java.util.Optional; import java.util.function.Consumer; public class ModernFixPlatformHooksImpl implements ModernFixPlatformHooks { @@ -55,17 +55,11 @@ public class ModernFixPlatformHooksImpl implements ModernFixPlatformHooks { return FMLLoader.getDist().isDedicatedServer(); } - private static String verString; + private static final String verString = Optional.ofNullable( + ModernFixMixinPlugin.class.getPackage().getImplementationVersion()) + .orElse("[unknown]"); public String getVersionString() { - if(verString == null) { - try { - verString = ModernFixMixinPlugin.class.getPackage().getImplementationVersion(); - Objects.requireNonNull(verString); - } catch(Throwable e) { - verString = "[unknown]"; - } - } return verString; }