After looking through the nodrop checks, it looks like in most places, it's just reading the pointer to the base item flag directly.
It seems like it would make sense to create an IsNoDrop() function inside the ItemInst class to check both flags. Then replace all the existing nodrop checks throughout ... as opposed to adding an instance no drop check each time a normal no drop check is done.
Also, any thoughts on how it worked when you looted an attuneable item on Live ? .. If the item auto equipped, did the "Are you sure you want to attune?" popup window work?
It seems like there was some odd issue with attuneable items back then, but It's been too long.
As usual, there's probably a function somewhere I missed and KLS will rewrite the whole thing in one line.
|