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

Supprimer id_nom de bib_noms #111

Closed
camillemonchicourt opened this issue Apr 6, 2017 · 8 comments
Closed

Supprimer id_nom de bib_noms #111

camillemonchicourt opened this issue Apr 6, 2017 · 8 comments
Assignees
Milestone

Comments

@camillemonchicourt
Copy link
Member

S'appuyer sur le cd_nom comme clé primaire et donc pour les jointures.

Nécessite un script de MAJ de toutes les tables 1-n et n-n, à moins que les contraintes ON UPDATE fassent le boulot.

@gildeluermoz
Copy link
Contributor

On peut mettre en place les contraintes nécessaires "on update cascade" avant update
puis updater les valeurs du id_nom avec celles du cd_nom.
puis modifier les noms de champs et les contraintes FK de id_nom vers cd_nom
puis supprimer le id_nom.
À tester dans une branche dédiées avec toutes les modifs nécessaires dans Taxhub au préalable.

À bien évaluer : cela veux dire que bib_noms ne peut plus contenir de taxons absents de taxref. Ou qu'il faut mettre en place un mécanisme de tables permettant l'ajout de taxons hors taxref (voir ce que pourrait permettre l'héritage)

Coté GeoNature, il faut faire une revue de code complète pour évaluer les modifications nécessaires : code, triggers, tables de la base, scripts de création de la base.

Le chantier peut être lourd et potentiellement chronophage.

@camillemonchicourt
Copy link
Member Author

Oui à garder au chaud du coup.
Je l'ai noté pour mémoire.
A voir si c'est à faire avant la V2 de GeoNature pour partir sur une base plus propre et simplifiée.
Si on se concentre sur la V2 de GeoNature, c'est peut-être quelque chose qu'@amandine-sahl pourrait faire.
C'est peut-être quelque chose à coupler avec le fait de rendre TaxHub plus générique et moins dépendant de TaxRef, si on veut pouvoir brancher TaxHub sur un autre référentiel taxonomique à terme.

@camillemonchicourt camillemonchicourt added this to the v1.4.0 milestone Sep 13, 2017
@camillemonchicourt
Copy link
Member Author

Voir la suite de la réflexion qui propose de supprimer bib_noms : #163

@amandine-sahl
Copy link
Contributor

bib_noms : suppression de id_nom

Base de données

  • Suppression de la colonne
  • Modification : taxonomie.cor_nom_liste
  • Modification : vues taxonomie.v_bibtaxon_attributs_[regne]
  • Modification : vue taxonomie.v_taxref_all_listes
  • Modification : vue personnel

Taxhub
Remplacer id_nom par cd_nom. Seule information apportée par id_nom : cd_nom présent dans la table bib_noms

Backend

  • model.py : BibNoms
  • model.py : CorNomListe
  • routesbiblistes.py: get_biblistes => changer count
  • routesbiblistes.py: getExporter_biblistesCSV => changer query join
  • routesbiblistes.py: getNoms_bibtaxons => changer select + subq + result
  • routesbiblistes.py: add_cornomliste => changer traitement du paramètre post de id_nom en cd_nom
  • routesbiblistes.py: delete_cornomliste => changer query
  • routesbibnoms.py: getOneSimple_bibtaxons => changer paramètre
  • routesbibnoms.py: getOneFull_bibtaxons => changer paramètre
  • routesbibnoms.py: getBibTaxonSynonymes => changer paramètre
  • routesbibnoms.py: insertUpdate_bibtaxons => changer paramètre + .....
  • routesbibnoms.py: delete_bibtaxons => changer paramètre + .....
  • routestaxref.py: genericTaxrefList => changer select

Frontend

  • bibliste_detail : changer routes vers détail taxon : #!/taxon/ #!/taxonform/edit
  • bibliste-populate : changer id_nom en cd_nom
  • bibNom-detail : changer route form
  • bibNom-form : changer appel api
  • bibNom-list : changer id_nom en cd_nom
  • taxref : remplacer id_nom qui permet de savoir qu'un taxon est ou non présent dans bib_nom par autre chose. A voir avec la route

GéoNature
a évaluer

t_medias :

  • Changer le cd_ref en cd_nom : changement des contraintes et généralisation de la fonction get_cdref pour récupérer les médias d'un taxon.

Contraintes:
-fk_t_media_bib_noms => est-ce utile

@gildeluermoz
Copy link
Contributor

Beau chantier valeureux guérriers !
AU PNE, il faut évaluer si l'id_nom est tjs présent ou pas dans la webapi pour les applications mobile V1 vers la base GN2
Forza !

@camillemonchicourt
Copy link
Member Author

Oui au niveau de GeoNature, on l'a utilisé le moins possible, voire pas du temps en anticipation.
En effet j'avais pas pensé à la webapi de GeoNature-mobile V1. Bien vu !
Cependant on va bientôt passer sur Occtax-mobile, une bonne version est prếte, déployée dans plusieurs structures, dont la réserve du Jura.

@gildeluermoz
Copy link
Contributor

Nickel ! Le Haut-Jura, tjs à la pointe ;)
Yes, pour le PNE, soit tu ne touches pas à la webapi et tu te bloques les montées de version de GN jusqu'au déploiement complet de la mobileV2, soit tu modifies la webapi et tu es libre de mettre en place toutes les évolutions.
Bon courage à tous.

@camillemonchicourt
Copy link
Member Author

La table bib_noms a été supprimée dans la version 2.0.0 de TaxHub. 🎉

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