mebis-lp / moodle-mod_mootimeter

Other
8 stars 7 forks source link

MBS-9094: Implement reset userdata #105

Closed PhMemmel closed 2 months ago

PhMemmel commented 2 months ago

Closes #92

pmaneggia commented 2 months ago

I think the solution of having a uniform handling of the reset in the tool helper class is very practical and I do not object. In principle though one would think it is up to each plugin to decide this sort of behaviour, particularly how to delete the answers? This can however of course be addressed in the future.

PhMemmel commented 2 months ago

I agree and this is already possible: I intentionally created the corresponding functions in the toolhelper class and did not do it directly in the lib.php hooks, because the tools inherit from toolhelper class and - if they have special requirements for implementing these functions - can simply overwrite these methods from the toolhelper base class.