Non-core contributed modules

File storage conversion HotPot

Details

  • Type: Task Task
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: 2.0.2
  • Fix Version/s: 2.0.2
  • Component/s: Module: HotPot
  • Labels:
    None
  • Affected Branches:
    MOODLE_20_STABLE
  • Fixed Branches:
    MOODLE_20_STABLE

Activity

Hide
Gordon Bateson added a comment -

Hi Petr,
for last couple of weeks I have been working on updating the HotPot module to be compliant with Moodle 2.0 in general, and the new API's for file access, renderers and backups in particular.

As you may know, HotPot has been moved out to "contrib/pugins/mod/hotpot". Today I committed a lot of changes for things I have been working on recently.

So far, I have modified the module to the point where a HotPot activity can be created (and edited), viewed, and submitted. I have not yet finished the reports, and backups.

Martin D. asked me to work with with you to make sure the new HotPot code conforms to Moodle's coding and security standards. I am not sure how best we can do that, but for starters, please could you download the new HotPot scripts (from contrib) and let me know what you think.

many thanks in advance
Gordon

Show
Gordon Bateson added a comment - Hi Petr, for last couple of weeks I have been working on updating the HotPot module to be compliant with Moodle 2.0 in general, and the new API's for file access, renderers and backups in particular. As you may know, HotPot has been moved out to "contrib/pugins/mod/hotpot". Today I committed a lot of changes for things I have been working on recently. So far, I have modified the module to the point where a HotPot activity can be created (and edited), viewed, and submitted. I have not yet finished the reports, and backups. Martin D. asked me to work with with you to make sure the new HotPot code conforms to Moodle's coding and security standards. I am not sure how best we can do that, but for starters, please could you download the new HotPot scripts (from contrib) and let me know what you think. many thanks in advance Gordon
Hide
Petr Škoda (skodak) added a comment -

Hello, I am afraid subplugins.php is not designed to include hundreds of directories - workshop and assignment are examples of how it was expected to be used. Unfortunately I do not have any time right now to do a review of your code, I am sorry.

Petr Skoda

Show
Petr Škoda (skodak) added a comment - Hello, I am afraid subplugins.php is not designed to include hundreds of directories - workshop and assignment are examples of how it was expected to be used. Unfortunately I do not have any time right now to do a review of your code, I am sorry. Petr Skoda
Hide
Gordon Bateson added a comment -

Thank you very much for responding so quickly Petr. I appreciate the feedback.

I will redo things a little so that only three items are listed in the subplugins.php.

I will also ask around for more feedback on the General Developer's forum.

all the best
Gordon

Show
Gordon Bateson added a comment - Thank you very much for responding so quickly Petr. I appreciate the feedback. I will redo things a little so that only three items are listed in the subplugins.php. I will also ask around for more feedback on the General Developer's forum. all the best Gordon
Hide
Gordon Bateson added a comment -

The file storage conversion for HotPot for Moodle 2.0 is now in place, and appears to work. I have tested it on a development site containing about 20 HotPots, and the update completed succesfully.

I will close this issue now, but please reopen it if you feel the file storage conversion for HotPot is still not working satisfactorily.

Show
Gordon Bateson added a comment - The file storage conversion for HotPot for Moodle 2.0 is now in place, and appears to work. I have tested it on a development site containing about 20 HotPots, and the update completed succesfully. I will close this issue now, but please reopen it if you feel the file storage conversion for HotPot is still not working satisfactorily.
Hide
Petr Škoda (skodak) added a comment -

thanks!

Show
Petr Škoda (skodak) added a comment - thanks!

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: