Fair enough, that commit is optional. My view is that the InventoryType is a reflection of the type of the view, and that's what I had in mind when I initially put it together, but if you've decided to use it in some other way, that's fine.