Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-46504

cvsimport reference in csvlib

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.6.4, 2.7.1, 2.8
    • Fix Version/s: 2.6.5, 2.7.2
    • Component/s: Libraries
    • Labels:
    • Testing Instructions:
      Hide

      Acceptance tests should pass

      Manual test:

      1. Go to $CFG->dataroot/temp (or $CFG->tempdir if you set it)
      2. Delete all contents
      3. Go to admin/tool/uploaduser/index.php and upload a csv with users
      4. The import SHOULD finish as expected
      5. List $CFG->dataroot/temp contents (or $CFG->tempdir if you set it)
      6. You SHOULD NOT see a cvsimport dir, only a csvimport
      Show
      Acceptance tests should pass Manual test: Go to $CFG->dataroot/temp (or $CFG->tempdir if you set it) Delete all contents Go to admin/tool/uploaduser/index.php and upload a csv with users The import SHOULD finish as expected List $CFG->dataroot/temp contents (or $CFG->tempdir if you set it) You SHOULD NOT see a cvsimport dir, only a csvimport
    • Affected Branches:
      MOODLE_26_STABLE, MOODLE_27_STABLE, MOODLE_28_STABLE
    • Fixed Branches:
      MOODLE_26_STABLE, MOODLE_27_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-46504_master

      Description

      While looking at MDL-44874 I've realized that csvlib is creating 2 directories inside tempdir, one of them named cvsimport, probably a typo, it is not a big issue as the file written there is deleted but we are creating an extra directory when there is no need.

        Attachments

          Activity

            People

            • Assignee:
              dmonllao David Monllaó
              Reporter:
              dmonllao David Monllaó
              Integrator:
              Dan Poltawski
              Tester:
              Damyon Wiese
              Participants:
              Component watchers:
              Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Sara Arjona (@sarjona), Víctor Déniz Falcón
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:
                Fix Release Date:
                8/Sep/14