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.

Sign up for GitHub

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

Jump to bottom

IllegalPluginAccessException when using EvtByReflection for non-event classes #39

Closed
FranKusmiruk opened this issue Mar 26, 2021 · 1 comment
Closed

IllegalPluginAccessException when using EvtByReflection for non-event classes #39

FranKusmiruk opened this issue Mar 26, 2021 · 1 comment
Labels
bug Something isn't working good first issue Good for newcomers

Comments

@FranKusmiruk
Copy link
Member

Describe the bug
As the title says, an IPAE is thrown when a class that doesn't implement Event is used as input.

To reproduce

import:
  java.util.function.Consumer


on Consumer:
  broadcast "%event%"

Expected behavior
I would expect the addon to do type-checking before casting or passing around the input, and throw an error if applicable.
@TPGamesNL TPGamesNL added the bug Something isn't working label Mar 26, 2021
TPGamesNL added a commit that referenced this issue Mar 26, 2021
@TPGamesNL
Make EvtByReflection check if input class is a Bukkit event (issue #39)
8e024e2
@TPGamesNL TPGamesNL added the good first issue Good for newcomers label Mar 26, 2021
@TPGamesNL
Copy link
Member

Fixed in 8e024e2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working good first issue Good for newcomers
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@TPGamesNL @FranKusmiruk