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] Shop now incompatible with Grief Prevention #409

Closed
1 task done
MacTh3Mac opened this issue Aug 21, 2023 · 2 comments
Closed
1 task done

[BUG] Shop now incompatible with Grief Prevention #409

MacTh3Mac opened this issue Aug 21, 2023 · 2 comments
Labels
bug Something isn't working

Comments

@MacTh3Mac
Copy link

Is there an existing issue for this?

  • I have searched the existing issues

Type of bug

Compatibility issue

Environment

- Server Version: Paper 1.20.1 (latest)
- Shop Version: 1.8.4.4

Server startup log

GriefPrevention/GriefPrevention#2134

Shop config files

GriefPrevention/GriefPrevention#2134

Error log (if applicable)

No response

Bug description

Grief Prevention now blocks the interact on shop signs.

We have attempted to have GP reverse the commit that causes the bug, however the author appears to believe that his solution is the correct one. Details here:

GriefPrevention/GriefPrevention#2134

Please amend shop to either run at at a higher priority, or... mark the sign as not editable or wax the signs.

Any of the 3 solutions will ensure ongoing compatibility.

n.b. Many other plugins signs are unaffected as they just lower their priority to LOWEST. this ensures your plugin always handles the interact first.

Steps to reproduce

click a shop sign in a grief prevention protected claim

Expected behaviour

interact should be possible to complete a purchase / sale

Actual behaviour

event is blocked by GP

Other information

No response

@MacTh3Mac MacTh3Mac added the bug Something isn't working label Aug 21, 2023
@MacTh3Mac
Copy link
Author

GP have now approved a PR GriefPrevention/GriefPrevention#2142 that resolves the issue partially. However I would suggest that future shop versions specify the iseditable tag. or potentially use a iswaxed attribute.

This would allow a greater compatibility with multiple protection software.

@OstlerDev
Copy link
Collaborator

I believe this bug has been fixed in the latest versions of Shop. Right clicking a sign does not edit it for me, however I have not tested it out with GriefPrevention. Since this is no longer an issue for you I am going to close this ticket, however if you are still having issues please feel free to log a new bug report!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants