Details

    • Type: Improvement
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 1.9.7
    • Fix Version/s: 2.0
    • Component/s: SCORM
    • Labels:
      None
    • Database:
      MySQL
    • Affected Branches:
      MOODLE_19_STABLE
    • Fixed Branches:
      MOODLE_20_STABLE

      Description

      I've added some changes to the scorm report, the modified files are attached.

      The modifications includes :
      Complete recoding of the report.php based on mod/quiz/report/overview/report.php, this enable the following features :

      • group select box
      • sortable table
      • collapsable table
      • download in Excel, ODS and text format
      • report preferences : number of elements per page, show scorm details or not, show all students, only attempted or not attempted results
      • tabs navigation to switch between the sco info page and report page. In the scorm/view.php i get rid of the top link "show xxx users"

      Two new files :

      • tabs.php for the tabs
      • reportsettings_form.php for the preference form (based on MDL-18836)

      Solution for this MDL-21421 and an improvement of MDL-16184 :
      The problem is in scorm_get_sco_runtime (locallib.php), because it takes all scoes track ordered by timemodified, and then select the scoes track again but ordered by scoid to take the first one (which is not necessarily the first accessed).
      For me, the changes i've made give more relevant results (the first access is really the first access and the last really the last).
      By using x.start.time, the previous patch takes in facts the last opening of the first sco and not really the first access.

      Sorry but my patch also includes this one : MDL-18835, because it's not included in Moodle 1.9.7

      And sorry for my bad english,

        Gliffy Diagrams

        1. delete_attempt_v2.patch
          2 kB
          valerian
        2. delete_attempt_v3.patch
          3 kB
          valerian
        3. delete_attempt.patch
          2 kB
          valerian
        4. lang_en_utf8.patch
          3 kB
          valerian
        5. lang_fr_utf8.patch
          4 kB
          valerian
        6. report.php
          34 kB
          valerian
        7. reportsettings_form.php
          1 kB
          valerian
        8. scorm_report.patch
          6 kB
          valerian
        9. tabs.php
          1 kB
          valerian

          Issue Links

            Activity

            Hide
            danmarsden Dan Marsden added a comment -

            Thanks Valerian,
            Just a quick note to let you know I've just returned from leave, but will try to have a look at your patches early next week!

            Show
            danmarsden Dan Marsden added a comment - Thanks Valerian, Just a quick note to let you know I've just returned from leave, but will try to have a look at your patches early next week!
            Hide
            valerian valerian added a comment -

            I'll post a new version of report.php soon because there are some bugs in the present file.

            Show
            valerian valerian added a comment - I'll post a new version of report.php soon because there are some bugs in the present file.
            Hide
            valerian valerian added a comment -

            ok, all the files needed and up-to-date for this patch are in report.zip

            Show
            valerian valerian added a comment - ok, all the files needed and up-to-date for this patch are in report.zip
            Hide
            m91932j MJ added a comment -

            How can i use this patch or how do I add this patch?

            Show
            m91932j MJ added a comment - How can i use this patch or how do I add this patch?
            Hide
            danmarsden Dan Marsden added a comment -
            Show
            danmarsden Dan Marsden added a comment - MJ - please see: http://docs.moodle.org/en/Development:How_to_apply_a_patch
            Hide
            admssm ivan added a comment -

            Hi,

            Id like to ask, how can i add this report to my moodle..?
            Thank you.

            IB

            Show
            admssm ivan added a comment - Hi, Id like to ask, how can i add this report to my moodle..? Thank you. IB
            Hide
            danmarsden Dan Marsden added a comment -

            I refactored this a bit and have now pushed to HEAD - thanks for the patch Valerian - it's a much better interface!

            I'd like to backport this to 1.9Stable at some point but it would be good if people could test the version in Head.

            thanks!

            Show
            danmarsden Dan Marsden added a comment - I refactored this a bit and have now pushed to HEAD - thanks for the patch Valerian - it's a much better interface! I'd like to backport this to 1.9Stable at some point but it would be good if people could test the version in Head. thanks!
            Hide
            valerian valerian added a comment -

            Hi Dan,

            I find a bug in the delete function : grades are note deleted from general gradebook when you delete one attemp. My patch is attached in the file delete_attempt.patch.

            Show
            valerian valerian added a comment - Hi Dan, I find a bug in the delete function : grades are note deleted from general gradebook when you delete one attemp. My patch is attached in the file delete_attempt.patch.
            Hide
            valerian valerian added a comment -

            Sorry, a mistake in my previous patch. The file delete_attempt_v2.patch is the correct one.

            Show
            valerian valerian added a comment - Sorry, a mistake in my previous patch. The file delete_attempt_v2.patch is the correct one.
            Hide
            valerian valerian added a comment -

            Again i'm sorry, please take the delete_attempt_v3.patch for this bug.

            Show
            valerian valerian added a comment - Again i'm sorry, please take the delete_attempt_v3.patch for this bug.
            Hide
            danmarsden Dan Marsden added a comment -

            Thanks for your work on this Valerian! - seems to work fine for me!

            Show
            danmarsden Dan Marsden added a comment - Thanks for your work on this Valerian! - seems to work fine for me!
            Hide
            danmarsden Dan Marsden added a comment -

            As Moodle 2.0 is almost out and because this makes significant changes to the UI I've decided to only push this into 2.0 - thanks for the Patch Valerian!

            Show
            danmarsden Dan Marsden added a comment - As Moodle 2.0 is almost out and because this makes significant changes to the UI I've decided to only push this into 2.0 - thanks for the Patch Valerian!

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Fix Release Date:
                  24/Nov/10