wise-old-man / wiseoldman-runelite-plugin

The RuneLite plugin for the Wise Old Man project.
BSD 2-Clause "Simplified" License
3 stars 7 forks source link

Use client thread & fix a NPE with syncButton #41

Closed JZomDev closed 7 months ago

JZomDev commented 7 months ago

null check syncButton on CLAN_SETTINGS_INFO_PAGE_WIDGET use clientThread for invoking

I think this is the only places where it could error, i couldn't get another java.lang.IllegalStateException: must be called on client thread infact i only ever got a NPE on syncButton being null. I did not get the complaint about which thread did the invoke, but only instances of clientThread.invoke are used now rather than generically doing a runnable IE this::syncMembers or () -> SyncMembers(false) so it should get less errors now.