Support #26582
commande pour changer le fichier tableau d'une CSVDataSource
100%
Description
Pour répondre à la demande du ticket #26299, proposition:
passerelle-manage csv-datasource change-csv <slug> blabla.csv
Fichiers
Révisions associées
Historique
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
- Fichier 0001-manage-resource-from-command-line-fixes-26582.patch 0001-manage-resource-from-command-line-fixes-26582.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Je n'ai pas encore ajouté les tests mais je veux bien des retours sur la forme générale du truc. Je suis parti sur l'idée d'avoir une commande à tout faire
sur les resources.
$ passerelle-manage resource list-type --resource-type csvdatasource.csvdatasource
- title
type: string
- description
type: string
- log_level
type: string
choices:
NOTSET: NOTSET
DEBUG: DEBUG
INFO: INFO
WARNING: WARNING
ERROR: ERROR
CRITICAL: CRITICAL
- csv_file
type: file
help text: Supported file formats: csv, ods, xls, xlsx
- columns_keynames
type: string
help text: ex: id,text,data1,data2
- skip_header
type: string
- sheet_name
type: string
$ resource new csvdatasource.csvdatasource --string-field title coucou --string-field log_level WARNING --file-field csv_file ~/tableau.csv --string-field description "blablabla"
$ ./manage.py resource list csvdatasource.csvdatasource
id : 11
title : coucou2
description : blablabla
slug : coucou2
log_level : WARNING
csv_file : csv/.bashrc_Nc1M7EC
columns_keynames : id, text
skip_header : False
_dialect_options : {u'lineterminator': u'\r\n', u'skipinitialspace': False, u'quoting': 0, u'delimiter': u' ', u'quotechar': u'"', u'doublequote': False}
sheet_name :
users : []
$ resource update csvdatasource.csvdatasource coucou --file-field csv_file ~/tableau.csv
$ resource delete csvdatasource.csvdatasource coucou --file-field csv_file ~/tableau.csv
Mis à jour par Frédéric Péters il y a plus de 5 ans
J'en serais resté à la demande, j'ai peur qu'on se galère sur des changements futurs à devoir garder en marche une commande à tout faire.
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
- Fichier 0001-new-command-to-change-csv_file-fixes-26582.patch 0001-new-command-to-change-csv_file-fixes-26582.patch ajouté
Voilà, beaucoup plus simple, aucun contrôle.
Mis à jour par Emmanuel Cazenave il y a plus de 5 ans
Niveau système de fichier, à la lecture du code je ne comprends pas si les csv se retrouveront au même endroit que si ils étaient uploadés dans l'UI.
Peut-être si tu pouvais mettre cela en exergue dans un test ?
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
Emmanuel Cazenave a écrit :
Niveau système de fichier, à la lecture du code je ne comprends pas si les csv se retrouveront au même endroit que si ils étaient uploadés dans l'UI.
C'est à dire au même endroit ? Le répertoire dépend du champ du modèle, chemin différera par contre, il va vouloir sauver le fichier avec le même nom, voir qu'un existe déjà, ajouter _1/2/3.etc.. et écrire.
Peut-être si tu pouvais mettre cela en exergue dans un test ?
Hmm.. je ne comprends pas.
Mis à jour par Emmanuel Cazenave il y a plus de 5 ans
Quand tu upload un fichier csv dans l'UI à la création du connecteur, il finit dans /var/lib/passerelle/tenants/nom-du-tenant/media/csv/.
Ma question est : "est ce que ta commande assure bien ça ?"
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
Emmanuel Cazenave a écrit :
Quand tu upload un fichier csv dans l'UI à la création du connecteur, il finit dans
/var/lib/passerelle/tenants/nom-du-tenant/media/csv/.
À partir du moment où tu lances bien la commande via "tenant_command" oui, c'est le backend de storage issu de hobo.multitenant qui gère cela.
Ma question est : "est ce que ta commande assure bien ça ?"
Non elle ne s'en occupe pas, c'est totomatique (je vois pas plus comment le code dans passerelle assure bien cela).
Mis à jour par Emmanuel Cazenave il y a plus de 5 ans
- Statut changé de Solution proposée à Solution validée
Bon j'ai fait le test à la main et effectivement la totomagie marche.
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
- Statut changé de Solution validée à Résolu (à déployer)
- % réalisé changé de 0 à 100
Appliqué par commit f219ffceb9e081cc91965078d171d82decb9757d.
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Fermé
new command to change csv_file (fixes #26582)