Merge pull request #16069 from owncloud/fix-15933
Fix: user login filter does not take username into account
This commit is contained in:
commit
6783699501
|
@ -41,6 +41,7 @@ if(isset($_POST['copyConfig'])) {
|
|||
$newConfig->setConfiguration($originalConfig->getConfiguration());
|
||||
} else {
|
||||
$configuration = new \OCA\user_ldap\lib\Configuration($nk, false);
|
||||
$newConfig->setConfiguration($configuration->getDefaults());
|
||||
$resultData['defaults'] = $configuration->getDefaults();
|
||||
}
|
||||
$newConfig->saveConfiguration();
|
||||
|
|
|
@ -207,7 +207,6 @@ OCA = OCA || {};
|
|||
*/
|
||||
onNewConfiguration: function(view, result) {
|
||||
if(result.isSuccess === true) {
|
||||
console.log('new config');
|
||||
var nthServer = view._configChooserNextServerNumber;
|
||||
view.$configChooser.find('option:selected').removeAttr('selected');
|
||||
var html = '<option value="'+result.configPrefix+'" selected="selected">'+t('user_ldap','{nthServer}. Server', {nthServer: nthServer})+'</option>';
|
||||
|
|
|
@ -958,7 +958,6 @@ class Wizard extends LDAPUtility {
|
|||
$userAttributes = array_change_key_case(array_flip($userAttributes));
|
||||
$parts = 0;
|
||||
|
||||
$x = $this->configuration->ldapLoginFilterUsername;
|
||||
if($this->configuration->ldapLoginFilterUsername === '1') {
|
||||
$attr = '';
|
||||
if(isset($userAttributes['uid'])) {
|
||||
|
|
Loading…
Reference in New Issue