Throughout the backend code we change Document objects into SearchResult objects. There are good reasons for this; SearchResult has metadata attached to it and everything is set up in a format more digestible to the rest of the code. However, we aren't always consistent and I think there are corner cases where we attempt to access one type but are provided another.