#3948 Additional LDAP fields not working

1.550
closed-fixed
5
2011-07-12
2011-07-12
No

When there are multiple "additional LDAP fields" defined, it does not work. Everything entered in all fields ends up in the first one when saved.

Diagnosis & fix:
The Edit User form is incorrect.

In the file ldap-useradmin-lib.pl, there is a loop to print inputs with names field_0, field_1, etc.. However, the index $i is never incremented inside the loop!

The following patch fixes this problem:
--- ldap-useradmin-lib.pl 2011-07-12 17:15:06.239838559 -0500
+++ ldap-useradmin-lib.pl.fixed 2011-07-12 17:14:59.819817399 -0500
@@ -936,6 +936,7 @@
}

print &ui_table_row($f->[1], $input);
+ $i++;
}
print &ui_table_end();
}

Discussion

  • Jamie Cameron

    Jamie Cameron - 2011-07-12

    Thanks - this fix will be included in the next Webmin release.

     
  • Jamie Cameron

    Jamie Cameron - 2011-07-12
    • status: open --> closed-fixed
     

Get latest updates about Open Source Projects, Conferences and News.

Sign up for the SourceForge newsletter:





No, thanks