fusiondirectory-plugins issueshttps://gitlab.fusiondirectory.org/fusiondirectory/fd-plugins/-/issues2020-07-22T12:18:33Zhttps://gitlab.fusiondirectory.org/fusiondirectory/fd-plugins/-/issues/5908Double check usages of count()2020-07-22T12:18:33ZCôme ChillietDouble check usages of count()To avoid problems similar to fd#5942To avoid problems similar to fd#5942FusionDirectory 1.3Côme ChillietCôme Chilliethttps://gitlab.fusiondirectory.org/fusiondirectory/fd-plugins/-/issues/5893sinaps plugin should have a main.inc2019-01-10T09:33:16Zbmortiersinaps plugin should have a main.inc### Description
sinaps plugin should have a main.inc
### Distribution Name and Version
Debian Stretch
### FusionDirectory Version
1.3-dev
### Plugin with the defect
sinaps
### PHP version used
7.0+49
### Origin of php packages
...### Description
sinaps plugin should have a main.inc
### Distribution Name and Version
Debian Stretch
### FusionDirectory Version
1.3-dev
### Plugin with the defect
sinaps
### PHP version used
7.0+49
### Origin of php packages
Debian stretch
### Steps to Reproduce
<!-- Required -->
1. ckick on sinaps in my account
you get
```
Plugin
Erreur fatale : Impossible de trouver une définition pour le plugin '?plug=188' ('..//plugins/personal/sinaps/main.inc' n'est pas un fichier) !
Veuillez corriger l'erreur ci-dessus et recharger la page.
```
**Expected behavior:**
having the same text as definied in ticket #5892
**Actual behavior:**
the plugin crash
**Reproduces how often:**
100%FusionDirectory 1.3Côme ChillietCôme Chilliethttps://gitlab.fusiondirectory.org/fusiondirectory/fd-plugins/-/issues/5885supannCodeEntiteParent should allow multiple values2019-01-16T08:35:45ZCôme ChillietsupannCodeEntiteParent should allow multiple valuesThe SupAnn norm says it can contain several values.The SupAnn norm says it can contain several values.FusionDirectory 1.3Côme ChillietCôme Chilliethttps://gitlab.fusiondirectory.org/fusiondirectory/fd-plugins/-/issues/5872Supann labels and attribute type review2018-11-08T13:38:35ZCôme ChillietSupann labels and attribute type review**SupAnn** should always be written this way
Phone numbers should use PhoneNumberAttribute
URLs should use URLAttribute**SupAnn** should always be written this way
Phone numbers should use PhoneNumberAttribute
URLs should use URLAttributeFusionDirectory 1.3Côme ChillietCôme Chilliethttps://gitlab.fusiondirectory.org/fusiondirectory/fd-plugins/-/issues/5861Update the whole copyright to 20182018-09-07T10:56:21ZbmortierUpdate the whole copyright to 2018Hello,
we need to update the copyright to 2018
A+Hello,
we need to update the copyright to 2018
A+FusionDirectory 1.3bmortierbmortierhttps://gitlab.fusiondirectory.org/fusiondirectory/fd-plugins/-/issues/5855Change the partage icon to be the new one2018-09-07T10:57:26ZbmortierChange the partage icon to be the new one## Descriptive title for this enhancement
Change the partage icon to be the new one
### Actual behavior
Actually we have the old partage icon in fusiondirectory
### Expected behavior
Use the new partage icon
### Benefits
Be in syn...## Descriptive title for this enhancement
Change the partage icon to be the new one
### Actual behavior
Actually we have the old partage icon in fusiondirectory
### Expected behavior
Use the new partage icon
### Benefits
Be in sync with the official partage design
### Possible Drawbacks
noneFusionDirectory 1.3bmortierbmortierhttps://gitlab.fusiondirectory.org/fusiondirectory/fd-plugins/-/issues/5849fusiondirectory-plugin-renater-partage : gosaMailAlternateAddress domain suff...2018-08-09T11:37:30Zagallavardinfusiondirectory-plugin-renater-partage : gosaMailAlternateAddress domain suffix domain must be checked### Description
When using the partage plugin, all mail inside the gosaMailAlternateAddress must have the same domain mail as the domain mail specified in partage configuration.
this check must be done for group, sympaAlias and user gos...### Description
When using the partage plugin, all mail inside the gosaMailAlternateAddress must have the same domain mail as the domain mail specified in partage configuration.
this check must be done for group, sympaAlias and user gosaMailAlternateAddress
### Distribution Name and Version
Debian stretch
### FusionDirectory Version
fusiondirectory (1.3-2~jenkinsbuild37)
### Plugin with the defect
fusiondirectory-plugin-renater-partage-schema (1.3-2~jenkinsbuild37)
### PHP version used
php7.0
### Origin of php packages
Debian stretch packages
### Steps to Reproduce
1. create a sympAlias
2. add partage information
3. add partage alternate adress with an domain different diffrent from partage settings ( @free.fr instead of @irstea.fr)
**Expected behavior:**
A error message should occurs like : :
an alternate e-mail address didn't correspond to the domain name specified on mail serveur ( allowed value : (server domain name here!))
**Actual behavior:**
Error 500 when partage plugin try to add the alternate adress
**Reproduces how often:**
100%FusionDirectory 1.3Côme ChillietCôme Chilliethttps://gitlab.fusiondirectory.org/fusiondirectory/fd-plugins/-/issues/5848Matching mail suffix between PARTAGE and FD2018-08-08T12:27:39ZagallavardinMatching mail suffix between PARTAGE and FD### Description
When adding or modifying, partage information related to Sympa Alias, it doesn't work because PARTAGE receive a non-valid request
### Distribution Name and Version
Debian Stretch
### FusionDirectory Version
`ii fusio...### Description
When adding or modifying, partage information related to Sympa Alias, it doesn't work because PARTAGE receive a non-valid request
### Distribution Name and Version
Debian Stretch
### FusionDirectory Version
`ii fusiondirectory 1.3-2~jenkinsbuild28 all Web Based LDAP Administration Program`
### Plugin with the defect
```
ii fusiondirectory-plugin-renater-partage 1.3-2~jenkinsbuild28 all Renater partage integration plugin for FusionDirectory
ii fusiondirectory-plugin-renater-partage-schema 1.3-2~jenkinsbuild28 all LDAP schema for FusionDirectory renater partage plugin
```
### PHP version used
`ii php7.0 7.0.30-0+deb9u1 all server-side,`
### Origin of php packages
From deb :
```
php7.0:
Installé : 7.0.30-0+deb9u1
Candidat : 7.0.30-0+deb9u1
Table de version :
*** 7.0.30-0+deb9u1 500
500 http://ftp.fr.debian.org/debian stretch/main amd64 Packages
500 http://security.debian.org/debian-security stretch/updates/main amd64 Packages
100 /var/lib/dpkg/status
```
### Steps to Reproduce
1. add a sympa alias list without partage information
2. Add partage information
3. Wait the error
**Expected behavior:**
Partage information must be created / updated on Partage Infra
**Actual behavior:**
Error :
on browser
`Impossible de mettre à jour le compte : Impossible de se connecter à https://api.partage.renater.fr/service/domain/: fopen(https://api.partage.renater.fr/service/domain//CreateGroup/22f97c76cf9a11405cdb5e740fc0aad3): failed to open stream: HTTP request failed! HTTP/1.1 403 Forbidden `
with debug :
```
DEBUG[512] (/usr/share/fusiondirectory/plugins/personal/mail/mail-methods/class_mail-methods-renater-partage.inc:query():85) - MAIL: Command: 'CreateGroup'
DEBUG[512] (/usr/share/fusiondirectory/plugins/personal/mail/mail-methods/class_mail-methods-renater-partage.inc:query():86) - MAIL: Query: 'name=fakelist%40lists.irstea.fr&description=liste+de+fake&displayName=FAKE+LIST&zimbraMailStatus=enabled&zimbraHideInGal=TRUE&zimbraDistributionListSendShareMessageToNewMembers=FALSE'
DEBUG[512] (/usr/share/fusiondirectory/plugins/personal/mail/mail-methods/class_mail-methods-renater-partage.inc:query():142) - MAIL: No Answer:
```
**Reproduces how often:**
100%
### Additional Information
Basically :
* our partage domain is : company.fr ( all created object as user, group .. MUST have the mail suffix @company.fr
* our Sympa domain list is : lists.company.fr
We want to have an alias for 2 reasons
* appearing lists in global catalogue of PARTAGE
* have an alias mylist@company.fr to mylist@lists.company.fr
so the group created on PARTAGE side MUST have the suffix @company.fr and a forwarding address to mylist@lists.company.fr
On debug we see :
MAIL: Query: 'name=fakelist%40lists.irstea.fr&descri ....
`'name=fakelist%40lists.irstea.fr isn"t correct !! It must be name=fakelist%40irstea.fr`
the mail domain must be taken from the PARTAGE COS used.
After creation of this group a new request is needed to add as member of fakelist%40irstea.fr the adress contain in CN field : fakelist%40lists.irstea.frFusionDirectory 1.3Côme ChillietCôme Chilliethttps://gitlab.fusiondirectory.org/fusiondirectory/fd-plugins/-/issues/5837More feature switch should be added to mail methods2019-03-06T11:17:26ZCôme ChillietMore feature switch should be added to mail methodsMail methods have a feature switch for vacation message, they should have some as well for alternate emails, forwarding and local only features.
This will allow to deactivate from Partage method features not supported by it.Mail methods have a feature switch for vacation message, they should have some as well for alternate emails, forwarding and local only features.
This will allow to deactivate from Partage method features not supported by it.FusionDirectory 1.3Côme ChillietCôme Chilliethttps://gitlab.fusiondirectory.org/fusiondirectory/fd-plugins/-/issues/5835add other source capabilities for supannEmpCorp2019-01-14T14:11:44Zagallavardinadd other source capabilities for supannEmpCorp### Actual behavior
actually supannEmpCorp contient values extract from BCN http://www.infocentre.education.fr/bcn/domaine/voir/id/30#
syntax is like : {NCORPS}836
## Expected behavior
Even if it isn't specified in https://services.re...### Actual behavior
actually supannEmpCorp contient values extract from BCN http://www.infocentre.education.fr/bcn/domaine/voir/id/30#
syntax is like : {NCORPS}836
## Expected behavior
Even if it isn't specified in https://services.renater.fr/documentation/supann/2009/documentcomplet
Could we add the behaviour to chose the list of value like
by the way, I think that this enhancement is necessary and planned on futur Supann : https://services.renater.fr/documentation/supann/supann2018/les_travaux_en_cours
### Benefits
add custom library of Corps
### Possible Drawbacks
duplicate item between NCORPS ans custom library but it's a user problemFusionDirectory 1.3Côme ChillietCôme Chilliethttps://gitlab.fusiondirectory.org/fusiondirectory/fd-plugins/-/issues/5814Adapt code to new Language class2018-09-07T11:07:50ZCôme ChillietAdapt code to new Language classBecause of refactor from fd#5820 some code needs to be adapted.Because of refactor from fd#5820 some code needs to be adapted.FusionDirectory 1.3Côme ChillietCôme Chilliethttps://gitlab.fusiondirectory.org/fusiondirectory/fd-plugins/-/issues/5811Missing SUBSTR matching rules for DN foreign keys2018-04-19T07:30:00ZCôme ChillietMissing SUBSTR matching rules for DN foreign keys### Description
<!-- Required -->
<!-- Description of the issue -->
The fields having a foreignKey on DN values need a SUBSTR matching rule.
See fd#5799### Description
<!-- Required -->
<!-- Description of the issue -->
The fields having a foreignKey on DN values need a SUBSTR matching rule.
See fd#5799FusionDirectory 1.3Côme ChillietCôme Chilliethttps://gitlab.fusiondirectory.org/fusiondirectory/fd-plugins/-/issues/5772Adapt code to new snapshots ACL system2018-03-27T09:10:37ZCôme ChillietAdapt code to new snapshots ACL systemSee fusiondirectory/fd#5743See fusiondirectory/fd#5743FusionDirectory 1.3Côme ChillietCôme Chilliethttps://gitlab.fusiondirectory.org/fusiondirectory/fd-plugins/-/issues/5767systems-fd.schema classes should allow "description" and "l" fields2018-01-16T15:06:42ZJonathan Swaelenssystems-fd.schema classes should allow "description" and "l" fields### Description
ObjectClass ipHost is linked to description and location
So we cannot let it empty and add description and location
### Distribution Name and Version
Ubuntu
### FusionDirectory Version
1.3-dev
### Plugin with the de...### Description
ObjectClass ipHost is linked to description and location
So we cannot let it empty and add description and location
### Distribution Name and Version
Ubuntu
### FusionDirectory Version
1.3-dev
### Plugin with the defect
Systems
### PHP version used
7.0.22
### Origin of php packages
Ubuntu official
### Steps to Reproduce
1. Set workstation ip as optionnal
2. Create a workstation with only cn and description
3. Read the error
**Expected behavior:**
Have ipHost as optionnal and set description and location
**Actual behavior:**
ipHost is mandatory if we want add description and location
**Reproduces how often:**
100%
### Additional Information
Related to fd-plugins#5705FusionDirectory 1.3Côme ChillietCôme Chilliethttps://gitlab.fusiondirectory.org/fusiondirectory/fd-plugins/-/issues/5735Viewing the log from the argonaut queue should not be fai dependent2018-04-05T12:20:18ZbmortierViewing the log from the argonaut queue should not be fai dependent### Requirements
* Filling out the template is required. Any Enhancement request that does not include enough information to be reviewed in a timely manner may be closed at the maintainers' discretion.
* All new code requires tests to e...### Requirements
* Filling out the template is required. Any Enhancement request that does not include enough information to be reviewed in a timely manner may be closed at the maintainers' discretion.
* All new code requires tests to ensure against regressions
## Descriptive title for this enhancement
Viewing the log from the argonaut queue should not be fai dependent
### Actual behavior
when clicking on view log in the argonaut queue, we get and error if fai is not installed
![image3](/uploads/96e91d481feb535ea06b4d99f86e0c6b/image3.png)
### Expected behavior
Not crashing and preferably be able to show the kind of log for the type op systems being installed
* fai
* opsi
### Step by step description of new behavior
<!-- Required -->
1. Click on the log icon
2. get the log from the kind of system installing
### Benefits
Be able to show log from either fai or opsi systems being installed
### Possible Drawbacks
Maybe difficult as we don't have the concept of type of systems inside the argonaut protocol.
### Applicable Issues
deploying mixed computers environmentFusionDirectory 1.3Côme ChillietCôme Chilliethttps://gitlab.fusiondirectory.org/fusiondirectory/fd-plugins/-/issues/5728Attribute gotoXdmcpServer is single-value in schema but multi-value in FD2017-12-14T19:57:24ZCôme ChillietAttribute gotoXdmcpServer is single-value in schema but multi-value in FD### Description
<!-- Required -->
<!-- Description of the issue -->
gotoXdmcpServer is marked as SINGLE-VALUE is systems-fd.schema but appears as a SetAttribute in systems plugin terminal tab.
### FusionDirectory Version
<!-- Required...### Description
<!-- Required -->
<!-- Description of the issue -->
gotoXdmcpServer is marked as SINGLE-VALUE is systems-fd.schema but appears as a SetAttribute in systems plugin terminal tab.
### FusionDirectory Version
<!-- Required -->
1.3-dev
### Plugin with the defect
<!-- Required -->
<!-- Name of the plugin -->
systems
### Steps to Reproduce
<!-- Required -->
1. Add several values for gotoXdmcpServer
**Expected behavior:**
<!-- What you expect to happen-->
We should be able to save only one gotoXdmcpServer or multiple without error.
**Actual behavior:**
<!-- What actually happens -->
Error: Constraint violation (gotoXdmcpServer: multiple values provided, while operating on 'cn=terminal,ou=terminals,ou=systems,dc=fusiondirectory,dc=org' using LDAP server 'ldap://demo-fixes.fusiondirectory.org')FusionDirectory 1.3Côme ChillietCôme Chilliethttps://gitlab.fusiondirectory.org/fusiondirectory/fd-plugins/-/issues/5723Split posix and ppolicy alert email settings2017-12-08T15:30:51ZCôme ChillietSplit posix and ppolicy alert email settings## Split posix and ppolicy alert email settings
<!-- required -->
See https://gitlab.fusiondirectory.org/fusiondirectory/fd-plugins/wikis/UserReminder and argonaut/argonaut#5701
### Actual behavior
<!-- What actually happens -->
Same ...## Split posix and ppolicy alert email settings
<!-- required -->
See https://gitlab.fusiondirectory.org/fusiondirectory/fd-plugins/wikis/UserReminder and argonaut/argonaut#5701
### Actual behavior
<!-- What actually happens -->
Same settings
### Expected behavior
<!-- What you expect to happen-->
New section for ppolicy reminderFusionDirectory 1.3Côme ChillietCôme Chilliethttps://gitlab.fusiondirectory.org/fusiondirectory/fd-plugins/-/issues/5722move how_to_contribute page from the wiki to developper documentation2018-04-05T12:12:40Zbmortiermove how_to_contribute page from the wiki to developper documentation### Requirements
* Filling out the template is required. Any Enhancement request that does not include enough information to be reviewed in a timely manner may be closed at the maintainers' discretion.
* All new code requires tests to e...### Requirements
* Filling out the template is required. Any Enhancement request that does not include enough information to be reviewed in a timely manner may be closed at the maintainers' discretion.
* All new code requires tests to ensure against regressions
## Descriptive title for this enhancement
Rewrite the page https://documentation.fusiondirectory.org/en/how_to_contribute, to help people contribute
### Actual behavior
The page is old an unmaintained, the part about translation should go into the developper manual
### Expected behavior
Have a link that send to the correct part of the documation on read the docs
### Step by step description of new behaviour
<!-- Required -->
1. Write the missing part in the developper documentation
2. Remove old content from the wiki, and link to the developper documentation
### Benefits
Have all development, contribution stored in one place
### Possible Drawbacks
none
### Applicable Issues
development and translations taken from https://gitlab.fusiondirectory.org/fusiondirectory/fd/issues/5723FusionDirectory 1.3bmortierbmortierhttps://gitlab.fusiondirectory.org/fusiondirectory/fd-plugins/-/issues/5714Adapt developers plugin to new plugin convention2018-04-05T12:07:35ZbmortierAdapt developers plugin to new plugin convention### Description
The remaining part of the developers plugin, debug-help is not correctly named followin our plugin convention
### Distribution Name and Version
Debian jessie
### FusionDirectory Version
1.3-dev
### Plugin with the d...### Description
The remaining part of the developers plugin, debug-help is not correctly named followin our plugin convention
### Distribution Name and Version
Debian jessie
### FusionDirectory Version
1.3-dev
### Plugin with the defect
developers plugin
### PHP version used
5.6
### Origin of php packages
Debian jessie
### Steps to Reproduce
just look at the sources
**Expected behavior:**
conform to plugin hierarchical directories
**Actual behavior:**
name of the folder is name of the plugin
**Reproduces how often:**
100%
### Additional Information
Packaging will ne to be adapted after thatFusionDirectory 1.3bmortierbmortierhttps://gitlab.fusiondirectory.org/fusiondirectory/fd-plugins/-/issues/5705Make mandatory IPs a configuration option in the systems plugins2018-07-19T10:40:15ZbmortierMake mandatory IPs a configuration option in the systems plugins### Description
It bugs me is having the IP field mandatory only in some cases, we should put it either mandatory for all systems or none, or have an option in the configuration back-end.
If it’s non-mandatory by default we can make it...### Description
It bugs me is having the IP field mandatory only in some cases, we should put it either mandatory for all systems or none, or have an option in the configuration back-end.
If it’s non-mandatory by default we can make it mandatory when some tabs are activated (argonautClient for instance needs it).
### FusionDirectory Version
1.3-dev
### Plugin with the defect
plugin-systems
### PHP version used
5.6
### Origin of php packages
debian
### Steps to Reproduce
<!-- Required -->
1. Create a system
2. try to save without an ip
3. doesnt work
**Expected behavior:**
- Make it non mandatory by default, code wise.
- Have an option in configuration back-end for each of the plugin thats need ip address.
- Active the option by default, to not change the fonctional usage of people already using it but allowing people to change it if needed.
**Actual behavior:**
ip is mandatory on all systems and we cannot change that
**Reproduces how often:**
100%
### Additional Information
Make it a configuration option since we changed this too many times already.
Multi-valuated configuration field listing the types for which IP is mandatory.
Note that ipHost have the field ipHostNumber as mandatory so we’ll need to add code to remove this objectClass when the field is empty. This is already what we are doing for phones.FusionDirectory 1.3Côme ChillietCôme Chilliet