I needed to add my custom behavior to the H5PAJAX::setFinished() method. I wasn't able to find any other way than to add event dispatcher directly before returning JsonResponse. It could be useful for the others, I think. Patch in the comment.

CommentFileSizeAuthor
#3 h5p-2986358-2.patch2.15 KBmilos.kroulik
#2 h5p-2986358-1.patch2.18 KBmilos.kroulik

Comments

milos.kroulik created an issue. See original summary.

milos.kroulik’s picture

milos.kroulik commented
StatusFileSize
new h5p-2986358-1.patch2.18 KB
milos.kroulik’s picture

milos.kroulik commented
StatusFileSize
new h5p-2986358-2.patch2.15 KB

Fixed coding standards.

thomasmars’s picture

thomasmars commented

I've created an issue for reviewing the patch in the public H5P issue tracker: https://h5ptechnology.atlassian.net/browse/HFP-2136. This will make sure it gets prioritized and dealt with. Thank you for the patch.

  • paalj committed 8406860 on 8.x-1.x authored by milos.kroulik 
    Issue #2986358 by milos.kroulik: Add event subscriber to allow custom...
paalj’s picture

paalj commented
Status: Active » Fixed
paalj’s picture

paalj commented

Thank you for the patch. It is now applied and will be part of the next release.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.