New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Better SceneTags #2361
Better SceneTags #2361
Conversation
src/main/java/emu/grasscutter/game/player/PlayerProgressManager.java
Outdated
Show resolved
Hide resolved
src/main/java/emu/grasscutter/server/packet/send/PacketPlayerWorldSceneInfoListNotify.java
Outdated
Show resolved
Hide resolved
src/main/java/emu/grasscutter/command/commands/SetSceneTagCommand.java
Outdated
Show resolved
Hide resolved
src/main/java/emu/grasscutter/command/commands/SetSceneTagCommand.java
Outdated
Show resolved
Hide resolved
src/main/java/emu/grasscutter/command/commands/SetSceneTagCommand.java
Outdated
Show resolved
Hide resolved
src/main/java/emu/grasscutter/command/commands/SetSceneTagCommand.java
Outdated
Show resolved
Hide resolved
logger.warn("[LUA] Call unimplemented AddSceneTag with {}, {}", sceneId, sceneTagId); | ||
//TODO implement | ||
logger.debug("[LUA] Call AddSceneTag with {}, {}", sceneId, sceneTagId); | ||
sceneScriptManager.get().getScene().getHost().getProgressManager().addSceneTag(sceneId, sceneTagId); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
there is probably a better way of getting the scene or the host
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I looked at other methods but it seemed like this was going to be the cleanest method from within ScriptLib. If you'd prefer new imports & methods I'm open to suggestion.
…and.java Co-authored-by: Magix <27646710+KingRainbow44@users.noreply.github.com>
…and.java Co-authored-by: Magix <27646710+KingRainbow44@users.noreply.github.com>
Description
Better handling and implementations for scene tags.
Type of changes
Checklist: