Uses enum to specify if AccountsIndexIterator should return items sorted or not

[brooksprumo]

Problem

The collect_all_unsorted field on the accounts index iterator is sometimes cumbersome due to (1) it being a bool, and (2) it feeling backwards/negative from its true/false meaning.

Summary of Changes

Use an enum instead. This way it is self documenting.