xwikisas / application-activedirectory

Provides the UI to set up LDAP for Active Directory
GNU Lesser General Public License v2.1
1 stars 2 forks source link

Mapping fields can't be deleted when not properly saved #67

Open CamiAndrei opened 1 year ago

CamiAndrei commented 1 year ago

Steps to reproduce:

Results: The field was not properly deleted. I've identified 3 cases depending on the number of fields added, position of the information added:

Even if you make the save with "Save" from the bottom of the page, may not work when this problem appears. The workaround found is to clear the field from XWiki.XWikiPreferences object from ActiveDirectory.CodeActiveDirectoryConfig

Even though "Check connection" was not meant to save the information below it (only the connection details). It saves the details added below it. It shouldn't save the information below it. This is also related with: #65 . Removing the Save for all fields on Check Connection could break some others.