Housekeeping: remove unneeded legacy version checks & clean up code. #6676
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
* Remove deprecated visibility method use. * Remove support checks from block change. * Fix mistake in SendTitle (and remove 1.13 warnings). * Remove check from make fly.
Moderocky
added
documentation
cheeezburga
reviewed
May 13, 2024
| if (Skript.methodExists(Player.class, "hasResourcePack")) | ||
| register(CondHasResourcePack.class, PropertyType.HAVE, "[a] resource pack [(loaded|installed)]", "players"); | ||
| register(CondHasResourcePack.class, PropertyType.HAVE, "[a] resource pack [(loaded|installed)]", "players"); |
There was a problem hiding this comment.
Doesn't this still require the check? I can't see the Player#hasResourcePack() in Spigot 1.13.2.
There was a problem hiding this comment.
Based on the fact @RequiredPlugins state "Paper 1.9 or newer" it is possible. As such I did go and check latest spigot releases and there's no method for hasResourcePack so this check should still remain
There was a problem hiding this comment.
yes this was only on the list for documentation update
Comment on lines
-45
to
+48
| if (Skript.methodExists(Creeper.class, "isIgnited")) { | ||
| Skript.registerCondition(CondIgnitionProcess.class, | ||
| "[creeper[s]] %livingentities% ((is|are)|1¦(isn't|is not|aren't|are not)) going to explode", | ||
| "[creeper[s]] %livingentities% ((is|are)|1¦(isn't|is not|aren't|are not)) in the (ignition|explosion) process", | ||
| "creeper[s] %livingentities% ((is|are)|1¦(isn't|is not|aren't|are not)) ignited"); | ||
| } | ||
| Skript.registerCondition(CondIgnitionProcess.class, | ||
| "[creeper[s]] %livingentities% ((is|are)|1¦(isn't|is not|aren't|are not)) going to explode", | ||
| "[creeper[s]] %livingentities% ((is|are)|1¦(isn't|is not|aren't|are not)) in the (ignition|explosion) process", | ||
| "creeper[s] %livingentities% ((is|are)|1¦(isn't|is not|aren't|are not)) ignited"); |
Removed some redundant existing checks and some other changes - Happy to revert other changes (changes that aren't only the existing checks) if they're out of the scope of this PR (6676)
This was referenced May 13, 2024
* Removed some redundant exists checks and made formatting more consistent in SimpleEvents * Removed unused imports * Put back the potiondata method handles --- Co-authored-by: Moderocky <[email protected]>
…ses (#6693) * Removed PassengerUtils class and replaced methods in referencing classes * Apply suggestions from code review * Formatting stuff --------- Co-authored-by: Moderocky <[email protected]>
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.
Description
This is the batch merge request for #6641.
It removes checks for methods existing that are standard as of 1.13.2, since we no longer support versions below this.
What can be removed?
Version checks for 1.13 and lower. Method & class existence checks that were standard as of Spigot for 1.13.2. Paper-only checks are still required for a few things that didn't arrive in Spigot until later (or at all).
See the linked issue #6641 for more details.
If you want to contribute to this project, open a pull request targeting
remove-legacy-codeand it will be included in this pull request.What next?
Another clean-up can be done before 2.10 for things under 1.17.
This will include a few bits and pieces that couldn't be fully cleaned up in this PR.
This is a branch merge request!
Individual pull requests were made by contributors to its feature branch: this can be merged as-is to preserve the Git change history.
Target Minecraft Versions: any
Requirements: none
Related Issues: #6641