Unnecessary expensive queries whenever the contribution selector is displayed

When doing a query that results in the contribution selector being displayed it doesn't make sense to add a whole lot of expensive joins and retrieve large numbers of fields if we then don't display the result.

When fields are not directly specified the 'defaultReturnProperties' are retrieved. Unfortunately these properties have been added onto over the years to include the properties that are required for specific searches in specific parts of the code. For example batch information is not relevant to the main contribution search screen. The batch screen should set it's own return properties - rather than contaminating the main array.