Skript

Skript

743k Downloads

Can't Spawn a Firework

Dsa-123 opened this issue ยท 18 comments

commented

Skript/Server Version

[18:53:08 INFO]: [Skript] Skript's aliases can be found here: https://github.com/SkriptLang/skript-aliases
[18:53:08 INFO]: [Skript] Skript's documentation can be found here: https://docs.skriptlang.org/
[18:53:08 INFO]: [Skript] Skript's tutorials can be found here: https://docs.skriptlang.org/tutorials
[18:53:08 INFO]: [Skript] Server Version: git-Paper-496 (MC: 1.20.4)
[18:53:08 INFO]: [Skript] Skript Version: 2.8.6 (skriptlang-github)
[18:53:08 INFO]: [Skript] Installed Skript Addons: None
[18:53:08 INFO]: [Skript] Installed dependencies:
[18:53:08 INFO]: [Skript]  - Vault v1.7.3-b131
[18:53:08 INFO]: [Skript]  - WorldGuard v7.0.7+216b061

Bug Description

Hello SkriptLang Team,
I've updated my Skript version from 2.7.4 to 2.8.6 recently and it seems that I can't spawn fireworks anymore

Expected Behavior

Expected to spawn a firework

Steps to Reproduce

After enabling Skript effect commands write !spawn firework at player in chat

Errors or Screenshots

No response

Other

No response

Agreement

  • I have read the guidelines above and affirm I am following them with this report.
commented

Sorry, I'm new to GitHub and I couldn't figure out how to try that pull request you mentioned

Click on the link to the PR, then click on the Checks tab, click on the Java 21 tab on the left, then download the skript-nightly under artifacts

commented

This is likely the same problem that #6760 fixes. Try that nightly, see if it fixes it.

commented

Sorry, I'm new to GitHub and I couldn't figure out how to try that pull request you mentioned

commented

Yeah, looks like we'll have to either force firework particle or firework spark for the particle name. Pickle's aware and will update the PR to fix this too.

commented

I've updated the PR again to remove the firework conflict. If you use the latest build it should work. Let us know if not.

commented

Perfect. I deleted my comment by accident. Thank you so much

commented

I've updated the PR again to remove the firework conflict. If you use the latest build it should work. Let us know if not.

I will test it in few moments

commented

Not working

commented

If you use the latest build it should work. Let us know if not.

Skript Version: 2.8.6-nightly-9326c82 (skriptlang-nightly) is that the latest build?

commented

OK I tried that and it still not letting me to spawn fireworks

commented

It doesn't display any error

commented

is it still printing an error?

commented

image

command test:
    trigger:

        broadcast "Started"
        loop 10 times:

            spawn a firework at player
            broadcast "%last spawned entity%"
            wait 5 ticks

        broadcast "The End"
commented

Also this is what send when using getClass method
image

commented

I tried to do some debugging because I thought it would be parsing error but no result at all
image

commented

Seems to be due to FIREWORK_ROCKET being marked as non-spawnable by Bukkit, while clearly being able to be spawned.

commented

@Dsa-123 Are you able to test with the latest nightly build from #6764? It should resolve this issue.