fusiondirectory-plugins issueshttps://gitlab.fusiondirectory.org/fusiondirectory/fd-plugins/-/issues2022-12-07T10:40:06Zhttps://gitlab.fusiondirectory.org/fusiondirectory/fd-plugins/-/issues/6023If mailbox creation failed at user creation, you get stuck2022-12-07T10:40:06ZJonathan SwaelensIf mailbox creation failed at user creation, you get stuck### Description
I was trying to create a user with a mailbox (on cyrus). The mailbox creation failed but when I wanted apply again FusionDirectory told me that an user with my uid already exist.
### Distribution Name and Version
Debia...### Description
I was trying to create a user with a mailbox (on cyrus). The mailbox creation failed but when I wanted apply again FusionDirectory told me that an user with my uid already exist.
### Distribution Name and Version
Debian Buster
### FusionDirectory Version
1.4
### Plugin with the defect
mail
### PHP version used
7
### Origin of php packages
Distribution
### Steps to Reproduce
1. Set fdMailAttribute on mail and disable cyrus unix style
2. Create an user with a mail on cyrus server
3. If you try to apply again after the mailbox creation error it will say the uid already exist
**Expected behavior:**
Not create the user if the mailbox creation fail
**Actual behavior:**
It create the user same if the mailbox creation fail
**Reproduces how often:**
100%FusionDirectory 1.5Jonathan SwaelensJonathan Swaelens2019-12-18https://gitlab.fusiondirectory.org/fusiondirectory/fd-plugins/-/issues/6104Mail methods refactor2022-03-03T11:00:44ZCôme ChillietMail methods refactorMail method should be refactored so that:
- We avoid duplicated code in mailAccount/mailGroup/sympaAliasPartage
- We avoid connecting to the mail server when it’s not needed
- Even lazily load quota info to speed-up account edition?
- Su...Mail method should be refactored so that:
- We avoid duplicated code in mailAccount/mailGroup/sympaAliasPartage
- We avoid connecting to the mail server when it’s not needed
- Even lazily load quota info to speed-up account edition?
- Support correctly server change (partly done)
- Maybe improve server change with the same method?
- Hide non-supported fields (or check that this is the case already)FusionDirectory 1.5dockx thibaultdockx thibault