search

turnitin / moodle-plagiarism_turnitin

Turnitin Plagiarism plugin for Moodle
http://www.turnitin.com
45 stars 65 forks source link

Show the 'accept EULA' prompt on new replies made in-page to a forum #615

Closed jonof closed 1 year ago

jonof commented 2 years ago

Currently the "Your file has not been submitted to Turnitin. Please click here to accept our EULA." message only shows on Forum posts when the page is first loaded, because that's when eula.js takes notice and shows .pp_turnitin_eula elements. If a user performs an in-page reply, any newly-introduced .pp_turnitin_eula element within the newly-added post remains hidden until the page is next reloaded fully. Since Moodle 3.8, an event is triggered by the Forum plugin when a new post has been added, and this patch listens for that event to then show the .pp_turnitin_eula element that may have come into being if the user hasn't previously accepted the EULA.

jonof commented 2 years ago

Note: ought to solve issue #499

dwinn commented 1 year ago

Thank you for your contribution @jonof , I have confirmed that your patch works and it will go out in our next release. We expect this to go out very soon.