Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed syntax issues #10

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

0x616469
Copy link

Declare member variables according to Moodle coding guidelines (https://docs.moodle.org/dev/Coding_style#Class_member_variables)

Declare member variables according to Moodle coding guidelines
Since the username is not unique the script should only look for
local users. It's possible that a user from a remote system
connected via MNET has the same username which could lead to
unintended consequences.
OdyX pushed a commit to liip-elearning/moodle_local_ldap that referenced this pull request Jul 18, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant