danmarsden
commented
2 years ago yeah - code's getting a bit messy, but I think this is a nice UX improvement so lets merge it as-is for now and one day we'll use reportbuilder and paginate the main report/shifting it into a class for rendering etc.
The patch is a bit awkward because it divides processing of the course size data and the user size data using
if ().. else if ()...statements. However, it retains the original processing flow, which ensures the minimal changes in the logic.