Originally Posted by Calamath
Yes, we should use the global constant MAX_JOURNAL_QUESTS (=25) instead as the loop count.
And for each index, you need to check if it is valid or not. 
- Calamath |
Originally Posted by sirinsidiator
I moved it to the general discussion forum, since it may help other authors in the future.
|
Thx both.
However, there still might be a little bug, cause the "can't share these quests" point is still valid.
It was not a made-up point to reinforce the argument, i actually had this a couple times.
An idea is, that even ZOS is doing a "1..GetNumJournalQuests()" for the sharing-feature. That would explain why sometimes you just can't share a quest, and after delete-and-regain that one, you can.
There are some other points, which can be confusing, such as some quests can't be shared to someone who has not done needed pre-quests (like 2 delve-dailys in Wrothgar) or you can't share to someone who has already done it that day, or has the same kind of quest already in his journal.
But i think, the importance of this case approaches zero, especially when we have a "way around"... (1..MAX_JOURNAL_QUESTS)
