[REF] XMLProcessor/Report - Extract method to make it more testable

[demeritcowboy]

Overview

I want to write a test for this but currently I'd have to compare against the full output html.

Technical Details

The way github displays this is a little weird but all I did was pull out everything in getCaseReport except the last two lines and put it into a new function _getCaseReport, and then call that from getCaseReport. I find it easier to see if you click the expandy icons near the line numbers on the left so that you can see it without a break in the middle.

Comments

I can't include the test yet in this PR because it fails for two reasons, one of which is what I was originally looking to fix, and the other is coming next.

[civibot]

(Standard links)

• If this is your first pull-request for CiviCRM, please browse CONTRIBUTING.md for information about the development and testing processes.
• If you are reviewing this pull-request, you may wish to consult the test sites and the Review Standards (long template, short template).

[demeritcowboy]

I think test fail is unrelated but jenkins retest this please.