Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bib_noms et vm_taxref_list_forautocomplete ? #227

Closed
jbrieuclp opened this issue Jun 26, 2020 · 5 comments
Closed

Bib_noms et vm_taxref_list_forautocomplete ? #227

jbrieuclp opened this issue Jun 26, 2020 · 5 comments

Comments

@jbrieuclp
Copy link

Il n'y a pas plus de référence vers la table bib_nom dans la création de cette vue, c'est volontaire ou c'est un oubli ?

CREATE MATERIALIZED VIEW taxonomie.vm_taxref_list_forautocomplete AS
SELECT
row_number() OVER() as gid,
t.cd_nom,
t.cd_ref,
t.search_name,
t.nom_valide,
t.lb_nom,
t.regne,
t.group2_inpn
FROM (
-- PARTIE NOM SCIENTIFIQUE : ici on prend TOUS les synonymes.
SELECT
t_1.cd_nom,
t_1.cd_ref,
concat(t_1.lb_nom, ' = <i> ', t_1.nom_valide, '</i>', ' - [', t_1.id_rang, ' - ', t_1.cd_nom , ']') AS search_name,
t_1.nom_valide,
t_1.lb_nom,
t_1.regne,
t_1.group2_inpn
FROM taxonomie.taxref t_1
UNION
-- PARTIE NOM FRANCAIS : ici on prend une seule fois (DISTINCT) dans Taxref tous les taxons de références
-- On ne prend pas les taxons qui n'ont pas de nom vern dans taxref,
-- donc si un taxon n'a pas de nom vern dans Taxref, il n'est accessible que par son nom scientifique.
SELECT DISTINCT
t_1.cd_nom,
t_1.cd_ref,
concat(split_part(t_1.nom_vern, ',', 1), ' = <i> ', t_1.nom_valide, '</i>', ' - [', t_1.id_rang, ' - ', t_1.cd_ref , ']' ) AS search_name,
t_1.nom_valide,
t_1.lb_nom,
t_1.regne,
t_1.group2_inpn
FROM taxonomie.taxref t_1
WHERE t_1.nom_vern IS NOT null and t_1.cd_nom = t_1.cd_ref
) t;
COMMENT ON MATERIALIZED VIEW taxonomie.vm_taxref_list_forautocomplete
IS 'Vue matérialisée permettant de faire des autocomplete construite à partir d''une requete sur tout taxref.';

Avant il y avait cette jointure :

JOIN taxonomie.bib_noms n ON n.cd_nom = t.cd_nom OR n.cd_ref = t.cd_ref
JOIN taxonomie.cor_nom_liste cnl ON cnl.id_nom = n.id_nom;

@camillemonchicourt
Copy link
Member

Oui c'est volontaire.
Comme ça, on met à plat tout Taxref une bonne fois pour toute et ensuite on est tranquille, indépendamment de bib_noms, dont on veut s'éloigner le plus possible, jusqu'à idéalement s'en séparer un jour. :-)

Voir #163 et #111

@camillemonchicourt camillemonchicourt changed the title Bib_noms ? Bib_noms et vm_taxref_list_forautocomplete ? Jun 26, 2020
@jbrieuclp
Copy link
Author

Oui, ok, c'est ce qui est exprimé dans les commentaires du script, j'allais modifier mon issue, mais tu as répondu trop vite. Mais du coup est-ce qu'il est toujours possible d'utiliser une liste de taxons à mettre à dispo à la saisie par exemple ?

@DonovanMaillard
Copy link
Contributor

Pour la saisie, ca passe par les listes, par exemple la liste Saisie Occtax qui est forcément définie dans taxhub. Ca ne semble pas avoir changé de ce coté :)

@jbrieuclp
Copy link
Author

si j'ai bien pigé (sans aller voir le code), la jointure sur les listes est faite en backend taxhub via la route get_AllTaxrefNameByListe(id_liste)
#219 (comment)

@camillemonchicourt
Copy link
Member

La route fonctionne pareil et renvoie toujours le même résultat. Pas de changement côté GeoNature.
C'est juste sa source qui a évolué.
Exemple : http://demo.geonature.fr/taxhub/api/taxref/allnamebylist/100?limit=100&offset=0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants