QuestInfo Bug

[@AlphaLemming]

Did anyone encounter this too?

GetJournalQuestInfo(questId) returns a empty array if on a german client and has crafting quest Alchemy writ with completed steps?

[votan]

Originally Posted by @AlphaLemming Did anyone encounter this too? GetJournalQuestInfo(questId) returns a empty array if on a german client and has crafting quest Alchemy writ with completed steps?

Yes. Wandamey figured out how it's correct:

for questIndex = 1, MAX_JOURNAL_QUESTS do
    if IsValidQuestIndex(questIndex) then
      questType = GetJournalQuestType(questIndex)
...

Because the questIndex does not change, if you complete another quest with a lower index.

[merlight]

Uh, so when I'm iterating for i = 1, GetNumJournalQuests() I'm doing it wrong? Because that function returns the number of active quests, but they can be index 5, 7 and 13?

[Fyrakin]

Originally Posted by merlight Uh, so when I'm iterating for i = 1, GetNumJournalQuests() I'm doing it wrong? Because that function returns the number of active quests, but they can be index 5, 7 and 13?

Quest index is not a quest ID. It is just an order representation number for the stock quest tool. So, you have to check the number if its valid before doing anything else.

[votan]

Originally Posted by merlight Uh, so when I'm iterating for i = 1, GetNumJournalQuests() I'm doing it wrong? Because that function returns the number of active quests, but they can be index 5, 7 and 13?

Yes. That was a trap in Potion Maker. It was reported, that the writ potion was not found. After Wandamey told me, that the index does not change and I should use MAX_JOURNAL_QUESTS, I found that IsValidQuestIndex function in ESO code.

Apparently the only purpose of GetNumJournalQuests is to display that "Quests : num/25"

but the indexes change as soon as you complete a new step of any quest i think (i didn't exactly figured that one), which made this hard to detect. (not to mention that getting no nil error when iterating an empty array is a bit counterintuitive)

I found an even easier way to iterate recently but never used it and i didn't keep the test, maybe something along getnextquest. But anyway, it's just a 25 lines array.

[@AlphaLemming]

What a mess, thanks all for the informations!