RetributionEQ / quests

MIT License
0 stars 11 forks source link

Tradeskills - Non Consumable Items #22

Closed RetributionEQ closed 1 year ago

RetributionEQ commented 1 year ago

When returning non-consumable tradeskill items (ie. Chisel, Hammer, etc.) players receive RC/Apoc versions of those items as well. Causing them to have to sell them and buy new ones or disenchant them.

What is the cleanest approach to resolve? If we just deleted the item entries for the RC/Apoc versions, would it return nothing or would it know to return the base item?

xackery commented 1 year ago

Deleting the RC/Apoc versions would indeed fix the return item upgrade on TS combine, you'll have to do an update on character inventory too though for those having them (or they may get their rc/apoc versions deleted as invalid items)

For example, Skinning Knife : 21565 RC Skinning Knife: 721565 Apoc Skinning Knife: 821565 e.g.

This unfortunately will be followed any time someone nags about it.

RetributionEQ commented 1 year ago

Just to confirm, in the example above it would be:

DELETE FROM items WHERE id = 821565 or id = 721565;

not

DELETE FROM items WHERE id = 21565 or id = 721565;

xackery commented 1 year ago

Just to confirm, in the example above it would be:

correct, updated my example to confirm

RetributionEQ commented 1 year ago

Using Xackery's suggested updates, all four items have been addressed.