[sourdiesel]

Small bug fix for counting weapons/medals, and losing minion due to death.

This fixes an issue where you have enough medals to be counted as complete, but you did them out of order.
This was due to how the counts are being done in a top to bottom priority, I'll go into more detail below
e.g. Komasan (7/7) and Bizzaria (9/9) were done first and then I started on Jibanyan, once Jibanyan got over 7/11, and being at the top of the list it started acting up.

To those still having issues with medal counts. Let me try to explain and hopefully not confuse the hell out of you ha!

I'll be the first to admit that this addon is not perfect. It was rushed trying to get it out while it would still be of some use to people and testing was limited. And being how shortlived this addon will be I'm not sure how much more time i really should spend on it, but at the same time i don't want to leave anyone hanging, ooh~ the agony lol

First off, when this module is counting medals and weapons it is going down the list top to bottom.
I wrote this with the expectation that it would just be AFK'd, and AFk'ing on my main, doing them all in order, it worked flawlessly.
I was done on my main, and working on a 2nd alt before I even decided to share the addon.

The issues seem to be happening when things are done in a random order.

TL;DR
So when it comes down to it. When you have enough medals of a certain minion to buy a weapon and havent yet.

• If you do not want it to count the minion and increment the weapon cost, uncheck the minion
  

• If you want it increment the weapon count as if you already have the weapon leave it minion checked, and if the minion is further down the list than the one you want to farm, you need to also check off its weapon
  

I ran some tests on a 3rd alt last night to check some things out.

Here is the scenario I tested with last.

I started with Komasan, and after getting a few medals for minion turn-ins i went and picked up Blizzaria, Jibanyan and Manjimutt.

So the initial test started with just Komasan and Blizzaria being checked as acquired.

It gathered 7/7 Komasan as it should a moved to Blizzaria. When Blizzaria got to 8/9 i added in Manjimutt and Jibanyan.
Once it got 9/9 of Blizzaria it moved to Jibanyan. Now here is where the confusion starts with how its counting medals and weapons with a top to bottom priority.

Since I had no weapons checked off, when it got to 7/11 on Jibanyan it changed over to Komasan again.

Komasan was the first one I did, so it was at 7/7, but since i was holding on to these medals and had no weapons checked as being acquired once Jibanyan got to 7/11 it wanted to count Jibanyan as having enough medals for the first weapon and changed Komasan to 7/11. You may be thinking, shouldn't this be 7/9. But since i already had 9/9 of Blizzaria the weapon count was getting incremented.

To resolve this i had to go ahead and check Komasan's and Blizzaria's weapons acting like i had already purchased them.

Let me try and break this down a little better

Example:
Minions checked are

• Jibanyan (9/11)
  
• Komasan (7/7)
  
• Blizzaria (9/9)
  

With no weapons checked off, it will start at the top of the list with a Weapon Count of 0

Counts Jibanyan,  since it's the first in the list its just looking for 7. We have 9 so it counts it as being done (Weapon Count = 1)
Counts Komasan, since Jibanyan was counted as complete, its looking for 9. We only have 7 so it flags this as being incomplete.
Counts Blizzaria, since Komasan was marked incomplete, its only looking for 9 here also. We have 9 so it counts it as complete. (Weapon Count = 2)

Now since Komasan was the only one marked as incomplete, and we have 2 other minons marked as complete. It will change Komasan to 9/11 and start trying to farm again.
Admittingly not being what we thought it would do, but is technically correct being that it found 3 minons, 2 complete and 1 incomplete (7, 9, 11)

So now to correct this, you need to check off the weapons for the minions you know you have enough medals for.

Weapons i checked here are

• Komasan (7/7)
  
• Blizzaria (9/9)
  

Now when Yo-Kai mode is started, it counts 2 weapons we have selected and starts farming Jibanyan (9/11) just as expected.

Module update posted in original post.