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

[BUG] Items with metadata are no longer detected as stock #1639

Open
6 of 7 tasks
quiquelhappy opened this issue Jul 9, 2024 · 2 comments
Open
6 of 7 tasks

[BUG] Items with metadata are no longer detected as stock #1639

quiquelhappy opened this issue Jul 9, 2024 · 2 comments
Labels
bug Something isn't working needs-tested Issue owner not reply this issue yet

Comments

@quiquelhappy
Copy link

Description

This affects a variety of items, such as

  • Spawners with metada (in our case our server uses lore to keep spawner type info)
  • Filled shulker boxes
  • ... probably more

Steps to reproduce

  • Create a shop
  • Place items with metadata on the shop

Expected Behaviour

The items should be taken into account, just like they did on QuickShop-Reremake

Screenshots

imagen
imagen

/quickshop paste URL

https://paste.helpch.at/raw/wuyexapagu

Additional Context

this along with the item exploit mentioned in #1632 are the only issues we personally found when migrating to hikari. It would be great to fix both specially now that people are migrating to the new version due to 1.21

Checklist

  • I'm running Paper or Spigot, and not a fork
  • I confirm that Paper/Spigot has been updated to the latest build
  • I confirm that QuickShop-Hikari has been updated to the latest stable version released on Modrinth (or the latest CI version)
  • I confirm that I have not read these checkboxes and therefore I just ticked them all.
  • I confirm that I'm using QuickShop-Hikari, not QuickShop-Reremake, and I'm well aware that they're maintained by different people, and that Reremake issues shouldn't be reported here.
  • I confirm that I am running a server that is not a Hybird Server, (e.g. Mohist, Magma, CatServer, Banner, etc.), and I am aware that QuickShop-Hikari may not function properly on a Forge/Fabric hybrid server, and I am running at my own risk on such a server program, and I am aware that the I run such server-side programs at my own risk and know that I will not receive any support or help for this behavior.
  • I am well aware that if the Issue Ticket is not filled out correctly and completely, it will simply be closed without any response or reason.
@YuanYuanOwO YuanYuanOwO added the bug Something isn't working label Jul 12, 2024
@creatorfromhell
Copy link
Contributor

creatorfromhell commented Jul 13, 2024

Looking at this, will probably add a new option for checks to this that I've been working on for another plugin

@creatorfromhell
Copy link
Contributor

creatorfromhell commented Jul 22, 2024

Description

This affects a variety of items, such as

  • Spawners with metada (in our case our server uses lore to keep spawner type info)
  • Filled shulker boxes
  • ... probably more

Steps to reproduce

  • Create a shop
  • Place items with metadata on the shop

Expected Behaviour

The items should be taken into account, just like they did on QuickShop-Reremake

Screenshots

imagen imagen

/quickshop paste URL

https://paste.helpch.at/raw/wuyexapagu

Additional Context

this along with the item exploit mentioned in #1632 are the only issues we personally found when migrating to hikari. It would be great to fix both specially now that people are migrating to the new version due to 1.21

Checklist

  • I'm running Paper or Spigot, and not a fork
  • I confirm that Paper/Spigot has been updated to the latest build
  • I confirm that QuickShop-Hikari has been updated to the latest stable version released on Modrinth (or the latest CI version)
  • I confirm that I have not read these checkboxes and therefore I just ticked them all.
  • I confirm that I'm using QuickShop-Hikari, not QuickShop-Reremake, and I'm well aware that they're maintained by different people, and that Reremake issues shouldn't be reported here.
  • I confirm that I am running a server that is not a Hybird Server, (e.g. Mohist, Magma, CatServer, Banner, etc.), and I am aware that QuickShop-Hikari may not function properly on a Forge/Fabric hybrid server, and I am running at my own risk on such a server program, and I am aware that the I run such server-side programs at my own risk and know that I will not receive any support or help for this behavior.
  • I am well aware that if the Issue Ticket is not filled out correctly and completely, it will simply be closed without any response or reason.

Try the jar in this zip: QuickShop-Hikari-6.2.0.6.zip and set work-type to 3.

@YuanYuanOwO YuanYuanOwO added the needs-tested Issue owner not reply this issue yet label Oct 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working needs-tested Issue owner not reply this issue yet
Projects
None yet
Development

No branches or pull requests

3 participants