catalyst / moodle-report_coursesize

upgraded coursesize report
14 stars 19 forks source link

Missing text-qualificator #47

Open BennyJuel opened 2 years ago

BennyJuel commented 2 years ago

I have found bug. There is no text-qualificator around the coursename like the categori. So if there is a comma in the coursename it is spread over more columns when importing to Excel.

It would be nice if the size not were mentioned in GB and MB. Then it would be possible to count on the size from different categories. 10.1GB should be mentioned as 10.100.000 instead.

danmarsden commented 2 years ago

thanks - Pull requests are welcome.

Longer term we've been thinking about replacing this report with a new report builder source for Moodle 4.0 and higher which would likely help fix both of those issues too.