This repository has been archived by the owner on Jan 15, 2018. It is now read-only.

get_links() should be more efficient #2

Closed

danmarsden opened this issue Mar 1, 2015 · 0 comments

danmarsden commented Mar 1, 2015

The call to plagiarism::get_links() is made in the core code in a lot of different places but as your code only supports the assignment module you should add a sanity check to ensure it comes from the assignment module before making a db call:
https://github.com/onno-schuit/copycheck/blob/master/moodle-plagiarism_copycheck/lib.php#L54

martijnspruijt added a commit that referenced this issue


          Fixed issue #1 and #2

641ce4c

martijnspruijt closed this as completed

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.