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

crowdsourcing: don't include menuoptionclicked event in movementdata

Browse Source 
Serializing the event no longer works as expected, but is also not a good idea in general
This commit is contained in:
Adam
2022-04-29 14:16:02 -04:00
parent c40b33a1e9
commit 62c0af064d
2 changed files with 10 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
runelite-client/src/main/java/net/runelite/client/plugins/crowdsourcing/movement/CrowdsourcingMovement.java
View File

@@ -69,7 +69,9 @@ public class CrowdsourcingMovement
int distance = nextPoint.distanceTo(lastPoint);
if (distance > 2 || nextIsInInstance != lastIsInInstance)
{
MovementData data = new MovementData(lastPoint, nextPoint, lastIsInInstance, nextIsInInstance, ticksStill, lastClick);
MovementData data = new MovementData(lastPoint, nextPoint, lastIsInInstance, nextIsInInstance,
ticksStill, lastClick.getMenuAction(), lastClick.getId(), lastClick.getMenuOption(),
lastClick.getMenuTarget(), lastClick.getParam0(), lastClick.getParam1());
manager.storeEvent(data);
}
if (distance > 0)

9
runelite-client/src/main/java/net/runelite/client/plugins/crowdsourcing/movement/MovementData.java
View File

@@ -27,8 +27,8 @@ package net.runelite.client.plugins.crowdsourcing.movement;
import lombok.AllArgsConstructor;
import lombok.Data;
import net.runelite.api.MenuAction;
import net.runelite.api.coords.WorldPoint;
import net.runelite.api.events.MenuOptionClicked;
@Data
@AllArgsConstructor
@@ -39,5 +39,10 @@ public class MovementData
private final boolean fromInstance;
private final boolean toInstance;
private final int ticks;
private MenuOptionClicked lastClick;
private MenuAction menuAction;
private int menuIdentifier;
private String menuOption;
private String menuTarget;
private int param0;
private int param1;
}