Skip to content
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

Can't delete target entity #2695

Closed
ShaneBeee opened this issue Dec 14, 2019 · 3 comments
Closed

Can't delete target entity #2695

ShaneBeee opened this issue Dec 14, 2019 · 3 comments
Labels
bug An issue that needs to be fixed. Alternatively, a PR fixing an issue. completed The issue has been fully resolved and the change will be in the next Skript update. priority: low Issues that are not harmful to the experience but are related to useful changes or additions.

Comments

@ShaneBeee
Copy link
Contributor

Description

When attempting to delete a target entity, nothing happens

Steps to Reproduce

example code:

delete target entity of player

When running this code, the entity is still there.

Expected Behavior

The entity should delete

Errors / Screenshots

no errors

Server Information

  • Server version/platform: 2.4
  • Skript version: git-Paper-5 (MC: 1.15)

Additional Context

I found the issue, will be doing a PR shortly.

@ShaneBeee ShaneBeee added bug An issue that needs to be fixed. Alternatively, a PR fixing an issue. priority: low Issues that are not harmful to the experience but are related to useful changes or additions. labels Dec 14, 2019
@Thibaut040
Copy link

You can always try to kill the target entity :)

@ShaneBeee
Copy link
Contributor Author

You can always try to kill the target entity :)

Yes, I am aware of what I can do, but Im not trying to kill an entity, I'm trying to delete them.

@Wealthyturtle
Copy link
Member

@Anoniempje1234 Killing the target entity will result in particles, and mob drops, which adds further complexity.

@TPGamesNL TPGamesNL added the PR available Issues which have a yet-to-be merged PR resolving it label Jul 30, 2021
@TheLimeGlass TheLimeGlass added completed The issue has been fully resolved and the change will be in the next Skript update. and removed PR available Issues which have a yet-to-be merged PR resolving it labels Apr 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug An issue that needs to be fixed. Alternatively, a PR fixing an issue. completed The issue has been fully resolved and the change will be in the next Skript update. priority: low Issues that are not harmful to the experience but are related to useful changes or additions.
Projects
None yet
Development

No branches or pull requests

5 participants