FIXED trade order item = bag

[Bob]

granted you could say user error ,
someone uniformed and not paying attention could loose items inside a bag in you're backpack.

How?

• accept a trade order
• trade order for 15 bags
• you buy 15 bags from npc
• click on trade crate context menu - fill from backpack
  • items automatically pop into trade order crate
    • including an additional bag you had been storing rare items in
      

point = when doing trade orders, its a good idea to remove Any items you want to keep from your backpack.

 

[Dan]

I will look into excluding containers from the trade order list of required goods.

 

[Dan]

Actually, I am going to handle it like this: Containers can still appear on the list but they will be rejected if they have any items inside of them.

TradeCrateMinorFix by DocKnight86 · Pull Request #1426 · TrueUO/TrueUO

https://discord.gg/5RtDbkY3fC. Contribute to TrueUO/TrueUO development by creating an account on GitHub.

github.com

if (item is Container container && container.Items.Count > 0)
{
    // If item is a container type, and it has items inside it, reject it.
    break;
}

 

[Dan]

This issue has been fixed.

@Bob has been awarded 5 Sovereigns.