Prevents an emptyView being created twice for a CollectionView

Fixes #1662
a call to `checkEmpty()` in `this.destroyChildren();`

Of the 2 evils, removal of `showEmptyView` here has the least impact. There's a lot of `CollectionView` riding on `checkEmpty()`