alhyss
commented
2 years ago @WREATCHED C'est fait, tu peux y aller. Désolée d'avoir tant traîné !
En y repensant, je t'avais mis ci-dessus la liste des fonctions concernées, mais le plus simple est encore de faire une recherche sur query_ dans tous tes fichiers. À l'exception des commandes d'import, ça te donnera exactement tous les endroits où tu auras des modifications à faire.
WREATCHED
L'idée générale est de faire en sorte que toutes les fonctions du module
plume.pg.queriespuissent être appelées exactement de la même façon.Tous les paramètres sont fournis à la fonction qui crée la requête. Par exemple, on aura désormais :
La fonction renvoie un tuple, qu'on éclate pour le passer en argument de
psycopg2.cursor.execute:Antérieurement, certains paramètres étaient passés directement à
psycopg2.cursor.executeplutôt qu'à la fonction elle-même, avec des comportements variables d'une fonction à l'autre. Les fonctions écrites récemment avaient déjà le fonctionnement décrit ci-avant, ce qui ajoutait encore un peu d'hétérogénéité à l'affaire.Cette évolution va nécessiter de revoir les appels aux fonctions suivantes :
query_is_relation_ownerquery_exists_extensionquery_get_relation_kindquery_update_table_commentquery_get_table_commentquery_list_templatesquery_get_categoriesquery_template_tabsquery_get_columnsquery_update_column_comment(je crois que celle-là n'est pas utilisée)query_update_columns_commentsquery_get_geom_sridquery_get_geom_extentquery_get_geom_centroidÀ noter que pour certaines d'entre elles tous les paramètres sont déjà au bon endroit, il y aura juste l'astérisque à ajouter.
Le message d'erreur à afficher lorsque la requête était supposée renvoyer un résultat et ne le fait pas est fourni par l'attribut
plume.pg.queries.PgQueryWithArgs.missing_mssgde la requête.@WREATCHED Attends mon feu vert pour appliquer les changements de ton côté, je suis en train d'écrire cette issue alors que je n'ai pas fini de modifier les fonctions.