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

allow student to clear SCORM attempt

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.2, 2.3
    • Fix Version/s: 2.2.1
    • Component/s: SCORM
    • Labels:
    • Testing Instructions:
      Hide

      Add a SCORM - set Display attempt status == yes
      Log in as student - enter the SCORM and get it to return some data in reports.
      check the main "entry" view.php page to make sure that the button "delete all SCORM attempts" isn't shown.
      Log in as Admin - give student role the new capability "deleteownresponses"
      Log back in as student - check to see if the button shows and works as expected by deleting responses.

      (it might be easier to test as admin user because the student can't "see" results in the reports page to see if the data has been deleted.

      Show
      Add a SCORM - set Display attempt status == yes Log in as student - enter the SCORM and get it to return some data in reports. check the main "entry" view.php page to make sure that the button "delete all SCORM attempts" isn't shown. Log in as Admin - give student role the new capability "deleteownresponses" Log back in as student - check to see if the button shows and works as expected by deleting responses. (it might be easier to test as admin user because the student can't "see" results in the reports page to see if the data has been deleted.
    • Affected Branches:
      MOODLE_22_STABLE, MOODLE_23_STABLE
    • Fixed Branches:
      MOODLE_22_STABLE
    • Pull Master Branch:
      master_MDL-28728

      Description

      I want this for ADL tests - it makes it easier if the student can clear their own attempts when running the ADL tests - that way we can automate it a bit better. Must be disabled by default.

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              Hide
              danmarsden Dan Marsden added a comment -

              NOTE - this adds a new capability that is deliberately not given to any roles.

              Show
              danmarsden Dan Marsden added a comment - NOTE - this adds a new capability that is deliberately not given to any roles.
              Hide
              danmarsden Dan Marsden added a comment -

              note - the Delete response button is only shown if user has the capability and if display attempt status == yes.

              Show
              danmarsden Dan Marsden added a comment - note - the Delete response button is only shown if user has the capability and if display attempt status == yes.
              Hide
              stronk7 Eloy Lafuente (stronk7) added a comment -

              The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week.

              TIA and ciao

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week. TIA and ciao
              Hide
              danmarsden Dan Marsden added a comment -

              rebased - happy for this to be applied to 22Stable as well if you are still keeping in sync.

              Show
              danmarsden Dan Marsden added a comment - rebased - happy for this to be applied to 22Stable as well if you are still keeping in sync.
              Hide
              nebgor Aparup Banerjee added a comment -

              Thanks Dan, thats been integrated .

              ps: i wonder though if we should document this somewhere with regards to its use for ADL.

              Show
              nebgor Aparup Banerjee added a comment - Thanks Dan, thats been integrated . ps: i wonder though if we should document this somewhere with regards to its use for ADL.
              Hide
              ankit_frenz Ankit Agarwal added a comment -

              Hi Dan,
              Altough the functionality is working great and as expected, but there is a missing string which is generating the following stack

              Invalid get_string() identifier: 'scorm:deleteownresponses' or component 'mod_scorm'. Perhaps you are missing $string['scorm:deleteownresponses'] = ''; in mod/scorm/lang/en/scorm.php?
               
                  line 6313 of \lib\moodlelib.php: call to debugging()
                  line 6905 of \lib\moodlelib.php: call to core_string_manager->get_string()
                  line 2719 of \lib\accesslib.php: call to get_string()
                  line 161 of \admin\tool\capability\index.php: call to get_capability_string()

              I can pass this once the string is added
              Thanks

              Show
              ankit_frenz Ankit Agarwal added a comment - Hi Dan, Altough the functionality is working great and as expected, but there is a missing string which is generating the following stack Invalid get_string() identifier: 'scorm:deleteownresponses' or component 'mod_scorm'. Perhaps you are missing $string['scorm:deleteownresponses'] = ''; in mod/scorm/lang/en/scorm.php?   line 6313 of \lib\moodlelib.php: call to debugging() line 6905 of \lib\moodlelib.php: call to core_string_manager->get_string() line 2719 of \lib\accesslib.php: call to get_string() line 161 of \admin\tool\capability\index.php: call to get_capability_string() I can pass this once the string is added Thanks
              Hide
              danmarsden Dan Marsden added a comment -

              doh! - I thought I'd pushed that through, fixed now thanks Ankit!

              Apu - good point, I have plans to do some work on the SCORM docs sometime pretty soon.

              Show
              danmarsden Dan Marsden added a comment - doh! - I thought I'd pushed that through, fixed now thanks Ankit! Apu - good point, I have plans to do some work on the SCORM docs sometime pretty soon.
              Hide
              nebgor Aparup Banerjee added a comment -

              Thanks, i've pulled that string in now. integrated and up for testing.

              Show
              nebgor Aparup Banerjee added a comment - Thanks, i've pulled that string in now. integrated and up for testing.
              Hide
              ankit_frenz Ankit Agarwal added a comment -

              all good now
              Thanks

              Show
              ankit_frenz Ankit Agarwal added a comment - all good now Thanks
              Hide
              stronk7 Eloy Lafuente (stronk7) added a comment -

              Whoever decided one week was worth 14 days had really one bad idea. Anyway, the nightmare is over, so thanks for your, once again, amazing contributions. Many, many thanks!

              Now... disconnect, relax and enjoy the next days, yay!

              Closing...ciao

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - Whoever decided one week was worth 14 days had really one bad idea. Anyway, the nightmare is over, so thanks for your, once again, amazing contributions. Many, many thanks! Now... disconnect, relax and enjoy the next days, yay! Closing...ciao
              Hide
              tsala Helen Foster added a comment -

              Documentation for this new capability is now available (better late than never!):

              http://docs.moodle.org/22/en/Capabilities/mod/scorm:deleteownresponses

              Show
              tsala Helen Foster added a comment - Documentation for this new capability is now available (better late than never!): http://docs.moodle.org/22/en/Capabilities/mod/scorm:deleteownresponses

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved:
                    Fix Release Date:
                    9/Jan/12