Skript icon indicating copy to clipboard operation
Skript copied to clipboard

Can't Spawn a Firework

Open Dsa-123 opened this issue 1 year ago • 18 comments

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

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

Dsa-123 avatar Jun 03 '24 15:06 Dsa-123

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

sovdeeth avatar Jun 03 '24 15:06 sovdeeth

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

Dsa-123 avatar Jun 03 '24 16:06 Dsa-123

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

sovdeeth avatar Jun 03 '24 16:06 sovdeeth

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.

sovdeeth avatar Jun 03 '24 16:06 sovdeeth

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.

APickledWalrus avatar Jun 03 '24 16:06 APickledWalrus

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

Dsa-123 avatar Jun 03 '24 16:06 Dsa-123

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

Dsa-123 avatar Jun 03 '24 16:06 Dsa-123

Not working

Dsa-123 avatar Jun 03 '24 16:06 Dsa-123

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?

Dsa-123 avatar Jun 03 '24 16:06 Dsa-123

This should be it https://github.com/SkriptLang/Skript/actions/runs/9353758434

APickledWalrus avatar Jun 03 '24 17:06 APickledWalrus

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

Dsa-123 avatar Jun 03 '24 17:06 Dsa-123

is it still printing an error?

APickledWalrus avatar Jun 03 '24 17:06 APickledWalrus

It doesn't display any error

Dsa-123 avatar Jun 03 '24 17:06 Dsa-123

image

command test:
    trigger:

        broadcast "Started"
        loop 10 times:

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

        broadcast "The End"

Dsa-123 avatar Jun 03 '24 17:06 Dsa-123

Also this is what send when using getClass method image

Dsa-123 avatar Jun 03 '24 17:06 Dsa-123

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

Dsa-123 avatar Jun 03 '24 17:06 Dsa-123

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

sovdeeth avatar Jun 03 '24 18:06 sovdeeth

@Dsa-123 Are you able to test with the latest nightly build from https://github.com/SkriptLang/Skript/pull/6764? It should resolve this issue.

APickledWalrus avatar Jun 06 '24 16:06 APickledWalrus