Projet

Général

Profil

Anomalie #628

Galette v0.7.4.3. Export de tables vides..

Ajouté par Roland Telle il y a environ 4 ans. Mis à jour il y a environ 4 ans.

Statut:
Fermé
Priorité:
Bas
Assigné à:
Catégorie:
Génération de fichiers
Version cible:
Début:
27/06/2013
Echéance:
% réalisé:

100%

Version utilisée:

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)

Révisions associées

Révision afee2bf6
Ajouté par Johan Cwiklinski il y a environ 4 ans

Prevent empty tables export; fixes #628

Historique

#1 Mis à jour par Johan Cwiklinski il y a environ 4 ans

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 Mis à jour par Roland Telle il y a environ 4 ans

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 Mis à jour par Roland Telle il y a environ 4 ans

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 Mis à jour par Johan Cwiklinski il y a environ 4 ans

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 Mis à jour par Johan Cwiklinski il y a environ 4 ans

  • Version cible mis à 0.7.5
  • Catégorie mis à Génération de fichiers
  • Assigné à mis à 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 Mis à jour par Johan Cwiklinski il y a environ 4 ans

  • Version cible 0.7.5 supprimé

#7 Mis à jour par Johan Cwiklinski il y a environ 4 ans

  • Version cible mis à 0.7.5

#8 Mis à jour par Johan Cwiklinski il y a environ 4 ans

  • Statut changé de Nouveau à Résolu
  • % réalisé changé de 0 à 100

#9 Mis à jour par Johan Cwiklinski il y a environ 4 ans

  • Statut changé de Résolu à Fermé

Formats disponibles : Atom PDF