Problème de concaténation sur valeur vide

[jbrieuclp]

Salut il faudrait modifier cette ligne qui concatène nom et prénom avec l'opérateur ||

UsersHub/data/usershub.sql

Line 308 in ff0f07a

(upper(a.nom_role::text) || ' '::text) || a.prenom_role::text AS nom_complet,

par celle-ci
concat_ws(' '::text, upper(NULLIF(a.nom_role::text, ''::text)), NULLIF(a.prenom_role::text, ''::text)) AS nom_complet,
L'opérateur de concaténation || renvoi une chaîne vide si l'un ou l'autre est vide (ce qui peut arriver sur un prénom).

[amandine-sahl]

Ce n'est pas la peine d'utiliser NULLIF il est géré par la fonction CONCAT

[jbrieuclp]

C'est plutôt s'il y a une chaîne vide, si c'est le cas concat_ws l'espace sera ajouté :

select concat_ws(' ', 'truc', '') => 'truc '
select concat_ws(' ', 'truc', NULLIF('', '')) => 'truc'

[amandine-sahl]

ok, effectivement.
Par ailleurs, c'est préférable de remplacer les chaines vides par des NULL.

[jbrieuclp]

T'as complètement raison sur ce point

[camillemonchicourt]

OK, ajouter un script d'update nettoyant cela.