Escaping CSV Headings to fix the case of a heading including a comma

mohabusama · mohabusama · commit b6a22cc764d7 · 2013-05-27T23:08:12.000+02:00
diff --git a/application/libraries/Format.php b/application/libraries/Format.php
@@ -197,7 +197,7 @@ public function to_csv()
 			$data = array($data);
 		}
 
-		$output = implode(',', $headings).PHP_EOL;
+		$output = '"'.implode('","', $headings).'"'.PHP_EOL;
 		foreach ($data as &$row)
 		{
 			$output .= '"'.implode('","', $row).'"'.PHP_EOL;

Original file line number	Diff line number	Diff line change
`@@ -197,7 +197,7 @@ public function to_csv()`
`197`	`197`	`$data = array($data);`
`198`	`198`	`}`
`199`	`199`
`200`		`- $output = implode(',', $headings).PHP_EOL;`
	`200`	`+ $output = '"'.implode('","', $headings).'"'.PHP_EOL;`
`201`	`201`	`foreach ($data as &$row)`
`202`	`202`	`{`
`203`	`203`	`$output .= '"'.implode('","', $row).'"'.PHP_EOL;`