Projekt

Allgemein

Aktionen

GX-Bug #52660

geschlossen

Temporäre Dateien von CSV Artikelexport vor neuem Export löschen

Von Wilken Haase vor fast 8 Jahren hinzugefügt. Vor mehr als 7 Jahren aktualisiert.

Status:
Erledigt
Priorität:
Normal
Zugewiesen an:
Moritz Bunjes
Kategorie:
CSV
Zielversion:
% erledigt:

100%

Geschätzter Aufwand:
Steps to reproduce:
Betroffene Versionen:
3.5.3.1
Release Notes Langtext:

CSV-Export: Temporäre Dateien werden vor einem neuem Export gelöscht, wenn diese zu alt sind


Beschreibung

Wenn ein Export abbricht bleibt manchmal eine temporäre Exportdatei liegen. Beginnt man nun einen neuen Export wird eine fehlerhafte Datei erzeugt, weil an die alte Datei von vorne angehangen wird.


Checkliste 0/0

Aktionen #1

Von Mirko Janssen vor mehr als 7 Jahren aktualisiert

  • Zugewiesen an wurde auf Mirko Janssen gesetzt

Das könnte Probleme bereiten... Der CSV Export merkt sich anhand der Datei, ob er gerade am exportieren ist. Wenn man also gerade einen aktiven Export laufen lässt (Cronjob) und dann mittendrin die temporäre Datei löscht weil man einen neuen Export anstößt, dann kann das seltsame und sehr unschöne Folgen haben.

Till und ich haben aber die Parameter des alten CSV Exports angepasst, so dass ein Klicken auf den Abbrechen Button nun schneller erkannt und verarbeitet wird. Meiner Auffassung nach, ist dass das Beste, was man mit dem momentanen System hier erreichen kann.

Aktionen #3

Von Wilken Haase vor mehr als 7 Jahren aktualisiert

  • Zugewiesen an wurde von Wilken Haase zu Mirko Janssen geändert

Lösung ist nicht ausreichend. Wenn eine temporäre datei liegenbleibtm wird ein neuer Export darin Daten anhängen. Wird ein Export durch Doppelklick oder Interferenz mit einem Cronjob 2x gestartet, ist das Ergebnis Müll.

Wir brauchen dann einen Lock, um einen doppelten Start eines Exports nicht zuzulassen und/oder ein Randomtoken im Namen während der Temp Export Erzeugung, um sicher valide Zustände des finalen Exports zu haben.

Aktionen