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.

Sign up for GitHub

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

Jump to bottom

Math is done in the wrong order #535

Closed
skripttpbug2 opened this issue Apr 29, 2017 · 2 comments
Closed

Math is done in the wrong order #535

skripttpbug2 opened this issue Apr 29, 2017 · 2 comments
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. priority: medium Issues that are detrimental to user experience (prohibitive bugs or lack of useful implementation).

Comments

@skripttpbug2
Copy link

skripttpbug2 commented Apr 29, 2017 •
edited

https://i.imgur.com/eAwNDJb.png

The first is actually doing "5-(2-3)" instead of what is should be doing ("(5-2)-3")

you talk about this in #121
@Syst3ms
Copy link
Contributor

Syst3ms commented Apr 29, 2017 •
edited

Yeah, I don't see a good way to make it atm. For now you can just use () to add priority

@bensku bensku added the bug An issue that needs to be fixed. Alternatively, a PR fixing an issue. label May 1, 2017
@Snow-Pyon Snow-Pyon added the priority: high Issues with potentially high impact that could be harmful to users. label Jan 31, 2018
@bensku bensku added priority: medium Issues that are detrimental to user experience (prohibitive bugs or lack of useful implementation). and removed priority: high Issues with potentially high impact that could be harmful to users. labels Mar 21, 2018
@bensku bensku mentioned this issue Jan 2, 2019
Closed
@bluelhf bluelhf mentioned this issue Mar 26, 2020
Closed
@bluelhf
Copy link
Contributor

bluelhf commented Mar 26, 2020

Bumping this since I just came across it, there's still nothing on this? I think it'd be better to parse arithmetic using a third-party math parser instead of reinventing the wheel

@FranKusmiruk FranKusmiruk mentioned this issue Jul 6, 2020
Closed
@TPGamesNL TPGamesNL mentioned this issue Feb 4, 2021
Merged
@TPGamesNL TPGamesNL added the PR available Issues which have a yet-to-be merged PR resolving it label Mar 23, 2021
@TPGamesNL TPGamesNL 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 May 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
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. priority: medium Issues that are detrimental to user experience (prohibitive bugs or lack of useful implementation).
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
7 participants
@bensku @ShaneBeee @Syst3ms @Snow-Pyon @skripttpbug2 @TPGamesNL @bluelhf