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

Cannot remove owners/editors when their external data source as removed #858

Closed
aepli opened this issue Feb 3, 2020 · 2 comments · Fixed by #905
Closed

Cannot remove owners/editors when their external data source as removed #858

aepli opened this issue Feb 3, 2020 · 2 comments · Fixed by #905
Assignees
Labels
Milestone

Comments

@aepli
Copy link
Contributor

aepli commented Feb 3, 2020

Version

Sympa 6.2.48

Installation method

From source

Expected behavior

When removing an owners/editors external data source, owners/editors from that source should be deleted from the owners/editors of the distribution list. The synchronization task should be deleted from the task folder.

Actual behavior

After removing an owners/editors external data source, they remain present as the owners/editors of the distribution list and cannot be deleted from the web interface (no checkbox). Currently, they must be removed from 'admin_table' in the database.

Additional information

OS: Debian GNU/Linux 9.11 (stretch)
Perl: 5.24.1-3+deb9u5 amd64

@ikedas ikedas added the bug label Feb 6, 2020
@ikedas ikedas self-assigned this Feb 15, 2020
@ikedas ikedas added this to the 6.2.56 milestone Feb 15, 2020
@ikedas
Copy link
Member

ikedas commented Feb 29, 2020

Hi @aepli ,

Could you please apply this patch and check if the problem will be solved? Thanks.

@aepli
Copy link
Contributor Author

aepli commented Mar 5, 2020

Hi @ikedas,

I tested the fix in the issue #857 .
You will find my comments there.

Regards.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants