-
Notifications
You must be signed in to change notification settings - Fork 10
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
negate the condition return true #146
Comments
Does this work correctly prior to 2.X? (On 1.X)? |
I made some test, and I don't know if it's normal or not : condition negate:
check:
negate the condition
[...code]
continue Does not work: condition negate:
check:
negate the condition BUT https://skripthub.net/docs/?id=2913 , the doc say that the negate effect stop the code and return false, (I understand that), So It's a bug or an error on the doc ? |
The docs are just unclear in this case. Calling negate the condition
basically just reverses the effect that the continue effect has. The idea
behind it is that you don't have to modify your logic to implement negation.
…On Wed, Oct 23, 2019 at 3:48 AM Anarchick ***@***.***> wrote:
I made some test, and I don't know if it's normal or not :
work:
condition negate:
check:
negate the condition
continue```Does not work:
```java
condition negate:
check:
negate the condition```
BUT https://skripthub.net/docs/?id=2913 , the doc say that the negate effect stop the code and return false, (I understand that), So It's a bug or an error on the doc ?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#146?email_source=notifications&email_token=AAXTMJBRRPM46GOVD53A4EDQQATY7A5CNFSM4JC4Q7YKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOECA6WGQ#issuecomment-545385242>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAXTMJHHKZTE3AGHKHIPY33QQATY7ANCNFSM4JC4Q7YA>
.
|
according to #147 you should allow the return effect to be used |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
it seems that "negate the condition" return true and not false , so it's useless to use this expression in skript-mirrror2.0.0SNAPSHOT custom condition. (skript2.4Beta9)
The text was updated successfully, but these errors were encountered: