[mcagnion]

Description of the problem being solved:

Trade searches can return items that look like good upgrades but leave the build short of attribute requirements once equipped, or that the character is not yet high enough level to use.

This adds two complementary controls:

• Attribute requirements in Query Options, checked by default, adds total Strength, Dexterity, and Intelligence minimums to the generated trade search when the build is short on attributes. The minimums account for increased or reduced attributes on the build and never ask for more than the build is short by. Flasks and builds using Omniscience get no attribute minimums.
• A requirement list under Sort By in the Trader, with the same choices as the requirement filter of the unique list in the Items tab. It defaults to Any requirements, so results stay visible unless the user picks a filter:
  • Current level hides results whose level requirement is above the character level.
  • Current attributes hides results that, once equipped in the slot they would replace, still leave unmet Strength, Dexterity, Intelligence, or Omniscience requirements. Rows without an equippable slot (the Megalomaniac and Watcher's Eye rows when no jewel socket is allocated) are not filtered by attributes.
  • Current useable applies both.

The Trader now keeps the level requirement shown on the trade listing, which reflects the item's mods, instead of the base item's requirement. Items with socketed gems keep the base requirement.

Steps taken to verify a working solution:

• Added tests for the generated attribute minimums (including increased and reduced Strength), the level and attribute filters, the listing level requirement, rows without an equippable slot, all results being filtered out, and searches that return no compatible item.
• Manually checked the Query Options checkbox, and that the Trader opens and lists Watcher's Eye in a new build without an allocated jewel socket.
• Replayed 85 captured trade listings for a ring on a level 66 copy of a build. Current level kept only the listings requiring level 66 or less (42 of them, against 83 if the base item's requirement were used), Current attributes kept the four rings that cover the Dexterity the build would lose, and Current useable kept the one ring meeting both.
• Opened the Trader on a build with ten jewel sockets: the requirement list fits under Sort By, all rows stay above Total Price, and Current useable kept a ring the build can use.

Screenshots:

Query Options

Trader

[mcagnion]

Updated the PR:

• The Trader checkbox is replaced by the requirement list of the unique list in the Items tab (Any requirements, Current level, Current attributes, Current useable), placed under Sort By. Current level is new. It uses the level requirement of the trade listing, which reflects its mods, rather than the base item's.
• Fixed a layout issue from an earlier rebase of this branch: the slot rows started one row lower, so with many jewel sockets the last visible row covered Total Price.
• A search with no compatible item shows "No compatible items found for this slot." again.

The title and description now cover the level filter.