-
-
Notifications
You must be signed in to change notification settings - Fork 104
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
Implement attachable tripwire mechanism and tags #2443
Conversation
@@ -8,9 +8,13 @@ | |||
import com.denizenscript.denizencore.objects.ObjectTag; | |||
import com.denizenscript.denizencore.objects.properties.Property; | |||
import com.denizenscript.denizencore.objects.properties.PropertyParser; | |||
import net.citizensnpcs.util.NMS; |
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.
Got a few unused imports here
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.
Resolved
@@ -53,6 +55,7 @@ public static void register() { | |||
// For a lantern, this returns whether it is hanging from the ceiling. | |||
// For a gate, this returns whether it is lowered to attach to a wall block. | |||
// For a mangrove_propagule, this returns whether it is hanging from the block above it. | |||
// For a tripwire, denotes whether a tripwire hook or string forms a complete tripwire circuit and is ready to trigger. |
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.
Should follow the format the rest of the entries above use, e.g. this returns whether...
Do not manually click "Resolve", that is a button for me/aya to use not a PR author |
...in/src/main/java/com/denizenscript/denizen/objects/properties/material/MaterialAttached.java
Show resolved
Hide resolved
@@ -129,6 +147,10 @@ public void adjust(Mechanism mechanism) { | |||
// For a lantern, this sets whether it is hanging from the ceiling. | |||
// For a gate, this sets whether it is lowered to attach to a wall block. | |||
// For a mangrove_propagule, this sets whether it is hanging from the block above it. | |||
// For a tripwire, this sets whether a tripwire hook or string forms a complete tripwire circuit and is ready to trigger. | |||
// Updating the property on a tripwire hook will change the texture to indicate a connected string, but will not have | |||
// any effect when used on the tripwire string itself. It may however still be used to check whether the string forms a circuit. |
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.
These lines are split really awkwardly. If you must insert a newline, place it after a period or a comma, not just in the middle.
Added tripwire and tripwirehook support to attachable as requested by ULSTICK on Discord: https://discord.com/channels/315163488085475337/1086271537872633937/1086271537872633937
Written with help from @tal5 and tested to be working successfully