TeamJM / journeymap-legacy

GNU Lesser General Public License v2.1
10 stars 13 forks source link

Added translations as primary dimName source #19

Closed AnrDaemon closed 7 months ago

CLAassistant commented 8 months ago

CLA assistant check
All committers have signed the CLA.

mysticdrew commented 8 months ago

CLA needs to be signed

mysticdrew commented 8 months ago

Also, can you add a before and after screenshot of what this PR accomplishes?

AnrDaemon commented 7 months ago

Also, can you add a before and after screenshot of what this PR accomplishes?

Missing dimension provider data (unchanged from original behavior): image

Original behavior (extra "overworld/superflat" dimension using FE multiworld module): image

New behavior (extra dimension name is taken from custom resource pack): image

AnrDaemon commented 7 months ago

I'm pondering an improvement to the caching mechanics. It should purge the cache on server reconnect and resource pack reload.

mysticdrew commented 7 months ago

I'm pondering an improvement to the caching mechanics. It should purge the cache on server reconnect and resource pack reload.

We do not need a cache honestly. It does not constantly poll this method.

AnrDaemon commented 7 months ago

Judging by the log (Constants.getString() log failures), it does.

mysticdrew commented 7 months ago

Judging by the log (Constants.getString() log failures), it does.

it shouldn't, I will fix that

mysticdrew commented 7 months ago

nevermind, I cannot easily fix that without some major refactoring. go ahead with your cache

mysticdrew commented 7 months ago

I'm pondering an improvement to the caching mechanics. It should purge the cache on server reconnect and resource pack reload.

purge it in JourneymapClient#reset method.

Then somewhere you'll want to add.

        ((IReloadableResourceManager)  Minecraft.getMinecraft().getResourceManager()).registerReloadListener(iResourceManager -> {
           // purge cache
        });