Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use instance context instead of config context for saved data #10

Open
dmitriim opened this issue May 6, 2024 · 0 comments
Open

Use instance context instead of config context for saved data #10

dmitriim opened this issue May 6, 2024 · 0 comments

Comments

@dmitriim
Copy link

dmitriim commented May 6, 2024

This came out as part of discussion in #9

Currently when saving field data config context is being used. There may be some reasons for doing that, but this just indicates that something is missing in Moodle core APIs that pushes developers to workaround them and this should be fixed in core.

So rough tasks here would be:

  • figure out core API limitations that forced to use config context
  • if there are any, then raise MDL to fix those limitations
  • when MDL is fixed (or if not needed), implement context using instance context instead of config context
  • write upgrade script to fix all affected records
  • release a new version of the plugin

@andrewhancox does this sound ok to you?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant