Details

    • Type: Task Task
    • Status: Resolved
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 1.9.16
    • Fix Version/s: 1.9.19
    • Component/s: Module: HotPot
    • Labels:
      None
    • Rank:
      38269

      Description

      Gordon has supplied the latest version of the HotPot module so that it can be integrated into 1.9 before support ends.

        Issue Links

          Activity

          Hide
          Gordon Bateson added a comment -

          Great - thanks for setting this up Michael.

          Show
          Gordon Bateson added a comment - Great - thanks for setting this up Michael.
          Hide
          Adrian Greeve added a comment -

          A git branch for this has been created and I also removed some of the white space so that it will pass through integration.

          Show
          Adrian Greeve added a comment - A git branch for this has been created and I also removed some of the white space so that it will pass through integration.
          Hide
          Michael de Raadt added a comment -

          Just looking at the code. This might need to have its version updated.

          Gordon, could you supply a hot potatoes file so we can test this?

          Show
          Michael de Raadt added a comment - Just looking at the code. This might need to have its version updated. Gordon, could you supply a hot potatoes file so we can test this?
          Hide
          Gordon Bateson added a comment -

          Uploaded the "hotpot.zip" file which includes simple samples of each of the 5 types of Hot Potatoes quiz.

          Show
          Gordon Bateson added a comment - Uploaded the "hotpot.zip" file which includes simple samples of each of the 5 types of Hot Potatoes quiz.
          Hide
          Michael de Raadt added a comment -

          Great. Thanks Gordon.

          I'll test this and see if we can get this integrated.

          Show
          Michael de Raadt added a comment - Great. Thanks Gordon. I'll test this and see if we can get this integrated.
          Hide
          Michael de Raadt added a comment -

          OK. I've had a chance to review and test this new version.

          The code looks fine to me. Most of the changes are innocuous language/help changes. The other changes seem OK. They don't conform completely to what we would expect of HQ programmers, but this module has never been maintained by us, so that is fine as far as I'm concerned.

          Functionally, the different types of questions seem to be working fine. I tested each of the samples Gordon supplied and they worked fine. I'm going to add some testing instructions to this issue, but I'd also be OK if this skipped testing.

          Show
          Michael de Raadt added a comment - OK. I've had a chance to review and test this new version. The code looks fine to me. Most of the changes are innocuous language/help changes. The other changes seem OK. They don't conform completely to what we would expect of HQ programmers, but this module has never been maintained by us, so that is fine as far as I'm concerned. Functionally, the different types of questions seem to be working fine. I tested each of the samples Gordon supplied and they worked fine. I'm going to add some testing instructions to this issue, but I'd also be OK if this skipped testing.
          Hide
          Michael de Raadt added a comment -

          Testing instructions:

          1. Update Moodle 1.9 to latest integration
          2. Log in as an admin
          3. Navigate to the site home page
          4. Navigate to Site admin > Server > Debugging and set the debugging level to the second highest level (All: ...)
          5. Navigate to Site admin > Modules > Activities > Manage activities
          6. Unhide the Hot Potatoes module
          7. Navigate to the site home page
          8. Navigate to a course
          9. Download the zip file attached to this issue and extract the files on your computer
          10. Add a Hot Potatoes Quiz activity from the Activities menu
          11. In the module configuration, next to File name click Choose or upload a file
          12. In the file browser click Upload a file
          13. Click browse and upload one of the extracted files
          14. Back in the file browser, click Choose next to the new file
          15. Back in the configuration, scroll to the bottom and click Save and display
          16. Test the quiz (it is a single question that should be easy)
          17. Repeat the last seven steps for each of the other extracted files, each representing a different type of question
          18. Navigate back to the course
          19. In the activities menu, click Hot Potatoes Quiz and see the instances collectively
          20. Click to view a report for one of the quizzes
          21. Try adjusting the settings for the report
          22. Export a report to text and Excel formats by altering the Format option in the report
          23. Navigate back to the course
          24. Click on Hot Potatoes Quiz in the Activities menu
          25. Regrade one of the quizzes
          Show
          Michael de Raadt added a comment - Testing instructions: Update Moodle 1.9 to latest integration Log in as an admin Navigate to the site home page Navigate to Site admin > Server > Debugging and set the debugging level to the second highest level (All: ...) Navigate to Site admin > Modules > Activities > Manage activities Unhide the Hot Potatoes module Navigate to the site home page Navigate to a course Download the zip file attached to this issue and extract the files on your computer Add a Hot Potatoes Quiz activity from the Activities menu In the module configuration, next to File name click Choose or upload a file In the file browser click Upload a file Click browse and upload one of the extracted files Back in the file browser, click Choose next to the new file Back in the configuration, scroll to the bottom and click Save and display Test the quiz (it is a single question that should be easy) Repeat the last seven steps for each of the other extracted files, each representing a different type of question Navigate back to the course In the activities menu, click Hot Potatoes Quiz and see the instances collectively Click to view a report for one of the quizzes Try adjusting the settings for the report Export a report to text and Excel formats by altering the Format option in the report Navigate back to the course Click on Hot Potatoes Quiz in the Activities menu Regrade one of the quizzes
          Hide
          Michael de Raadt added a comment -

          I was hoping this was going to be integrated a few months ago, but alas...

          I've asked again and hopefully it will happen soon.

          Show
          Michael de Raadt added a comment - I was hoping this was going to be integrated a few months ago, but alas... I've asked again and hopefully it will happen soon.
          Hide
          Gordon Bateson added a comment -

          Thanks for you continued hard work on this issue Michael.

          Show
          Gordon Bateson added a comment - Thanks for you continued hard work on this issue Michael.
          Hide
          Martin Dougiamas added a comment -

          Thank you too, Gordon!

          Show
          Martin Dougiamas added a comment - Thank you too, Gordon!
          Hide
          Aparup Banerjee added a comment -

          Hello,
          i've had a quick look and just 1 thing:

          • the $Id$ : i don't think that needs to be updated anymore as we're now using git ($Id$ was for CVS). that should take away updates to lots of files in the current patch.

          ps: I'm adding Dan Marsden here so he can add his say here before putting this through into 1.9

          Show
          Aparup Banerjee added a comment - Hello, i've had a quick look and just 1 thing: the $Id$ : i don't think that needs to be updated anymore as we're now using git ($Id$ was for CVS). that should take away updates to lots of files in the current patch. ps: I'm adding Dan Marsden here so he can add his say here before putting this through into 1.9
          Hide
          Dan Marsden added a comment -

          Thanks Aparup +1 from me!! - although it would be really good if we could strip out the $id stuff before integration if possible. - thanks Gordon!

          Show
          Dan Marsden added a comment - Thanks Aparup +1 from me!! - although it would be really good if we could strip out the $id stuff before integration if possible. - thanks Gordon!
          Hide
          Aparup Banerjee added a comment -

          ok. I've taken out the $Id$ changes. This has been integrated into MOODLE_19_STABLE.

          Feel free to test this! (test instructions provided above by Michael)

          Show
          Aparup Banerjee added a comment - ok. I've taken out the $Id$ changes. This has been integrated into MOODLE_19_STABLE. Feel free to test this! (test instructions provided above by Michael)
          Hide
          Michael de Raadt added a comment -

          Thanks for integrating this, Apu. Thanks for your input, Dan. Thanks for helping out, Adrian.

          And a big thanks to Gordon for creating the code and sticking by it.

          Show
          Michael de Raadt added a comment - Thanks for integrating this, Apu. Thanks for your input, Dan. Thanks for helping out, Adrian. And a big thanks to Gordon for creating the code and sticking by it.
          Hide
          Gordon Bateson added a comment -

          Sorry to say that the recent commit for the HotPot module in Moodle 1.9 introduced a fatal error in the HotPot restore script (mod/hotpot/restorelib.php)

          Please see CONTRIB-3662 for details of symptoms and fix (easy!)

          Show
          Gordon Bateson added a comment - Sorry to say that the recent commit for the HotPot module in Moodle 1.9 introduced a fatal error in the HotPot restore script (mod/hotpot/restorelib.php) Please see CONTRIB-3662 for details of symptoms and fix (easy!)

            People

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

              Dates

              • Created:
                Updated:
                Resolved:

                Development