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

Arithmetic returning Object when multiple valid operations causing unexpected parsing #6623

Closed
1 task done
sovdeeth opened this issue Apr 29, 2024 · 0 comments
Closed
1 task done
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. patch-ready A PR/issue that has been approved and is ready to be merged/closed for the next patch version.

Comments

@sovdeeth
Copy link
Member

Skript/Server Version

[12:52:55 INFO]: [Skript] Skript's aliases can be found here: https://github.com/SkriptLang/skript-aliases
[12:52:55 INFO]: [Skript] Skript's documentation can be found here: https://docs.skriptlang.org/
[12:52:55 INFO]: [Skript] Skript's tutorials can be found here: https://docs.skriptlang.org/tutorials
[12:52:55 INFO]: [Skript] Server Version: git-Paper-390 (MC: 1.20.4)
[12:52:55 INFO]: [Skript] Skript Version: 2.8.4-nightly-aa8af9c (skriptlang-nightly)
[12:52:55 INFO]: [Skript] Installed Skript Addons:
[12:52:55 INFO]: [Skript]  - SkBee v3.3.0 (https://github.com/ShaneBeee/SkBee)
[12:52:55 INFO]: [Skript] Installed dependencies:
[12:52:55 INFO]: [Skript]  - WorldGuard v7.0.9+5934e49

Bug Description

send health of {z} / max health of {z} parses as send health of ({z} / max health of {z}) since the latter's return type is Object, it is accepted by the livingentity input of health of.

Expected Behavior

Parse as send (health of {z}) / (max health of {z})

Steps to Reproduce

> !spawn pig at spawn of world "world"
[12:50:58 INFO]: executing 'spawn pig at spawn of world "world"'
> !set {z} to last spawned pig
[12:51:03 INFO]: executing 'set {z} to last spawned pig'
> !send health of {z}
[12:51:07 INFO]: executing 'send health of {z}'
[12:51:07 INFO]: 5
> !send max health of {z}
[12:51:10 INFO]: executing 'send max health of {z}'
[12:51:10 INFO]: 5
> !send health of {z} / max health of {z}
[12:51:19 INFO]: executing 'send health of {z} / max health of {z}'
> !send "%(health of {z}) / (max health of {z})%"
[12:51:49 INFO]: executing 'send "%(health of {z}) / (max health of {z})%"'
[12:51:49 INFO]: 1

Errors or Screenshots

No response

Other

No response

Agreement

  • I have read the guidelines above and affirm I am following them with this report.
@sovdeeth sovdeeth added the bug An issue that needs to be fixed. Alternatively, a PR fixing an issue. label Apr 29, 2024
@Moderocky Moderocky added the PR available Issues which have a yet-to-be merged PR resolving it label Apr 29, 2024
@sovdeeth sovdeeth added the patch-ready A PR/issue that has been approved and is ready to be merged/closed for the next patch version. label May 29, 2024
@APickledWalrus APickledWalrus 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 Jun 1, 2024
@sovdeeth sovdeeth closed this as completed Jun 1, 2024
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. patch-ready A PR/issue that has been approved and is ready to be merged/closed for the next patch version.
Projects
None yet
Development

No branches or pull requests

3 participants