Améliorations pour une version 0.9.0

rldhont commented 3 years ago

~Ajout d'un champs virtuel longueur aux segments~ (cherry-pick on master)
~Déplacement du champs longueur dans les formulaires~ (cherry-pick on master)
~Renommer l'action split~ (cherry-pick on master)
~Réorganisation de l'arbre des couches~ (CF #101 )
Suppression des champs de subvention sur itinéraires
Mise à jour des styles itinéraires suite à suppression des champs

Gustry commented 3 years ago

Je viens de corriger le test des migrations sur master, il faut rebaser la branche du PR. Il faut faire docker pull kartoza/postgis:10.0-2.4 chez vous pour récupérer la dernière.

Je fais une relecture/test plus tard de la partie Python.

Gustry commented 3 years ago

Rebased sur le PR mais l'installation d'une nouvelle base plante car il faut mettre à jour les données de test. (et donc les tests QGIS plantent aussi à cause de ca)

Gustry commented 3 years ago

J'ai pris le commit "Réorganisation de l'arbre des couches" et refait le python pour éviter la duplication du code dans #101 et pour garder ce PR SQL seulement. PR rebased et suppression du commit cité du PR.

rldhont commented 3 years ago

@Gustry dans ce cas peux-tu extraire les commits suivnats pour ne conserver que le SQL ?

Ajout d'un champs virtuel longueur aux segments
Déplacement du champs longueur dans les formulaires
Renommer l'action split

pdrillin commented 3 years ago

@Gustry @rldhont Pour les erreurs test-qgis vous avez une idée ? En local j'ai testé des trucs mais rien de convainquant

Gustry commented 3 years ago

Ce sont l'import des données de test @drillinP

Les champs étant différents, il faut refaire le "test_data.sql" en y modifiant les champs. C'est pour ca que les tests plantent.

L'installation d'une nouvelle base avec de données de test provoque l'erreur.

Il y a une commande makefile pour les ré-exporter.

pdrillin commented 3 years ago

d'accord

Gustry commented 3 years ago

I have extracted other commits @rldhont and merged into master.

FYI @rldhont and @drillinP the hint to regenerate test data in a working state :

git checkout master
# Install the database with the plugin and ADD test data in it, version 0.8.0
git checkout 0.9.0
# Make migration from 0.8.0 to 0.9.0
make export_test_data
# Edit the file manually to remove QGIS metadata info
# Commit

Now there is only a single test failing :

 ERROR: test_trigger_vitineraire_insert (veloroutes_voies_vertes.test.test_views.TestViews)
Test if v_itineraire view is editable
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/root/.local/share/QGIS/QGIS3/profiles/default/python/plugins/veloroutes_voies_vertes/test/test_views.py", line 80, in test_trigger_vitineraire_insert
    self.cursor.execute(sql)
psycopg2.OperationalError: cannot insert into view "v_itineraire"
DETAIL:  Views containing UNION, INTERSECT, or EXCEPT are not automatically updatable.
HINT:  To enable inserting into the view, provide an INSTEAD OF INSERT trigger or an unconditional ON INSERT DO INSTEAD rule.

Linked to your changes @rldhont ?

Trying to have a look

rldhont commented 3 years ago

Merci @Gustry, je regarde le test

rldhont commented 3 years ago

@Gustry @drillinP SQL corrigé, vous pouvez faire la review

pdrillin commented 3 years ago

@rldhont I found the problem, to correct it you need for each actions ine the layer v_portion, to replace the name of the function in the python action remove_selected_segment --> update_selected_feature and add_selected_segment --> update_selected_feature

Gustry commented 3 years ago

Thanks @rldhont. Hopefully we got test to notice this ;-)

3liz / qgis-veloroutes_voies_vertes-plugin

Améliorations pour une version 0.9.0 #100