Projekt

Allgemein

Profil

Aktionen

GX-Bug #52660

geschlossen

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

Von Wilken Haase vor mehr als 6 Jahren hinzugefügt. Vor etwa 6 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.


Aktionen #1

Von Mirko Janssen vor mehr als 6 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 #2

Von Mirko Janssen vor mehr als 6 Jahren aktualisiert

  • Status wurde von Gemeldet zu Feedback geändert
  • Zugewiesen an wurde von Mirko Janssen zu Wilken Haase geändert
Aktionen #3

Von Wilken Haase vor mehr als 6 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 #4

Von Mirko Janssen vor etwa 6 Jahren aktualisiert

  • Status wurde von Feedback zu In Bearbeitung geändert
  • Zielversion wurde von 133 zu 3.9.1.0 beta3 geändert
Aktionen #5

Von Mirko Janssen vor etwa 6 Jahren aktualisiert

  • Status wurde von In Bearbeitung zu Testing geändert
  • Zugewiesen an wurde von Mirko Janssen zu Core-Entwickler geändert
  • Release Notes Langtext aktualisiert (Vergleich)
Aktionen #6

Von Moritz Bunjes vor etwa 6 Jahren aktualisiert

  • Zugewiesen an wurde von Core-Entwickler zu Moritz Bunjes geändert
Aktionen #7

Von Moritz Bunjes vor etwa 6 Jahren aktualisiert

  • Status wurde von Testing zu Erledigt geändert
  • % erledigt wurde von 0 zu 100 geändert
Aktionen

Auch abrufbar als: Atom PDF