Project

General

Profile

Anomalie #628

Galette v0.7.4.3. Export de tables vides..

Added by Roland Telle about 6 years ago. Updated about 6 years ago.

Status:
Fermé
Priority:
Bas
Category:
Génération de fichiers
Target version:
Start date:
06/27/2013
Due date:
% Done:

100%

Estimated time:
Version utilisée:
Vote:

Description

Galette accepte de procéder à l'export des tables vides ; elles existent bien dans le répertoire /exports/ mais il est impossible de les télécharger et surtout de les détruire à partir de la page Export CSV de la base de données (MySql)

Associated revisions

Revision afee2bf6 (diff)
Added by Johan Cwiklinski about 6 years ago

Prevent empty tables export; fixes #628

History

#1

Updated by Johan Cwiklinski about 6 years ago

En effet, l'export de tables vides est possible (ce serait un peu lourd de vérifier celles qui ne le sont pas).

Lors de la récupération, Galette indique dans ses logs que le fichier n'existe pas, ce qui est effectivement incorrect (il faudrait un message explicite plutôt qu'une page blanche). En revanche, j'ai été en mesure de supprimer le fichier sans problèmes :/

Aurais-tu d'avantage d'informations (os, etc) que je puisse reproduire le pépin ?

#2

Updated by Roland Telle about 6 years ago

Mal dit ...

1. Galette accepte de procéder à l'export de tables vides,
2. Les tables figurent bien dans le répertoire /exports/,
3. Si on veut télécharger l'export csv, le navigateur annonce : "Firefox ne peut trouver le fichier à l'adresse

http://xyz.abc.org/get_export.php?file=galette_maps_towns_full.csv
.
4. Le "flèche retour" du navigateur renvoie la page Export CSV de la base de données avec la mention Le fichier « galette_maps_towns_full.csv » a été supprimé du disque. (ce qui est bien le cas).

#3

Updated by Roland Telle about 6 years ago

Et une requête en boucle pour tester et exclure celles qui sont vides ('IS NULL'), ca serait lourd ?
Cela dit, faut être un peu bête de chercher à exporter une table vide qui apparemment s'autodétruit ;)

#4

Updated by Johan Cwiklinski about 6 years ago

Roland Telle a écrit :

Et une requête en boucle pour tester et exclure celles qui sont vides ('IS NULL'), ca serait lourd ?

Actuellement, Galette ne fait que lister les tables ; donc aller chercher le nombre d'enregistrements contenu serait forcément plus lourd, mais pas forcément si lourd que ça.

Cela dit, faut être un peu bête de chercher à exporter une table vide qui apparemment s'autodétruit ;)

Ouais, voilà :) J'ai proposé l'ensemble des tables présentes pour être sûr de ne pas en oublier, et pour prendre en compte les plugins au besoin...

Lors de l'export, on peut parfaitement refuser de créer le fichier s'il ne contiendra aucun enregistrement (je pense que c'est vers cette solution que je vais me tourner d'ailleurs) :)

Merci pour les précisions, je vais investiguer.

#5

Updated by Johan Cwiklinski about 6 years ago

  • Target version set to 0.7.5
  • Category set to Génération de fichiers
  • Assignee set to Johan Cwiklinski

Roland Telle a écrit :

3. Si on veut télécharger l'export csv, le navigateur annonce : "Firefox ne peut trouver le fichier à l'adresse [...].
4. Le "flèche retour" du navigateur renvoie la page Export CSV de la base de données avec la mention Le fichier « galette_maps_towns_full.csv » a été supprimé du disque. (ce qui est bien le cas).

Je ne parviens pas à reproduire ces deux derniers points... Mias puisque le mieux serait que Galette refuse lâchement d'effectuer un export vide, ils ne se poseront pas à priori :)

#6

Updated by Johan Cwiklinski about 6 years ago

  • Target version deleted (0.7.5)
#7

Updated by Johan Cwiklinski about 6 years ago

  • Target version set to 0.7.5
#8

Updated by Johan Cwiklinski about 6 years ago

  • Status changed from Nouveau to Résolu
  • % Done changed from 0 to 100
#9

Updated by Johan Cwiklinski about 6 years ago

  • Status changed from Résolu to Fermé

Also available in: Atom PDF