Add WhosAFK integration #6
Conversation
Add a method that checks if a player is AFK. For now we can just check whether WhosAFK has added the player to the afkers team, in the future the method could do a number of checks to establish whether the player has been set AFK by more plugins.
|
@Phoenix616 Any feedback would be appreciated. I have tried to keep this change simple and without drawbacks, but I am open to suggestions. :) |
|
You should directly use the methods provided by the plugin to check whether or not a player is afk and also make sure that the plugin is actually running on the server. I'm closing this pr in favour of a more clean, future version. (Which shouldn't unnecessarily move the plugin.yml and config.yml to another location) |
|
Thanks for your feedback. Moving the yml files wasn't meant to be committed, my mistake! WhosAFK only exposes events for when a player becomes AFK and leaves AFK, so IMO this is a better solution than the one in my initial (separate) plugin. If you were to use the events exposed by WhosAFK, you would have to keep track of which players are currently AFK. This may be needed anyway in the future to integrate with other AFK plugins, but seems unnecessarily complicated for this case. |
|
It looks like you should be able to get whether or not a player is afk with
WhosAFK.playerIsAFK(Player p).
|
This adds a very simple method that checks if a player has been added to the
afkersteam (as done by WhosAFK).It should be fairly simple to extend the method in the future to check other AFK plugins too, maybe even decide which AFK plugins to check in the config.
See issue #4