Thanks for the extra testing. Sounds like we should be all set to get this added in.
If the client is deleting lore items, it would be nice to know which one it will delete (the existing/old one(s) or the newly acquired one). Either way, it sounds like we just need to make sure that the scenario of duplicate lore items never happens such as this case. Makes me wonder if there may be other cases of this out there as well. I haven't seen any zone crashes in relation to augmenting items in quite a while, but I have definitely had reports of item loss while augmenting. This fix sounds like it should resolve the issue with the new inventory based aug sealer, but we may still have more aug code changes needed to really remove the possibility of item loss during augmentation.
I noticed a few things in the augmenting code that could probably use some rewriting. Some of it stood out as potential issues, but it will take another closer look to make sure changes are required. Maybe I will do a rewrite of some of it when I have some time.
|