Using the upload widget doesn't seem to work when used inside the Paragraphs module. This is almost same as on 1.x branch #2945317: H5P content lost when collapsing/opening in Paragraphs
1. Add H5P field inside Paragraphs component. Set field to use Upload widget.
2. Create node with Paragraphs with H5P field, save the node.
3. H5P file is not uploaded due to parent element validation in src/Plugin/Field/FieldWidget/H5PUploadWidget.php
Match the validation better.
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | h5p-3308118-paragraphs-support-3.patch | 4.56 KB | jhuhta |
Show commandsStart within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
changes
Comments
Comment #2
sokru commentedThis should work both with and without Paragraphs/IEF etc.
Comment #3
jhuhta commented#2 was not quite enough. The problem with the Paragraphs use seems to be that the H5P module expects the field name to be unique, and that's not the case if H5P is a paragraph type. This patch should fix it.
This one also contains an idea from #3308102: Indication of uploaded .h5p file, but it didn't quite work standalone without this one either.
Comment #4
illeace commentedThis patch still applies as of the release of 2.0.0-alpha6, and seems to fix the problem indicated. I did a bit of testing using the H5P file upload widget within a paragraph and also in a standard node, using single and mulit-value fields. Everything I tried worked. Marking RTBC.
Comment #6
sim_1Comment #7
sim_1Created a MR and tried to copy over all the changes from the patch file. I'll double-check that they look the same before merging.
Comment #10
sim_1Great! Thanks, all! Merged and marking as fixed.