Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Wool comparing problem #3669

Closed
DahunDev opened this issue Jan 1, 2021 · 3 comments
Closed

Wool comparing problem #3669

DahunDev opened this issue Jan 1, 2021 · 3 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.

Comments

@DahunDev
Copy link

DahunDev commented Jan 1, 2021

Description

Steps to Reproduce

When write the code as below:

command /wooltest [<text>]:
	usage: /wooltest 
	trigger:
		if player has 64 of orange wool:
			remove 64 of orange wool from the player
			give player yellow wool named "&atest"

If player has 64 of orange wool and 64 of yellow wool, and then type "/wooltest" two times,
the yellow wool is removed from inventory.

Expected Behavior

If the player types "wooltest" 2 times, it should not remove yellow wool as yellow wool is not orange wool.

It seems problem at loading aliases or checking item that has data of such as "{Damage:1}"

It seems the might be the same problem as #3668

This problem does not happen in 2.3.6.

Errors / Screenshots

Server Information

  • Server version/platform: Paper version git-Paper-1618 (MC: 1.12.2) (Implementing API version 1.12.2-R0.1-SNAPSHOT)
  • **Skript version:**2.5.3

Additional Context

It seems related problem as #3668

@EtoNikei
Copy link

EtoNikei commented Feb 2, 2021

Recommend using types of items with many data values. (e.g if player has type of orange wool -> remove orange wool from player's inventory)

@DahunDev
Copy link
Author

DahunDev commented Feb 2, 2021

Recommend using types of items with many data values. (e.g if player has type of orange wool -> remove orange wool from player's inventory)

I already uses as orange wool .

@APickledWalrus
Copy link
Member

Seems to be fixed with #4162

@APickledWalrus APickledWalrus added bug An issue that needs to be fixed. Alternatively, a PR fixing an issue. PR available Issues which have a yet-to-be merged PR resolving it labels Jul 21, 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 Oct 30, 2021
# for free to join this conversation on GitHub. Already have an account? # 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.
Projects
None yet
Development

No branches or pull requests

4 participants