ItemKeepDecision Rework

[DysfunctionaI]

When used on an unidentified item:

item.KeepDecision == ItemKeepDecision.LooksGood

'CalculateItemKeepDecision' will always return 'LooksGood', regardless if the item actually needs to be identified to evaluate any pickit lines.

How it should work:

If the item is unidentified, return true against any pickit line that specifies affixes where the item needs to be identified first. This preserves the idea that it "Looks Good" at the time. and can be re-calculated once the item is identified (against additional pickit lines, if any).

The result is that you could use the pickit files on unidentified items, which would be very handy when they first drop on the ground, or if they're in your inventory, eliminating the need to pick them up, or even identify them.

Example pickit line when used on an unidentified item:

+1h = name=In-geom & *only_ancient*

Expected: LooksGood
Returns: LooksGood

-1h = name=In-geom

Expected: DefinitelyBad
Returns: LooksGood