Skip to content

Setup settings properly

username-removed-1144264 requested to merge mk-setup-settings-properly into master

What does this MR do?

This MR is really just to make CE and EE consistent when it comes to setting up Settings.

Why was this MR needed?

In EE, there is at least one place in the code where we use the LDAP Settings in such a way as to need them to be setup in a particular way. If we don't do this, then at some point a Hash is returned, and we cannot use the typical dot syntax to access its values.

See https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/2690

Does this MR meet the acceptance criteria?

Merge request reports