-
Notifications
You must be signed in to change notification settings - Fork 9
Issues with reading lang files #15
Comments
So, this is to do with the lang system of haruhi. Relevant line. Though I'm not sure why it can't find the lang I tested the release before posting it, it worked. I guess I'll have to register the menus in a suitable event and see if that fixes it. Dang hastily implemented lang discovery system |
I've just tested it again and this error never pops up, but I am being a slight dummy with the code. It should really wait until the program is finished loading before using translated strings |
@kinker31 do you get the same issue with this fix? |
Nope, but I do get this:
...alongside a debug trace like this:
|
Alright, does it still happen in this? |
Nope, I get this:
With a debug trace like this:
|
My bad, I forgot to test that one and get the same error. How's this one? |
No dice, sadly. Error:
Debug Trace:
|
Right. I honestly don't get why you're having this, but I'm not. Unless you feel like debugging the plugin class loader for why it won't load the resources, I think I'm just going to have to ditch the lang system |
I asked a friend to help debug this - with the exact same code as me, they're getting the same error and I don't. We weren't able to figure this out. So I'm removing the lang system but leaving this open for if anyone can/wants to help |
Can reproduce by running gradle |
If you can reproduce it, you're welcome to make a fork it, make a branch at 7cdfb7c and try to fix it. But it's more important to me that it works for people rather than if it can theoretically be translated, so I'm just removing the lang system |
@kinker31, how's this? I've removed the localization system, which should stop those errors |
Finally works! |
Seems to be working. |
Great. I'm leaving this open as I do want a localization system so it can be translated, even though this bug is related to haruhi, that doesn't get the traction this project does. I'm really not sure why the resources aren't being loaded. The first step for anyone who wants to debug this, if you're reading this, is to debug the plugin class loader in |
Error Stack
System Info
Debug Trace
The text was updated successfully, but these errors were encountered: