Skip to content

Entity/Block Persistence + Entity Despawnable#7564

Merged
Efnilite merged 9 commits into
SkriptLang:dev/featurefrom
Absolutionism:dev/Persistence+Despawnable
Mar 20, 2025
Merged

Entity/Block Persistence + Entity Despawnable#7564
Efnilite merged 9 commits into
SkriptLang:dev/featurefrom
Absolutionism:dev/Persistence+Despawnable

Conversation

@Absolutionism
Copy link
Copy Markdown
Contributor

@Absolutionism Absolutionism commented Feb 1, 2025

Description

This PR aims to add the ability of changing the persistence of entities and blocks (leaves) and change if an entity should despawn when far away.

No test for EntityUnload because it requires a tick.
Yes, I did try the JUnit thing, but was not viable as sometimes it succeeded and other times failed.
Granted if I did it correctly, look at the Initial Commit


Target Minecraft Versions: any
Requirements: none
Related Issues: #7154

Copy link
Copy Markdown
Member

@Efnilite Efnilite left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

your junit attempt did not call the actual event with java code, which is why it didnt work

Comment thread src/main/java/ch/njol/skript/effects/EffPersistent.java Outdated
Comment thread src/main/java/ch/njol/skript/conditions/CondEntityUnload.java Outdated
@sovdeeth sovdeeth added the enhancement Feature request, an issue about something that could be improved, or a PR improving something. label Feb 2, 2025
Comment thread src/main/java/ch/njol/skript/conditions/CondIsPersistent.java Outdated
Comment thread src/main/java/ch/njol/skript/conditions/CondIsPersistent.java Outdated
Comment thread src/main/java/ch/njol/skript/conditions/CondIsPersistent.java Outdated
Comment thread src/main/java/ch/njol/skript/conditions/CondIsPersistent.java
Comment thread src/main/java/ch/njol/skript/effects/EffEntityUnload.java Outdated
Comment thread src/main/java/ch/njol/skript/effects/EffEntityUnload.java
Comment thread src/main/java/ch/njol/skript/effects/EffPersistent.java
Comment thread src/main/java/ch/njol/skript/effects/EffPersistent.java Outdated
Comment thread src/main/java/ch/njol/skript/effects/EffPersistent.java Outdated
Comment thread src/test/skript/tests/syntaxes/effects/EffPersistent.sk
@sovdeeth sovdeeth added the feature-ready A PR/issue that has been approved, tested and can be merged/closed in the next feature version. label Feb 22, 2025
@Fusezion
Copy link
Copy Markdown
Contributor

While looking through issues found #6794 which this completes

@Absolutionism Absolutionism requested a review from a team as a code owner March 19, 2025 22:33
@Absolutionism Absolutionism requested review from UnderscoreTud and erenkarakal and removed request for ShaneBeee March 19, 2025 22:33
@Efnilite Efnilite merged commit c0f6ba4 into SkriptLang:dev/feature Mar 20, 2025
erenkarakal pushed a commit to erenkarakal/Skript that referenced this pull request Nov 26, 2025
* Initial Commit

* Remove JUnit Test

Discrepancies

* Fix Example

* Update Docs

* Requested Changes

* Readd "not"

* Fixxy Fix

---------

Co-authored-by: Efnilite <35348263+Efnilite@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement Feature request, an issue about something that could be improved, or a PR improving something. feature-ready A PR/issue that has been approved, tested and can be merged/closed in the next feature version.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants