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.

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

Users and Groups mappings are not displayed after save #26

Closed
ilieandriuta opened this issue Mar 9, 2020 · 0 comments
Closed

Users and Groups mappings are not displayed after save #26

ilieandriuta opened this issue Mar 9, 2020 · 0 comments
Assignees
Milestone

Comments

@ilieandriuta
Copy link

STEPS TO REPRODUCE

  1. Go to Administer Wiki > Other > Active Directory
  2. In Advanced Configuration section, set a mapping for User fields and a mapping for Groups
  3. Click 'Save'

EXPECTED RESULTS

After save, the previously set mappings are displayed in their corresponding section (e.g. User fields and/or Groups).

ACTUAL RESULTS

The mappings are not displayed anymore:

Before SAVE:
Before_save_AD

After SAVE:
After_save_AD

Environment: Edge 80, using a local instance XWiki 12.1 on PostgreSQL 12, Docker

@ilieandriuta ilieandriuta added this to the 1.8 milestone Mar 9, 2020
@acotiuga acotiuga self-assigned this Mar 10, 2020
acotiuga added a commit that referenced this issue Mar 10, 2020
* the value stored in the configuration document was not added in the input values
* improve variables naming to increase code readability
acotiuga added a commit that referenced this issue Mar 10, 2020
acotiuga added a commit that referenced this issue Mar 24, 2020
* code refactor: improve jquery code organization by extracting individual functionalities into functions
acotiuga added a commit that referenced this issue Mar 24, 2020
* migrate the code to jQuery
* make sure to update the mapping values in textarea when the input value is updated
* remove leftover styling from XWiki 8.4 (get rid of "half" class on form)
acotiuga added a commit that referenced this issue Mar 25, 2020
* handle the case of adding a new property mapping
acotiuga added a commit that referenced this issue Mar 25, 2020
* handle the case of adding a new property mapping, when the second input(active directory) is first completed
acotiuga added a commit that referenced this issue Mar 27, 2020
* the values were wrongly saved for the input suggests
* remove leftover code that handled old properties options
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants