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

Remove chest protection inventory check #977

Merged
merged 1 commit into from
Mar 20, 2022

Conversation

Pugzy
Copy link
Contributor

@Pugzy Pugzy commented Mar 20, 2022

To replicate, join a team, start the match and try to change your inventory within the first 2 seconds, the event is cancelled.

For the first 2 seconds of a match non-player inventory interactions are cancelled with the aim of preventing modifications with chests for some unknown reason (maybe a bug of the past). The way this check is written currently prevents regular player inventory actions as the inventory type of a player is set as CRAFTING (a legacy (craft/sport)bukkit issue) this causes the event to cancel as it does not match the allowed types of PLAYER and CREATIVE.

This check is not present within the OvercastNetwork ProjectAres final release and was removed in the same way as this commit. The inventory interact event check aimed mainly at observers which is mixed in with this random chest feature is also moved to the EventFilterMatchModule with other similar purpose listners.

Signed-off-by: Pugzy [email protected]

@Pugzy Pugzy requested a review from Electroid as a code owner March 20, 2022 17:14
@Electroid Electroid merged commit 963f02a into PGMDev:dev Mar 20, 2022
applenick added a commit to applenick/PGM that referenced this pull request Mar 25, 2022
applenick added a commit to applenick/PGM that referenced this pull request Apr 21, 2022
applenick added a commit to applenick/PGM that referenced this pull request Apr 22, 2022
applenick added a commit to applenick/PGM that referenced this pull request May 13, 2022
applenick added a commit to applenick/PGM that referenced this pull request Jun 8, 2022
applenick added a commit to applenick/PGM that referenced this pull request Jun 12, 2022
applenick added a commit to applenick/PGM that referenced this pull request Jun 26, 2022
applenick added a commit to applenick/PGM that referenced this pull request Jul 17, 2022
applenick added a commit to applenick/PGM that referenced this pull request Jul 20, 2022
applenick added a commit to applenick/PGM that referenced this pull request Jul 23, 2022
applenick added a commit to applenick/PGM that referenced this pull request Aug 1, 2022
applenick added a commit to applenick/PGM that referenced this pull request Aug 3, 2022
applenick added a commit to applenick/PGM that referenced this pull request Aug 4, 2022
applenick added a commit to applenick/PGM that referenced this pull request Sep 3, 2022
applenick added a commit to applenick/PGM that referenced this pull request Sep 13, 2022
applenick added a commit to applenick/PGM that referenced this pull request Sep 18, 2022
applenick added a commit to applenick/PGM that referenced this pull request Sep 23, 2022
applenick added a commit to applenick/PGM that referenced this pull request Sep 24, 2022
applenick added a commit to applenick/PGM that referenced this pull request Sep 24, 2022
applenick added a commit to applenick/PGM that referenced this pull request Sep 24, 2022
applenick added a commit to applenick/PGM that referenced this pull request Oct 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants