Skript

Skript

743k Downloads

ExprValueWithin and ExprEntities conflict

sovdeeth opened this issue ยท 0 comments

commented

Skript/Server Version

[22:22:13 INFO]: [Skript] Skript's aliases can be found here: https://github.com/SkriptLang/skript-aliases
[22:22:13 INFO]: [Skript] Skript's documentation can be found here: https://docs.skriptlang.org/
[22:22:13 INFO]: [Skript] Skript's tutorials can be found here: https://docs.skriptlang.org/tutorials
[22:22:13 INFO]: [Skript] Server Version: 1.21.3-27-661839e (MC: 1.21.3)
[22:22:13 INFO]: [Skript] Skript Version: 2.9.5 (selfbuilt-unknown) (also reported on 2.9.2)
[22:22:13 INFO]: [Skript] Installed Skript Addons:
[22:22:13 INFO]: [Skript]  - skript-reflect v2.5.1 (https://github.com/SkriptLang/skript-reflect)
[22:22:13 INFO]: [Skript]  - SkBee v3.6.1 (https://github.com/ShaneBeee/SkBee)
[22:22:13 INFO]: [Skript] Installed dependencies: None

Bug Description

When using loop entities within {_a} and {_b}, it is parsed as loop (entities within ({_a} and {_b})) using ExprValueWithin rather than ExprEntities and loop (entities within {_a} and {_b})

This, however, can be avoided by using loop all entities within {_a} and {_b} to force ExprEntities

Expected Behavior

Who should get priority is debatable, but I think ExprEntities is the much more common usecase and should take priority. That said, it would make using ExprValueWithin much harder on two variables. Perhaps it should be left as is.

Steps to Reproduce

loop entities within {_a} and {_b} run under debug verbosity shows the issue.

Errors or Screenshots

image
image

Other

No response

Agreement

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