Moodle

Add pipe to list of offered CSV delimiters in csv_import_reader class

Details

  • Type: Improvement Improvement
  • Status: Open Open
  • Priority: Minor Minor
  • Resolution: Unresolved
  • Affects Version/s: 1.9.1
  • Fix Version/s: DEV backlog
  • Component/s: Administration
  • Labels:
    None
  • Database:
    Any
  • Affected Branches:
    MOODLE_19_STABLE

Description

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

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Robert Allerstorfer added a comment -

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

Show
Robert Allerstorfer added a comment - The attached patch adds the pipe. Would be great to see it committed.
Hide
Permalink
Robert Allerstorfer added a comment -

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.

Show
Robert Allerstorfer added a comment - 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.
Hide
Permalink
Robert Allerstorfer added a comment -

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

Show
Robert Allerstorfer added a comment - Hm, do you have a reason for not committing this patch into the current 1.9 branch? It is really trivial.

People

Vote (1)
Watch (3)

Dates

  • Created:
    Updated:
Atlassian JIRA (v4.4.3#663-r165197) | Report a problem
Powered by a free Atlassian JIRA open source license for Moodle. Try JIRA - bug tracking software for your team.