Skip to content
This repository has been archived by the owner on Jun 17, 2020. It is now read-only.

Why low level access to DB if there is an API? #4

Open
mudrd8mz opened this issue Apr 13, 2018 · 0 comments
Open

Why low level access to DB if there is an API? #4

mudrd8mz opened this issue Apr 13, 2018 · 0 comments

Comments

@mudrd8mz
Copy link

It seem your plugin is directly inserting records into the enrol_lti_tools table. Why not to use enrol_lti_plugin::add_instance() API instead? Of course after checking other methods such as can_add_instance().

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant