mirror of
https://github.com/moodle/moodle.git
synced 2025-04-20 07:56:06 +02:00
course/import/groups: MDL-20876 Fix split() call and remove redundant escaping.
This commit is contained in:
parent
caf0021225
commit
d44bdfb068
@ -72,15 +72,13 @@ die;
|
||||
} else {
|
||||
$csv_encode = '/\&\#44/';
|
||||
if (isset($CFG->CSV_DELIMITER)) {
|
||||
$csv_delimiter = '\\' . $CFG->CSV_DELIMITER;
|
||||
$csv_delimiter2 = $CFG->CSV_DELIMITER;
|
||||
$csv_delimiter = $CFG->CSV_DELIMITER;
|
||||
|
||||
if (isset($CFG->CSV_ENCODE)) {
|
||||
$csv_encode = '/\&\#' . $CFG->CSV_ENCODE . '/';
|
||||
}
|
||||
} else {
|
||||
$csv_delimiter = "\,";
|
||||
$csv_delimiter2 = ",";
|
||||
$csv_delimiter = ",";
|
||||
}
|
||||
|
||||
// prepare temp file
|
||||
@ -107,7 +105,7 @@ die;
|
||||
"hidepicture" => 1);
|
||||
|
||||
// --- get header (field names) ---
|
||||
$header = split($csv_delimiter, fgets($fp,1024));
|
||||
$header = explode($csv_delimiter, fgets($fp,1024));
|
||||
// check for valid field names
|
||||
foreach ($header as $i => $h) {
|
||||
$h = trim($h); $header[$i] = $h; // remove whitespace
|
||||
@ -136,10 +134,10 @@ die;
|
||||
}
|
||||
//Note: commas within a field should be encoded as , (for comma separated csv files)
|
||||
//Note: semicolon within a field should be encoded as ; (for semicolon separated csv files)
|
||||
$line = split($csv_delimiter, fgets($fp,1024));
|
||||
$line = explode($csv_delimiter, fgets($fp,1024));
|
||||
foreach ($line as $key => $value) {
|
||||
//decode encoded commas
|
||||
$record[$header[$key]] = preg_replace($csv_encode,$csv_delimiter2,trim($value));
|
||||
$record[$header[$key]] = preg_replace($csv_encode,$csv_delimiter,trim($value));
|
||||
}
|
||||
if ($record[$header[0]]) {
|
||||
// add a new group to the database
|
||||
|
Loading…
x
Reference in New Issue
Block a user