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.

Sign up for GitHub

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

Jump to bottom

Adding LDAP server reuses last number in the list #15766

Closed
jnfrmarks opened this issue Apr 20, 2015 · 3 comments
Closed

Adding LDAP server reuses last number in the list #15766

jnfrmarks opened this issue Apr 20, 2015 · 3 comments
Labels
Type:Bug
Milestone
8.1

Comments

@jnfrmarks
Copy link

Steps to reproduce

  1. Install 8.1 enterprise
  2. Enable LDAP app
  3. Navigate to admin page
  4. Add a bunch of new LDAP servers (I added 5)
  5. Delete some of the middle ones (deleting one will do)
  6. Add a new LDAP server

Expected behaviour

New LDAP server should get the next numerical value available

Actual behaviour

The next LDAP server created reuses the last number:

p321

Server configuration

Ubuntu 14.04
PHP 5.5.9
ownCloud Enterprise Edition 8.1 alpha 1 (daily) Build:2015-04-16T03:08:13+00:00 f50798a
@karlitschek
Copy link
Contributor

@blizzz

@blizzz
Copy link
Contributor

blizzz commented Apr 20, 2015

Confirmed. The important part is the delete action. This is also reproducible on older versions.

@blizzz blizzz added this to the 8.1-current milestone Apr 21, 2015
@blizzz
Copy link
Contributor

blizzz commented Apr 21, 2015

Fixed with #15773

@blizzz blizzz closed this as completed in 8593415 Apr 28, 2015
DeepDiver1975 added a commit that referenced this issue Apr 28, 2015
@DeepDiver1975
Merge pull request #15773 from owncloud/fix-15766
5fa5f46 
LDAP Wizard: increasing server numbers and avoid configID collisions, fixes #15766
mmattel pushed a commit to mmattel/core that referenced this issue May 22, 2015
@blizzz @mmattel
LDAP Wizard: have always-increasing 'nth Server' number per full page…
c15f16b 
… load, fixes owncloud#15766
mmattel pushed a commit to mmattel/core that referenced this issue May 22, 2015
@DeepDiver1975 @mmattel
Merge pull request owncloud#15773 from owncloud/fix-15766
9b953d7 
LDAP Wizard: increasing server numbers and avoid configID collisions, fixes owncloud#15766
@lock lock bot locked as resolved and limited conversation to collaborators Aug 12, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
Type:Bug
Projects
None yet
Milestone
8.1
Development

No branches or pull requests
3 participants
@karlitschek @blizzz @jnfrmarks