[Done] Async Events

[MrNemo64]

Right now the events don't specify if they are sync or async but the economy response is defined to be async.
When a consumer triggers an operation that may trigger an event, does the event run sync or async? Should we consider this?

[lokka30]

My current opinion: I think the economy provider should be able to choose which thread they would like to use. Ideally not the main thread, but I don't mind if they choose it.

[MrNemo64]

Maybe we should add an isAsynchronous() method?

[lokka30]

Is there an easy way for us to tell if the event is on the main thread?

[MrNemo64]

There is Bukkit#isPrimaryThread() but I don't know if it checks that you are on the main thread or on a thread that is synchronized with the main thread

[MrIvanPlays]

Thats an easy one, add the async parameter into the constructors. An economy provider may choose to call them sync.

[MrNemo64]

That was my first idea and probably the best one

[lokka30]

Sounds great. Assuming it still has the isAsynchronous method.

[MrNemo64]

Sounds great. Assuming it still has the isAsynchronous method.

Yes, and would return the boolean

[MrIvanPlays]

Bukkit event already has an async parameter. Just a constructor parameter and calling the super constructor is enough.

[MrIvanPlays]

5a9b94a