The above line actually returns a MutableBlockPos, which is reused by the iterator. This means that the locations being stored are changed in subsequent steps of the loop, and will inevitably include non-chorus blocks, causing the logic to fail. Would suggest getting an immutable version of the object for storage in the lists.