The is_selected inspection is on the data item side -- it is an attempt to be flexible for different ways that people might want to code their model data, for using SelectableDataItem, for declaring the boolean, or for defining a function. If it is always OK to require that data items subclass or mix in SelectableDataItem, then yes, that makes it easy to just delete the is_selected stuff.