Incorrect state when RCing fast

[Infinitay]

It seems like the bug came back because when I was in GOTR RCing fast, two pouches would track properly, but then the other two ended up being wrong

[Infinitay]

• Interacting with altar feels like you can do each action faster than a tick, but it seems that it's on every tick
• If you empty your pouch too fast, and there is essence in your inventory already with free space, that emptied essence will "group" up and craft together if you do the empty action
  • Of course, you have to re-interact with the altar after emptying again, so it seems like it will take precedence
  • Maybe this proves that altar crafting is every tick
  • Or is it such that the first altar interaction you wait until the next game tick, but then once you get into that cycle you can freely dump your pouches and craft as long as they happen on the same ticks as you craft and not break the cycle afterwards?

[Infinitay]

I suppose this is partially why Adam used ticks with his similar PouchActions class; however, his plugin had tick issues too - even inside the bank when filling pouches. It was even worse when emptying and crafting. I just tried it yet again to confirm once more, and all 4 pouches were inaccurate versus just my last two (medium and large) which was repeated in the image above.

I'm thinking of a few courses of action

1. Start manually tracking when crafting since currently that's where the problem lies
2. Experiment with sending ticks on actions and see if we need to be blocking/clearing and queues and other variables based on the first action in the queue and it's tick time
   • However, what if it's only that one action with the outdated tick? We'd be clearing the whole queue which is not right
   • Could cause issues with current implementation because if it allows onItemContainerChanged to run while we are still manipulating with the queue, there could be issues. Will have to experiment to confirm non-breaking
   • Could use a PriorityQueue with tick being the prio, but they shouldn't be out of order anyways

I'll begin working on manually tracking since it seems viable currently and go in hand with the rest of the spaghetti I've created

• Listen for the experience drop -> Clear all essence from inventory
• Verify we should listen for the experience if the user action was to "Craft-rune" on an "Altar"
  • Actions could vary based on altar so just confirm
    • Don't worry about the different blood altars. The different one in Kourend uses dark essence fragments which we can't store in the pouch anyways, and the option is "Bind"
    • Ignore soul altar too because that also uses dark essence fragments

[Infinitay]

I pushed a PR #16 that greatly mitigates if not resolves this issue. I'll be testing it more tomorrow and hopefully accepting the PR and merging it onto master.

[Infinitay]

This has been resolved by #13 from my testing for a few hours at GOTR and < an hour normal RCing