I have been playing around with it for a bit now and there are other things you might consider in the future since this is a very nice and powerful addon!
Here are some findings to consider:
Finding Number 1: Creating "Jewel of Ancient"
SetItem="0" - This only checks if item is required to be ancient. There is no limitation to this, so if an item is ancient it already meets condition 0 and condition 1.
This is an example of a Jewel that turn ANY item into ancient.
<Item Cat="-1" Index="-1" ItemMinLevel="0" ItemMaxLevel="15" Exc="0" SetItem="0" Success="100" RewardCat="-1" RewardIndex="-1" RewardLevel="0" FailPenalty="-1" RewardSetItem="1" RewardExc="0" RewardSocket="0" />
Problem:
Once the item has been turned into ancient with the jewel, you can still spam the jewel on the same item and it will do nothing.
Solution: (change the current 1 and 0 system ) to:
1 - Any item
0 - must be regular
1- must be ancient
Finding Number 2:
Would be useful to extend <IgnoreItemList> function with special cases such as SetIntem="1" and not only Cat and index.
Example
<IgnoreItemList> <Item Cat="4" Index="7" ItemMinLevel="0" ItemMaxLevel="0" Exc="0" SetItem="1" > <Item Cat="-1" Index="-1" ItemMinLevel="0" ItemMaxLevel="0" Exc="0" SetItem="1" > </IgnoreItemList>
Why?
A. If you have item levels you can then include items such as box of luck +1,2,3,4 etc.
B. You can restrict the system from using a jewel on specific case items like Ancient items. (this also solves the first finding as you would then restrict set items )
Finding Number 3:
RewardExc="1" - Adds 1 Extra Excellent option, that is nice, but the problem is it can add the same option = wasted jewel again. In reality is a new "roll" of excellent option not addition.
Solution:
Add excellent options according to index or some way to prevent adding the same option on top of another.
Edited by zander1989
Recommended Comments