Moodle

Gradebook csv import wraps headers if they're too long

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: 1.9.4
  • Fix Version/s: 1.9.5
  • Component/s: Gradebook
  • Labels:
    None
  • Affected Branches:
    MOODLE_19_STABLE
  • Fixed Branches:
    MOODLE_19_STABLE

Description

See http://moodle.org/mod/forum/discuss.php?d=116180 for explanation.

The reason is that the import limits each line to 1024 bytes. I'm increasing this to 4096, it should cover any reasonable length of header line.

Issue Links

Activity

Hide
Tim Hunt added a comment -

Hmm. Nico. I am not very impressed with this fix. You seem to have changed a lot of hard-coded 1024 numbers to 4096. That is just asking for trouble in future. Surely you should be using a constant?

Show
Tim Hunt added a comment - Hmm. Nico. I am not very impressed with this fix. You seem to have changed a lot of hard-coded 1024 numbers to 4096. That is just asking for trouble in future. Surely you should be using a constant?
Hide
Tim Hunt added a comment -

Thanks, that's better nico. Closing.

Show
Tim Hunt added a comment - Thanks, that's better nico. Closing.

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: