SpoutDev/SpoutPlugin — pull request #15 by Snaipe (2012-02-27 13:30:15+00:00)
Title: Removed the normal click handling when the event's result is set to Result.ALLOW

As I said here : https://github.com/SpoutDev/Spout/issues/1627#issuecomment-4193127.

Setting the InventoryClickEvent's result to Result.ALLOW should mean that the results of the click are not behaving the same way as Minecraft does.
Therefore, its handling should be manual, not assisted in any ways by the netServerHandler, as it would end in having unexpected results
The actual code simulates a normal click before updating the stacks, and as such, will never create the expected result if you set manually the cursor and the clicked stack - which is the whole point of setting the result to ALLOW.