[#MDL-15074] Add pipe to list of offered CSV delimiters in csv_import_reader class

Issue Details (XML | Word | Printable)

Key:	MDL-15074
Type:	Improvement
Status:	Open
Priority:	Minor
Assignee:	Petr Škoda (skodak)
Reporter:	Robert Allerstorfer
Votes:	1
Watchers:	3

Operations

Add/Edit UI Mockup to this issue

If you were logged in you would be able to see more operations.

Moodle

Add pipe to list of offered CSV delimiters in csv_import_reader class

Created: 31/May/08 11:43 PM Updated: 26/Nov/09 12:22 AM

Component/s:

Administration

Affects Version/s:

1.9.1

Fix Version/s:

1.9.8

File Attachments:

MDL-15074.patch (1 kB)

Database:	Any
Participants:	Petr Škoda (skodak) and Robert Allerstorfer
Security Level:	None
QA Assignee:	Robert Allerstorfer
Affected Branches:	MOODLE_19_STABLE
Fixed Branches:	MOODLE_19_STABLE

Description

« Hide

See discussion @ MDL-15065
Affected file: lib/csvlib.class.php

All

Comments

Change History

Version Control

Sort Order:

[ Permalink | « Hide ]

Robert Allerstorfer added a comment - 01/Jun/08 01:37 AM

The attached patch adds the pipe. Would be great to see it committed.

[ Permalink | « Hide ]

Robert Allerstorfer added a comment - 11/Jun/08 11:31 PM

Petr,

could you please comment if you are going to commit that patch, as discussed in MDL-15065? My CSV export code currently relays on your class, thus it would be very useful to get the improvement added here.

[ Permalink | « Hide ]

Robert Allerstorfer added a comment - 07/Oct/08 08:15 AM

Hm, do you have a reason for not committing this patch into the current 1.9 branch? It is really trivial.

All content on this web site is made available under the GNU General Public License, unless otherwise stated.