Project

General

Profile

Bug #75073

api: ne pas chercher à normaliser la valeur de du paramètre ?q=... lors de la recherche dans les demandes

Added by Serghei Mihai 9 months ago. Updated 9 months ago.

Status:
Fermé
Priority:
Normal
Target version:
-
Start date:
02 March 2023
Due date:
% Done:

0%

Estimated time:
Patch proposed:
No
Planning:
No

Description

Cas d'usage: cellule recherche dans Combo agent et l'agent cherche des demandes par leur numéro.

Lorsque l'agent fait des recherches par le form_number (par exemple 53-542) l'appel https://wcs/api/forms/?status=all&ignore-roles=on&include-anonymised=off&q=53-542 ne retourne pas de résultats même si une telle demande existe.

La demande n'est pas retrouvée car la valeur est interprétée comme numéro de téléphone et est modifiée au niveau du critère FtsMatch:

            phone_match = re.match(r'.*?(?P<phone>\b\+?[-\(\)\d\.\s/]{6,20}\b)', self.value)
            if phone_match:
                # if it looks like a phone number, normalize it to its
                # "international/E164" format to match what's stored in the
                # database.
                phone_value = misc.normalize_phone_number_for_fts(phone_match.group('phone').strip())

Cela donne:

In [2]: FtsMatch('53-542').value
Out[2]: '+3353542'

In [3]: FtsMatch('53-54').value
Out[3]: '53-54'

La normalisation de la valeur de la query n'est peut-être pas nécessaire lors de la recherche des demandes.


Related issues

Related to w.c.s. - Bug #75302: Une recherche sur un numéro dans le listing d'une démarche ne retourne plus rienFermé10 March 2023

Actions

Associated revisions

Revision 1a5b265b (diff)
Added by Frédéric Péters 9 months ago

misc: do not normalize form identifiers as phone numbers (#75073)

History

#2

Updated by Frédéric Péters 9 months ago

  • Assignee set to Frédéric Péters

La normalisation de la valeur de la query n'est peut-être pas nécessaire lors de la recherche des demandes.

Elle est nécessaire pour que la recherche puisse correctement se faire par numéro de téléphone.

Le truc ici serait d'affiner la détection, soit en éliminant de manière spécifique le cas \d+-\d+, soit en adaptant la regex de détection du téléphone.

#3

Updated by Robot Gitea 9 months ago

  • Status changed from Nouveau to Solution proposée

Frédéric Péters (fpeters) a ouvert une pull request sur Gitea concernant cette demande :

#4

Updated by Robot Gitea 9 months ago

  • Status changed from Solution proposée to Solution validée

Lauréline Guérin (lguerin) a approuvé une pull request sur Gitea concernant cette demande :

#5

Updated by Robot Gitea 9 months ago

  • Status changed from Solution validée to Résolu (à déployer)

Frédéric Péters (fpeters) a mergé une pull request sur Gitea concernant cette demande :

#6

Updated by Transition automatique 9 months ago

  • Status changed from Résolu (à déployer) to Solution déployée
#7

Updated by Benjamin Dauvergne 9 months ago

  • Related to Bug #75302: Une recherche sur un numéro dans le listing d'une démarche ne retourne plus rien added
#8

Updated by Transition automatique 7 months ago

Automatic expiration

Also available in: Atom PDF