Details

    • Type: Sub-task Sub-task
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.4
    • Fix Version/s: 2.4
    • Component/s: SCORM
    • Labels:
    • Testing Instructions:
      Hide
      1. Add a scorm package to your course, incase you donot already have one.
      2. Make a few attempts as different students. (Existing attempts are fine as well)
      3. Goto scorm>reports>basic as a teacher
      4. Export the report as text.
      5. make sure there is no error during download and the downloaded file contains all data correctly with correct headers in a tab separated format.
      Show
      Add a scorm package to your course, incase you donot already have one. Make a few attempts as different students. (Existing attempts are fine as well) Goto scorm>reports>basic as a teacher Export the report as text. make sure there is no error during download and the downloaded file contains all data correctly with correct headers in a tab separated format.
    • Affected Branches:
      MOODLE_24_STABLE
    • Fixed Branches:
      MOODLE_24_STABLE
    • Pull Master Branch:
      MDL-34337-master
    • Rank:
      42707

      Description

      Use the new class in lib/csvlib.class.php to export data into a csv, tsv or other delimited file.
      Examples can be found in:

      admin/user/usre_bulk_download.php
      grade/export/txt/grade_export_txt.php
      lib/tablelib.php
      mod/data/lib.php
      This issue deals with Basic reports of scorm

        Issue Links

          Activity

          Hide
          Michael de Raadt added a comment -

          Jinx.

          Show
          Michael de Raadt added a comment - Jinx.
          Hide
          Ankit Agarwal added a comment -

          Reopening as the linked issue is different from this one

          Show
          Ankit Agarwal added a comment - Reopening as the linked issue is different from this one
          Hide
          Michael de Raadt added a comment -

          It was subtle.

          Show
          Michael de Raadt added a comment - It was subtle.
          Hide
          Ankit Agarwal added a comment -

          @integrator

          • Master only
          • MDL-34074 must be integrated before this goes in.

          Thanks

          Show
          Ankit Agarwal added a comment - @integrator Master only MDL-34074 must be integrated before this goes in. Thanks
          Hide
          Adrian Greeve added a comment -

          Thanks Ankit.

          This also looks fine.

          Just waiting for MDL-34074 to be integrated.

          Show
          Adrian Greeve added a comment - Thanks Ankit. This also looks fine. Just waiting for MDL-34074 to be integrated.
          Hide
          Dan Poltawski added a comment -

          It'd be great to get a +1 from Dan M for this.

          Show
          Dan Poltawski added a comment - It'd be great to get a +1 from Dan M for this.
          Hide
          Dan Marsden added a comment -

          +1

          Show
          Dan Marsden added a comment - +1
          Hide
          Eloy Lafuente (stronk7) added a comment -

          The integration of this issue has been delayed to next week because the integration period is over (Monday, Tuesday) and testing must happen on Wednesday.

          This change to a more rigid timeframe on each integration/testing cycle aims to produce a better and clear separation and organization of tasks for everybody.

          This is a bulk-automated message, so if you want to blame somebody/thing/where, don't do it here (use git instead) :-D :-P

          Apologizes for the inconvenient, this will be integrated next week. Thanks for your collaboration & ciao

          Show
          Eloy Lafuente (stronk7) added a comment - The integration of this issue has been delayed to next week because the integration period is over (Monday, Tuesday) and testing must happen on Wednesday. This change to a more rigid timeframe on each integration/testing cycle aims to produce a better and clear separation and organization of tasks for everybody. This is a bulk-automated message, so if you want to blame somebody/thing/where, don't do it here (use git instead) :-D :-P Apologizes for the inconvenient, this will be integrated next week. Thanks for your collaboration & ciao
          Hide
          Ankit Agarwal added a comment -

          rebased
          Thanks!

          Show
          Ankit Agarwal added a comment - rebased Thanks!
          Hide
          Aparup Banerjee added a comment -

          Thanks! That's been integrated into master.

          Show
          Aparup Banerjee added a comment - Thanks! That's been integrated into master.
          Hide
          Andrew Davis added a comment -

          Seems fine. Passing.

          Show
          Andrew Davis added a comment - Seems fine. Passing.
          Hide
          Eloy Lafuente (stronk7) added a comment -

          For the good and the bad... this is now part of Moodle and people around the world will start using it immediately, what a responsibility!

          Many thanks for your collaboration, yay!

          Closing, ciao

          Show
          Eloy Lafuente (stronk7) added a comment - For the good and the bad... this is now part of Moodle and people around the world will start using it immediately, what a responsibility! Many thanks for your collaboration, yay! Closing, ciao

            People

            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: