MDL-83541 Add qtype restore code documentation

[marxjohnson]

This describes the required updates to qtype plugins following to implementation of MDL-83541 to avoid duplication of questions during restores.

[netlify]

✅ Deploy Preview for moodledevdocs ready!

Built without sensitive environment variables

Name	Link
🔨 Latest commit	1c474db
🔍 Latest deploy log	https://app.netlify.com/sites/moodledevdocs/deploys/67d788c2e526880008a9c692
😎 Deploy Preview	https://deploy-preview-1280--moodledevdocs.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[timhunt]

Thanks Mark. That looks like a really good new docs page.

The only bit I don't really like is the use of $name for the plugin name. Is this a convention used elsewhere in the docs? I managed to avoid it comletely on https://moodledev.io/docs/5.0/apis/plugintypes/qtype/newquestiondefaults. On https://moodledev.io/docs/5.0/apis/plugintypes/qbank/filters we use qbank_myplugin. Up to you to make a final call.

[andrewnicols]

I'd also second not making this a var. Another option would be to use a placeholder:

In your  `restore_qtype_<name>_plugin`  class, add an override of the `remove_excluded_question_data()` method, which removes this

[timhunt]

I like Andrew's suggestion. Feel free to updated qbank/filters while you are at it Mark (he suggested cheekily :-))

[HuongNV13]

Thanks, Mark
It looks good to me

[HuongNV13]

I have rebased the branch, and the failures are gone!

[binhdv1983]

Hi @marxjohnson,

We’ve developed three new question types, and when I run the unit test located at mod/quiz/tests/backup/repeated_restore_test.php, no errors are reported — even though we added an additional path to the $paths[] array in the define_question_plugin_structure() method.

My question is: should we still apply the changes described in this documentation, even if no errors are currently detected?