remove a redundant call to `recalculateTypeArraysForItem:`

This method is called by `reloadSets:` in response to the notification
on the line above.
get entry contents directly

Calling `contents` does a lot of unnecessary work to make sure the
contents iVar is updated. `reloadSets:` only runs in response the the
contents iVar being updated, so it can just be used as is.
remove a redundant notification

The QSCatalogEntryIndexed notification is sent by `loadIndex`, which is
called in a loop a few lines above.