ra/runelite
1
0
Fork 0
Code Issues Pull Requests Actions 3 Packages Projects Releases Wiki Activity

Fix NPEs on startup from PlayerIndicators

Browse Source 
- Change the PlayerIndicators model to use injection
- Create new set from overlays when getOverlays is called

Signed-off-by: Tomas Slusny <slusnucky@gmail.com>
This commit is contained in:
Tomas Slusny
2018-03-14 17:48:25 +01:00
parent 57fd8b4283
commit 380d4faba7
4 changed files with 18 additions and 22 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
runelite-client/src/main/java/net/runelite/client/plugins/playerindicators/PlayerIndicatorsMinimapOverlay.java
View File

@@ -28,6 +28,8 @@ import java.awt.Color;
import java.awt.Dimension;
import java.awt.Graphics2D;
import java.awt.Point;
import javax.inject.Inject;
import javax.inject.Singleton;
import net.runelite.api.Player;
import net.runelite.client.ui.overlay.Overlay;
import net.runelite.client.ui.overlay.OverlayLayer;
@@ -35,12 +37,14 @@ import net.runelite.client.ui.overlay.OverlayPosition;
import net.runelite.client.ui.overlay.OverlayPriority;
import net.runelite.client.ui.overlay.OverlayUtil;
@Singleton
public class PlayerIndicatorsMinimapOverlay extends Overlay
{
private final PlayerIndicatorsService playerIndicatorsService;
private final PlayerIndicatorsConfig config;
PlayerIndicatorsMinimapOverlay(PlayerIndicatorsConfig config, PlayerIndicatorsService playerIndicatorsService)
@Inject
private PlayerIndicatorsMinimapOverlay(PlayerIndicatorsConfig config, PlayerIndicatorsService playerIndicatorsService)
{
this.config = config;
this.playerIndicatorsService = playerIndicatorsService;

6
runelite-client/src/main/java/net/runelite/client/plugins/playerindicators/PlayerIndicatorsOverlay.java
View File

@@ -29,18 +29,22 @@ import java.awt.Dimension;
import java.awt.Graphics2D;
import java.awt.Point;
import java.awt.Polygon;
import javax.inject.Inject;
import javax.inject.Singleton;
import net.runelite.api.Player;
import net.runelite.client.ui.overlay.Overlay;
import net.runelite.client.ui.overlay.OverlayPosition;
import net.runelite.client.ui.overlay.OverlayPriority;
import net.runelite.client.ui.overlay.OverlayUtil;
@Singleton
public class PlayerIndicatorsOverlay extends Overlay
{
private final PlayerIndicatorsService playerIndicatorsService;
private final PlayerIndicatorsConfig config;
PlayerIndicatorsOverlay(PlayerIndicatorsConfig config, PlayerIndicatorsService playerIndicatorsService)
@Inject
private PlayerIndicatorsOverlay(PlayerIndicatorsConfig config, PlayerIndicatorsService playerIndicatorsService)
{
this.config = config;
this.playerIndicatorsService = playerIndicatorsService;

22
runelite-client/src/main/java/net/runelite/client/plugins/playerindicators/PlayerIndicatorsPlugin.java
View File

@@ -28,7 +28,6 @@ import com.google.common.collect.Sets;
import com.google.inject.Provides;
import java.util.Collection;
import javax.inject.Inject;
import net.runelite.api.Client;
import net.runelite.client.config.ConfigManager;
import net.runelite.client.plugins.Plugin;
import net.runelite.client.plugins.PluginDescriptor;
@@ -40,12 +39,10 @@ import net.runelite.client.ui.overlay.Overlay;
public class PlayerIndicatorsPlugin extends Plugin
{
@Inject
private Client client;
private PlayerIndicatorsOverlay playerIndicatorsOverlay;
@Inject
private PlayerIndicatorsConfig config;
private Collection<Overlay> overlays;
private PlayerIndicatorsMinimapOverlay playerIndicatorsMinimapOverlay;
@Provides
PlayerIndicatorsConfig provideConfig(ConfigManager configManager)
@@ -53,22 +50,9 @@ public class PlayerIndicatorsPlugin extends Plugin
return configManager.getConfig(PlayerIndicatorsConfig.class);
}
@Override
protected void startUp() throws Exception
{
final PlayerIndicatorsService playerIndicatorsService =
new PlayerIndicatorsService(client, config);
final PlayerIndicatorsOverlay playerIndicatorsOverlay =
new PlayerIndicatorsOverlay(config, playerIndicatorsService);
final PlayerIndicatorsMinimapOverlay playerIndicatorsMinimapOverlay =
new PlayerIndicatorsMinimapOverlay(config, playerIndicatorsService);
overlays = Sets.newHashSet(playerIndicatorsOverlay, playerIndicatorsMinimapOverlay);
}
@Override
public Collection<Overlay> getOverlays()
{
return overlays;
return Sets.newHashSet(playerIndicatorsOverlay, playerIndicatorsMinimapOverlay);
}
}

6
runelite-client/src/main/java/net/runelite/client/plugins/playerindicators/PlayerIndicatorsService.java
View File

@@ -26,15 +26,19 @@ package net.runelite.client.plugins.playerindicators;
import java.awt.Color;
import java.util.function.BiConsumer;
import javax.inject.Inject;
import javax.inject.Singleton;
import net.runelite.api.Client;
import net.runelite.api.Player;
@Singleton
public class PlayerIndicatorsService
{
private final Client client;
private final PlayerIndicatorsConfig config;
PlayerIndicatorsService(Client client, PlayerIndicatorsConfig config)
@Inject
private PlayerIndicatorsService(Client client, PlayerIndicatorsConfig config)
{
this.config = config;
this.client = client;