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

      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

        Gliffy Diagrams

          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: