GX-Bug #52660
geschlossen
Temporäre Dateien von CSV Artikelexport vor neuem Export löschen
100%
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.
Von Mirko Janssen vor etwa 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.
Von Mirko Janssen vor etwa 7 Jahren aktualisiert
- Status wurde von Gemeldet zu Feedback geändert
- Zugewiesen an wurde von Mirko Janssen zu Wilken Haase geändert
Von Wilken Haase vor etwa 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.
Von Mirko Janssen vor fast 7 Jahren aktualisiert
- Status wurde von Feedback zu In Bearbeitung geändert
- Zielversion wurde von 133 zu 3.9.1.0 beta3 geändert
Von Mirko Janssen vor fast 7 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)
Von Moritz Bunjes vor fast 7 Jahren aktualisiert
- Zugewiesen an wurde von Core-Entwickler zu Moritz Bunjes geändert
Von Moritz Bunjes vor fast 7 Jahren aktualisiert
- Status wurde von Testing zu Erledigt geändert
- % erledigt wurde von 0 zu 100 geändert