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

Internal error while trying to execute this command. #3198

Closed
gagedevs opened this issue Jul 23, 2020 · 10 comments
Closed

Internal error while trying to execute this command. #3198

gagedevs opened this issue Jul 23, 2020 · 10 comments
Assignees
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: low Issues that are not harmful to the experience but are related to useful changes or additions.

Comments

@gagedevs
Copy link

gagedevs commented Jul 23, 2020

Description

I tried to make a /sell command and I got the error on multiple servers including my localhost test server and a Minehut server. I got the error on both servers, and I tried disabling my addons and still didn't work.

Steps to Reproduce

https://pastebin.com/e5DVvhjP (Code)

Expected Behavior

I wanted it to sell all the ores in the player's inventory by setting variables to the amount of that item in the player's inventory. Then, it would add them all together and add them to the temp. variable {_made} so I can send how much they made and then add that temp. variable to {m::%player%} (their balance's variable)

Errors / Screenshots

https://prnt.sc/tniv8x

Console: https://gist.github.com/oHeckGage/9132dda272a506d5e410005e3172fed8

Server Information

I've used a test server and Minehut:

MH: 1.16 TS: 1.16
MH Skript Version: 2.4.1MH (Idrk if that's the "real" version) TS Skript Version: 2.5-alpha6(rev1) and 2.4.1
(2.4.1 is the latest stable version)

Additional Context

@ShaneBeee
Copy link
Contributor

Duplicate of #2752

@ShaneBeee ShaneBeee marked this as a duplicate of #2752 Jul 23, 2020
@ShaneBeee
Copy link
Contributor

err wait, I was thinking something wrong here.

@ShaneBeee ShaneBeee reopened this Jul 23, 2020
@ShaneBeee
Copy link
Contributor

Please provide test results from our fork of Skript, not another persons fork.

@ShaneBeee ShaneBeee added the waiting for reply The report needs a response from the reporter to determine course of action. label Jul 23, 2020
@gagedevs
Copy link
Author

Dunno what you mean, I got Skript.jar from github.com/SkriptLang/Skript/releases I think, am I supposed to download it from somewhere else?

@APickledWalrus
Copy link
Member

He meant if you could send the error report that you get using THIS fork, instead of Minehut’s.

@gagedevs
Copy link
Author

Oh sure

@gagedevs
Copy link
Author

@APickledWalrus
Copy link
Member

Thanks! Your code should error, but for some reason it isn’t. I’m assuming you’re trying to find the total amount of those materials, so you should add them together instead of putting them in a list.

e.g. set {_made} to {_iron} + {_gold} + ...

I believe this is what you are trying to do 🙂

We will keep this issue open for now though since it is technically a bug

@APickledWalrus APickledWalrus added bug An issue that needs to be fixed. Alternatively, a PR fixing an issue. and removed waiting for reply The report needs a response from the reporter to determine course of action. labels Jul 24, 2020
@gagedevs
Copy link
Author

Yes it is what I wanted, I will see if that fixes it and see if there are errors!

@gagedevs
Copy link
Author

Thanks! It works!

@APickledWalrus APickledWalrus added the priority: low Issues that are not harmful to the experience but are related to useful changes or additions. label Jul 30, 2020
@APickledWalrus APickledWalrus self-assigned this Oct 4, 2020
@ShaneBeee ShaneBeee added the completed The issue has been fully resolved and the change will be in the next Skript update. label Oct 22, 2020
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. priority: low Issues that are not harmful to the experience but are related to useful changes or additions.
Projects
None yet
Development

No branches or pull requests

3 participants