macro for Sha / Galleon lock

[Voxxel]

Hello,

I'm curious how can I check if my character is saved for Sha of Anger or Galleon on a given week.
It comes to an issue when you play 7-8 characters everyday and just cannot follow which character is already locked and which one is not.

I can't find any in-built UI option to check this and this isn't included in /raidinfo list but I heard around the internet there is a macro that can tell you in a way if your current character is saved (or already looted) or not.

Could you share it please?

(I know there is a version for AILO that can track Sha but I really don't want to use one more completely addon just for this minor issue.)

[Dridzt]

/run for k,v in pairs{GetQuestsCompleted()} do if k=32099 then print("Sha done")break end print("Sha not done")end

might work for Sha.
drycoded.

[Clamsoda]

/run local z,t,s={[32099]='Sha of Anger this week.'},GetQuestsCompleted();for c,v in pairs(z) do if t[c] then s='' else s=' not' end print('You have'..s,'done',v) end

Copied from WoWHead, apparently works.

@Dridzt: Jerk <.<

[Voxxel]

Thanks a lot, guys!

[Haleth]

I've seen that Wowhead script before and I find it mind-boggling that someone would create a table just for storing one number and one string, both of which are only used once. It's not even less characters. Here's what I would do, seems to work:

/run print("You have"..(GetQuestsCompleted()[32099] and "" or " not").." done Sha of Anger this week.")

Dridzt; Your code will iterate over every key in the table until it gets to 32099, and then it doesn't actually check whether or not the value for that key is true. You can simply use the number as the key directly and then check its value.

[Dridzt]

I did warn it's drycoded (as in if there was an obvious oversight I expected someone to correct it)
To clarify I typed it from memory and assumed the non-completed quests are nil not false, so no check was necessary.

[Clamsoda]

Additionally, Galleon's Loot quest ID is "32098".

/run print("You have"..(GetQuestsCompleted()[32098] and "" or " not").." done Galleon this week.")

That will perform the same task for Galleon, using Haleth's code of course.

[Haleth]

Originally Posted by Dridzt I did warn it's drycoded (as in if there was an obvious oversight I expected someone to correct it) To clarify I typed it from memory and assumed the non-completed quests are nil not false, so no check was necessary.

You still don't have to iterate over a key-value table though, you can just use the key directly. That's what hash keys are there for after all