GX-Bug #52660

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

Added by Wilken Haase almost 2 years ago. Updated over 1 year ago.

Status:
Erledigt
Priority name:
Normal
Assigned To:
Moritz Bunjes
Category name:
CSV
Target version:
% Done:

100%

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

Story points-Velocity based estimate-

Description

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.

History

#1 Updated by Mirko Janssen over 1 year ago

  • Assigned To set to Mirko Janssen

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.

#2 Updated by Mirko Janssen over 1 year ago

  • Status changed from Gemeldet to Feedback
  • Assigned To changed from Mirko Janssen to Wilken Haase

#3 Updated by Wilken Haase over 1 year ago

  • Assigned To changed from Wilken Haase to Mirko Janssen

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.

#4 Updated by Mirko Janssen over 1 year ago

  • Status changed from Feedback to In Bearbeitung
  • Target version changed from SP-Candidates to 3.9.1.0 beta3

#5 Updated by Mirko Janssen over 1 year ago

  • Status changed from In Bearbeitung to Testing
  • Assigned To changed from Mirko Janssen to Core-Entwickler
  • Release Notes Langtext updated (diff)

#6 Updated by Moritz Bunjes over 1 year ago

  • Assigned To changed from Core-Entwickler to Moritz Bunjes

#7 Updated by Moritz Bunjes over 1 year ago

  • Status changed from Testing to Erledigt
  • % Done changed from 0 to 100

Also available in: Atom PDF

Add picture from clipboard (Maximum size: 5 MB)