Skip to content

Fix remaining offline visibility checks - #5466

Merged
JRoy merged 1 commit into
EssentialsX:2.xfrom
JRoy:stupid-fork
Aug 5, 2023
Merged

Fix remaining offline visibility checks#5466
JRoy merged 1 commit into
EssentialsX:2.xfrom
JRoy:stupid-fork

Conversation

@JRoy

@JRoyJRoy commented Aug 5, 2023

Copy link
Copy Markdown
Member

Purpur added a new self-referencing method that
we don't implement with our stub class. This will
prevent Player#canSee from being called for offline players (it would be false anyway).

Mitigates #5318 and #5465
Continuation of a1fa1e3

Purpur added a new self-referencing method that
we don't implement with our stub class. This will
prevent Player#canSee from being called for offline
players (it would be false anyway).
@JRoyJRoy added type: bugfix PRs that fix bugs in EssentialsX. module: main Issues or PRs for the main Essentials module labels Aug 5, 2023
@JRoyJRoy added this to the 2.20.1 milestone Aug 5, 2023
@JRoy
JRoy merged commit d7e5c10 into EssentialsX:2.xAug 5, 2023
@JRoy
JRoy deleted the stupid-fork branch August 5, 2023 15:44
Starmism pushed a commit to valence-smp/Essentials that referenced this pull request Jan 8, 2025
Purpur added a new self-referencing method that
we don't implement with our stub class. This will
prevent Player#canSee from being called for
offline players (it would be false anyway).
rewsdjad1 pushed a commit to rewsdjad1/EssentialsXPlus that referenced this pull request Aug 13, 2026
Purpur added a new self-referencing method that
we don't implement with our stub class. This will
prevent Player#canSee from being called for
offline players (it would be false anyway).
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

module: mainIssues or PRs for the main Essentials moduletype: bugfixPRs that fix bugs in EssentialsX.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@JRoy@mdcfe