Next/Current Loop Value seems to break after using stop

Question

Next/Current Loop Value seems to break after using stop

Closed this issue 2 months ago · 0 comments

Fusezion commented 3 months ago

Skript/Server Version

[13:09:05 INFO]: [Skript] Skript's aliases can be found here: https://github.com/SkriptLang/skript-aliases
[13:09:05 INFO]: [Skript] Skript's documentation can be found here: https://docs.skriptlang.org/
[13:09:05 INFO]: [Skript] Skript's tutorials can be found here: https://docs.skriptlang.org/tutorials
[13:09:05 INFO]: [Skript] Server Version: 1.21.4-232-12d8fe0 (MC: 1.21.4)
[13:09:05 INFO]: [Skript] Skript Version: 2.11.2 (skriptlang-github)
[13:09:05 INFO]: [Skript] Installed Skript Addons:
[13:09:05 INFO]: [Skript]  - SkBee v3.11.3 (https://github.com/ShaneBeee/SkBee)
[13:09:05 INFO]: [Skript]  - skript-gui v1.3.1 (https://github.com/APickledWalrus/skript-gui)
[13:09:05 INFO]: [Skript]  - skript-reflect v2.6 (https://github.com/SkriptLang/skript-reflect)
[13:09:05 INFO]: [Skript]  - oopsk v1.0-beta1 (https://github.com/sovdeeth/oopsk)
[13:09:05 INFO]: [Skript]  - skript-yaml v1.6.1
[13:09:05 INFO]: [Skript] Installed dependencies:
[13:09:05 INFO]: [Skript]  - Vault v1.7.3-b131
[13:09:05 INFO]: [Skript]  - WorldGuard v7.0.13+82fdc65

Bug Description

Skript seems to retain some contextual information regarding loop context for next loop-value and loop-value whenever a user does stop within the loop.

Expected Behavior

Skript to correctly start the loop over and not carry any pre existing data.

Steps to Reproduce

run this code 2-3 times

command /testing-loop-context:
    trigger:
        broadcast "%now%"
        loop 4 times:
            broadcast "previous: %previous loop-value%"
            broadcast "current: %loop-value%"
            broadcast "next: %next loop-value%"
            stop

Errors or Screenshots

First image of using the command with a stop [loop]

Second image of using the command without a stop

Other

I'm not honestly sure what to call this issue, as it does just seem to be carrying data over, I am unable to really look too much more into this as I am tight on time rn. This was just a quick reproducible action I could make

Agreement

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

Share to