MDL-37834 assign: avoid newlines in status column of csv
authorDan Poltawski <dan@moodle.com>
Wed, 18 Nov 2015 16:45:31 +0000 (16:45 +0000)
committerDan Poltawski <dan@moodle.com>
Fri, 20 Nov 2015 18:36:30 +0000 (18:36 +0000)
Previously we were generating a CSV file with newlines in it, which
seems to cause problems importing into some versions of MS Excel.

Instead seperate status lines by '-' - this column is only used to
provide context to the teacher in the spreadsheet.

mod/assign/gradingtable.php

index 6ffecb8..5ca0f11 100644 (file)
@@ -972,7 +972,7 @@ class assign_grading_table extends table_sql implements renderable {
         }
 
         if ($this->is_downloading()) {
-            $o = strip_tags(str_replace('</div>', "\n", $o));
+            $o = strip_tags(rtrim(str_replace('</div>', ' - ', $o), '- '));
         }
 
         return $o;