Originally Posted by Eommus
@MunkDev, the current code stores only the newly acquired items, so, I will note your suggestion about C_NewItems, but I will not use it for the moment.
|
It seemingly only stores the items you acquire because you're overwriting the table entry for each existing item on every BAG_UPDATE. This isn't the same as tracking the items you acquire, you are simply tracking the items you have. What this also means is that since you're not wiping the log on every update, you'll also keep any items that you already have logged but no longer possess.
It's unclear whether you want your addon to track items that you have acquired but no longer possess or just the items you currently have. Logging the items you acquire can mean both of these things and using C_NewItems is certainly not necessary in the latter case. If you, as an example, want to track how many of a particular item you've acquired over time then your current solution would not work.