turnitin / moodle-plagiarism_turnitin

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

Quiz Essay question type should support scanning file attachments #575

Closed aspark21 closed 3 years ago

aspark21 commented 3 years ago

At the moment, the text of the response is scanned but file attachments are not. The filename gets appended to the text but it's not possible for the attachments to get properly scanned. This is a feature implemented by Dan Marsden which TII doesn't seem to support. I would expect separate reports for each attachment & the text area.

https://tracker.moodle.org/browse/MDL-32226?focusedCommentId=841751&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#comment-841751

danmarsden commented 3 years ago

fyi - looks like you'll need to add something like

 $files = $qa->get_last_qt_files('attachments', $context->id);
foreach ($files as $file) {
   // Queue file for sending to Turnitin
}

Somewhere in this block. https://github.com/turnitin/moodle-plagiarism_turnitin/blob/cf3952c67833d3af76133fe3038497c03b765b0c/lib.php#L2571

and then check that the get_links stuff returns the results of those files in the UI

aspark21 commented 3 years ago

@jmcgettrick We're in need of this at relatively short notice for the upcoming exam period, too short to consider lobbying Turnitin HQ to put this into it's roadmap so we are looking to develop it for ourselves & submit MR.

I think our devs might want to have a chat with you at some point to make sure it's all done properly though.

jmcgettrick commented 3 years ago

Hi @aspark21 that would be great, we stopped working on the feature while waiting for it to be released in Moodle. Feel free for them to get in touch, they can get me on here or via the Moodle thread on telegram.

jmcgettrick commented 3 years ago

I should add, we will be picking up the feature to finish off soon!

jmcgettrick commented 3 years ago

Thanks @danmarsden - First pass: https://github.com/turnitin/moodle-plagiarism_turnitin/pull/579

aspark21 commented 3 years ago

Included in v2021060801

Thanks John!