Project

General

Profile

Anomalie #17113

Erreur lors d'une redirection nominative car les user_id sont des chiffres

Added by Robin SALDINGER about 1 year ago. Updated 12 months ago.

Status:
Développé / Analysé (S)
Priority:
1-Majeur
Assignee:
Florian AZIZIAN
Target version:
Start date:
05/18/2021
Due date:
05/27/2021
Tags Courrier:
21.03.4

Description

Bonjour,

Depuis l'injection des users et entities du client, l'erreur suivante apparaît lors d'une redirection vers un utilisateur; empêchant l'action d'avoir lieu :


Analyse (avec Ludovic) :

--> Suite à des tests effectués via rested, il s'avere que ce problème est survient car les user_id des utilisateurs de la cnam sont des chiffres.

ex : avec l'utilisateur "074349" - NOT OK
url = http://2103si.maarchcourrier.com/cnam/rest/resourcesList/users/154/groups/8/baskets/15/actions/539
mdp cchaplin = maarch
requete = {"resources":[117],"data":{"onlyRedirectDest":true,"listInstances":[{"difflist_type":"entity_id","item_mode":"dest","item_type":"user_id","item_id":"07349","labelToDisplay":"PAVOT CELINE","descriptionToDisplay":"Direction du réseau administratif et de la contractualisation"}]},"note":{"content":"","entities":[]}}

erreur :

ex : avec superadmin - OK
url = http://2103si.maarchcourrier.com/cnam/rest/resourcesList/users/154/groups/8/baskets/15/actions/539
mdp cchaplin = maarch
requete = {"resources":[117],"data":{"onlyRedirectDest":true,"listInstances":[{"difflist_type":"entity_id","item_mode":"dest","item_type":"user_id","item_id":"superadmin","labelToDisplay":"PAVOT CELINE","descriptionToDisplay":"Direction du réseau administratif et de la contractualisation"}]},"note":{"content":"","entities":[]}}

History

#2 Updated by Emmanuel DILLARD about 1 year ago

  • Status changed from A qualifier to A étudier
  • Assignee changed from Emmanuel DILLARD to Florian AZIZIAN

#3 Updated by Florian AZIZIAN about 1 year ago

  • Project changed from Backlog to CURRENT SPRINT
  • Due date set to 05/27/2021
  • Status changed from A étudier to Prêt à développer
  • Assignee deleted (Florian AZIZIAN)
  • Target version changed from 21.03 (stable) to Courrier Develop
  • Tags Courrier 21.03.4 added

Il y a effectivement une condition is_numeric dans la fonction updateListInstance de src/app/entity/controllers/ListInstanceController.php
Vérifier si cette condition est vraiment nécessaire

#4 Updated by Florian AZIZIAN about 1 year ago

  • Status changed from Prêt à développer to En cours de dev (S)

#5 Updated by Florian AZIZIAN about 1 year ago

  • Assignee set to Florian AZIZIAN

#6 Updated by GIT LAB about 1 year ago

Commit ajouté sur la branche develop de MaarchCourrier
FIX #17113 TIME 1 Can not redirect document when user_id contains only number
https://labs.maarch.org/maarch/MaarchCourrier/commit/abfe61a9f872d2b2cb2dc119711589ac2d45dbe7

#8 Updated by Florian AZIZIAN about 1 year ago

  • Status changed from En cours de dev (S) to A tester (S)

#9 Updated by GIT LAB about 1 year ago

Commit ajouté sur la branche 21.03 de MaarchCourrier
FIX #17113 TIME 0:10 Improve control in user redirection
https://labs.maarch.org/maarch/MaarchCourrier/commit/dabc243f57548665dbb591abf9396f9febe1b4b3

#10 Updated by Damien BUREL about 1 year ago

  • Status changed from A tester (S) to Développé / Analysé (S)

#11 Updated by Emmanuel DILLARD 12 months ago

  • Project changed from CURRENT SPRINT to Backlog

#12 Updated by Emmanuel DILLARD 12 months ago

  • Project changed from Backlog to Backlog Courrier
  • Target version changed from Courrier Develop to Develop

Also available in: Atom PDF