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

"contains" is not working properly. #4046

Closed
startree236 opened this issue Jun 1, 2021 · 5 comments
Closed

"contains" is not working properly. #4046

startree236 opened this issue Jun 1, 2021 · 5 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

@startree236
Copy link

startree236 commented Jun 1, 2021

Description

"contains" is not working properly.

Steps to Reproduce

Several attempts were made using temporary variables, etc.

Expected Behavior

if mid contain "_" or "기본공격" or "비약" or "기타-도약" or "피버" or "신의$축복":
		stop

The above syntax doesn't work.

Errors / Screenshots

Server Information

  • Server version/platform: The server version is 1.12.2 and I am using Paper-1618.
  • Skript version: The script version is using 2.6-alpha1 .

Additional Context

@vjh0107
Copy link

vjh0107 commented Jun 1, 2021

it's removed at skript version 2.6, you may use loop syntax to Implement similarly

@TPGamesNL
Copy link
Member

it's removed at skript version 2.6, you may use loop syntax to Implement similarly

The contains condition was not removed, two syntaxes from it where removed:
image

@vjh0107
Copy link

vjh0107 commented Jun 1, 2021

My mistake, that's what I meant.

@TPGamesNL TPGamesNL added bug An issue that needs to be fixed. Alternatively, a PR fixing an issue. priority: medium Issues that are detrimental to user experience (prohibitive bugs or lack of useful implementation). labels Jun 1, 2021
@TPGamesNL TPGamesNL added the PR available Issues which have a yet-to-be merged PR resolving it label Jun 1, 2021
@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 9, 2021
@Zzheyd
Copy link

Zzheyd commented Jun 10, 2021

It's same for worlguard's region?
For exemple, -> if "%region at player%" contains "one" or "two" or "three":
?

@TPGamesNL
Copy link
Member

It's same for worlguard's region?
For exemple, -> if "%region at player%" contains "one" or "two" or "three":
?

Yep, that's the same issue

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: medium Issues that are detrimental to user experience (prohibitive bugs or lack of useful implementation).
Projects
None yet
Development

No branches or pull requests

6 participants