Error when identity is spread from SINAPS "Cannot delete non-existing user {LDAPUUID}"

Description

strange error during "diffusion" : spread an identity from SINAPS to FD on SINAPS side : identity didn't have FD cross-reference on FD side : account exist (tested with a prefilled LDAPUUID and without a pre filled LDAPUUID)

Distribution Name and Version

Debian 9.8

FusionDirectory Version

1.3 from integration

Plugin with the defect

fusiondirectory-sinaps 1.3.1 from gz from sources plugins install with fusiondirectory-setup --plugins

PHP version used

php 7

Origin of php packages

from oficiel Deb repos

Steps to Reproduce

1. spread to FusionDirectory an identity with no cross reference, on FD there a corresponding user ( same sn and givenname) with a pre filled LDAPUUID
2. check the xml-acquittement

Expected behavior:

on current installation ( 1.3 ) we have an error like "The entry 'uid=john.doe,ou=people,ou=site,dc=company,dc=fr' already use this 'Login' attribute!b8048447-f719-4c2d-bb9e-b94dffcb9eba.xml

3f646d29-e346-4095-a6b2-ef61d93b6619-acquittement.xml

b8048447-f719-4c2d-bb9e-b94dffcb9eba-acquittement.xml

3f646d29-e346-4095-a6b2-ef61d93b6619.xml

Actual behavior:

we have the following error :

Erreur Fonctionnelle - Code réponse 200 : Données pour la transaction 4e5a42f3-7d5e-4146-aad0-87b5f8bb6096 non intégrées - Cannot delete non-existing user {LDAPUUID}e2dda2de-c842-4e8f-8163-d0c174fd4ddc

If diffusion is done again the LDAPUUID in error message change . those LDAPUUID didn't exist inside LDAP tree

Reproduces how often:

100 % with the same user

assigned to @MCMic

Same error occurs when trying to create an account on FD from SINAPS 1.3.1 release seems broken ?

I think I found the bug. a casting is missing ( transform xml leaf object in string) (dirty?) patch as attachement[]class_sinapsRequest.inc.diff

changed milestone to %FusionDirectory 1.3.1

added PJ1807-0216 plugin-sinaps labels

assigned to @bmortier and unassigned @MCMic

hello @gallak,

1.3.1 doesn't exist right now, you mean you used 1.3-fixes packaged debian ?

Cheers

added 5m of time spent at 2019-04-23

changed due date to April 25, 2019

assigned to @MCMic and unassigned @bmortier

mentioned in commit b0b73671

changed title from wrong error when Identity is spread to FD from SINAPS "Cannot delete non-existing user {LDAPUUID}" to Error when identity is spread from SINAPS "Cannot delete non-existing user {LDAPUUID}"

Added the missing casts

added 1h of time spent at 2019-04-25

added To Be Tested label

it seeams that not all the cases were manager ( case of PRA)

we have again error :

<?xml version="1.0" encoding="UTF-8"?>
<acq:Acquittement xmlns:acq="http://referentiels.SINAPS.amue.fr/acquittementFonctionnel">
  <ResponseCode>200</ResponseCode>
  <messageAcquittement>Cannot delete non-existing user {LDAPUUID}70643af0-1f9c-4305-896d-e36fb2e2ad29</messageAcquittement>
  <codeAcquittement>16</codeAcquittement>
  <identifiantObjApp/>
</acq:Acquittement>

I f we relaunched diffusion the LDAPPUID code change as attachement an PRA example cases corto.maltese-PRA.xml

If identity came as PR , it's works !