Dragon tags & mechs #2620
Open
Dragon tags & mechs #2620
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
paper/src/main/java/com/denizenscript/denizen/paper/properties/PaperWorldExtensions.java
Show resolved
Hide resolved
paper/src/main/java/com/denizenscript/denizen/paper/properties/PaperWorldExtensions.java
Show resolved
Hide resolved
| if (!mechanism.hasValue()) { | ||
| battle.spawnNewGateway(); | ||
| } | ||
| if (mechanism.requireObject(LocationTag.class)) { |
| // --> | ||
| tagProcessor.registerMechanism("respawn_dragon", false, (object, mechanism) -> { | ||
| if (object.getWorld() == null) { | ||
| mechanism.echoError("World '" + object.world_name + "' is unloaded, cannot set respawn phase."); |
There was a problem hiding this comment.
... when is this a real concern?
| // <--[mechanism] | ||
| // @object WorldTag | ||
| // @name respawn_phase | ||
| // @input ElementTag(String) |
mcmonkey4eva
approved these changes
Jul 5, 2024
tal5
reviewed
Aug 31, 2024
| // @returns ElementTag(Number) | ||
| // @description | ||
| // Returns the number of end gateway portals. | ||
| // Only works in end worlds. |
There was a problem hiding this comment.
Minor nitpick, but in sounds more like the player being in an end world while the script is running or something - maybe on, for, if the world is an end world, etc.?
tal5
reviewed
Aug 31, 2024
| // @attribute <WorldTag.healing_crystals> | ||
| // @returns ListTag(EntityTag) | ||
| // @description | ||
| // Returns a ListTag of the healing crystals located on top of the obsidian towers. |
There was a problem hiding this comment.
I'd mention ender/end crystals as a search aid or in the description somewhere, healing crystals is technically right but most people probably won't know them by that name.
tal5
reviewed
Aug 31, 2024
| }); | ||
|
|
||
| // <--[tag] | ||
| // @attribute <WorldTag.healing_crystals> |
tal5
reviewed
Aug 31, 2024
| // @Plugin Paper | ||
| // @group paper | ||
| // @description | ||
| // If no location is specified, tries to spawn a new end gateway using default game mechanics. Otherwise, spawns a new end gateway portal at the specified location. |
There was a problem hiding this comment.
Nitpick, but meta is usually the other way around (spawns gateway at specified location, specify no input to spawn vanilla one)
tal5
reviewed
Aug 31, 2024
| battle.spawnNewGateway(); | ||
| } | ||
| else if (mechanism.requireObject(LocationTag.class)) { | ||
| battle.spawnNewGateway(mechanism.getValue().asType(LocationTag.class, mechanism.context)); |
There was a problem hiding this comment.
There should be a #valueAsType iirc?
tal5
reviewed
Aug 31, 2024
| // @name respawn_phase | ||
| // @input ElementTag | ||
| // @description | ||
| // Set the current respawn phase of the ender dragon. Valid phases can be found at <@link url https://jd.papermc.io/paper/1.20/org/bukkit/boss/DragonBattle.RespawnPhase.html> |
tal5
reviewed
Aug 31, 2024
Comment on lines
+1068
to
+1075
| tagProcessor.registerMechanism("respawn_phase", false, ElementTag.class, (object, mechanism, input) -> { | ||
| DragonBattle battle = object.getWorld().getEnderDragonBattle(); | ||
| if (battle == null) { | ||
| mechanism.echoError("Provided world is not an end world!"); | ||
| return; | ||
| } | ||
| battle.setRespawnPhase(input.asEnum(DragonBattle.RespawnPhase.class)); | ||
| }); |
tal5
reviewed
Aug 31, 2024
|
|
||
| // <--[mechanism] | ||
| // @object WorldTag | ||
| // @name respawn_phase |
tal5
reviewed
Aug 31, 2024
Comment on lines
+1087
to
+1094
| tagProcessor.registerMechanism("first_dragon_killed", false, ElementTag.class, (object, mechanism, input) -> { | ||
| DragonBattle battle = object.getWorld().getEnderDragonBattle(); | ||
| if (battle == null) { | ||
| mechanism.echoError("Provided world is not an end world!"); | ||
| return; | ||
| } | ||
| battle.setPreviouslyKilled(input.asBoolean()); | ||
| }); |
tal5
reviewed
Aug 31, 2024
Comment on lines
+1084
to
+1085
| // Set whether the first ender dragon was killed already. | ||
| // Only works in end worlds. |
There was a problem hiding this comment.
Might be nice to document exactly what this does - does it reset the end and respawn the dragon? is it just a value that doesn't really affect anything?
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
gateway_counthealing_crystalsrespawn_crystalsspawn_gatewayfirst_dragon_killedrespawn_dragonreset_crystalsrespawn_phasefirst_dragon_killedI opened this as a draft PR because I haven't tested any of this/just wrote this up, and want it open so that I don't lose it in my sea of local and published branches. Feel free to comment/review though!