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

Fun with metadata and h5p #24

Open
stuartlamour opened this issue Oct 21, 2019 · 3 comments
Open

Fun with metadata and h5p #24

stuartlamour opened this issue Oct 21, 2019 · 3 comments

Comments

@stuartlamour
Copy link

stuartlamour commented Oct 21, 2019

Hi Mike,

using the "Use metadata for modules" with h5p. Works fine when you create the resource, but gives an error when you try and edit.

Screenshot 2019-10-21 at 15 18 52

line 494 of /lib/setuplib.php: moodle_exception thrown
line 182 of /mod/hvp/mod_form.php: call to print_error()
line 481 of /course/moodleform_mod.php: call to mod_hvp_mod_form->data_preprocessing()
line 206 of /local/metadata/context/module/classes/context_handler.php: call to moodleform_mod->set_data()
line 421 of /local/metadata/lib.php: call to metadatacontext_module\context_handler->coursemodule_standard_elements()
line 749 of /course/moodleform_mod.php: call to local_metadata_coursemodule_standard_elements()
line 737 of /course/moodleform_mod.php: call to moodleform_mod->plugin_extend_coursemodule_standard_elements()
line 118 of /mod/hvp/mod_form.php: call to moodleform_mod->standard_coursemodule_elements()
line 206 of /lib/formslib.php: call to mod_hvp_mod_form->definition()
line 99 of /course/moodleform_mod.php: call to moodleform->__construct()
line 141 of /course/modedit.php: call to moodleform_mod->__construct()

Anyone else found this or a workaround?

Cheers
Stuart

@seb-vial
Copy link

Hi there, see #18 and closed PR #8 more specifically this comment #8 (comment)_

@stuartlamour
Copy link
Author

Thanks @Birssan - added that as a patch locally and working so far!

@mhughes2k
Copy link

So is this "technically" a fault with H5P, rather than local_metadata.

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

3 participants