From 742818c2caa24e25c74308f5ef1be0e8b7712e5a Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Gina=20H=C3=A4u=C3=9Fge?= <osd@foosel.net>
Date: Wed, 10 Feb 2016 14:34:33 +0100
Subject: [PATCH] Delete unrendered legacy timelapses when encountering them

---
 src/octoprint/timelapse.py | 24 +++++++++++++++++++++++-
 1 file changed, 23 insertions(+), 1 deletion(-)

diff --git a/src/octoprint/timelapse.py b/src/octoprint/timelapse.py
index 4be6025d..1887c96e 100644
--- a/src/octoprint/timelapse.py
+++ b/src/octoprint/timelapse.py
@@ -22,6 +22,8 @@ from octoprint.events import eventManager, Events
 import sarge
 import collections
 
+import re
+
 # currently configured timelapse
 current = None
 
@@ -32,6 +34,10 @@ current_render_job = None
 _capture_format = "{prefix}-%d.jpg"
 _output_format = "{prefix}.mpg"
 
+# old capture format, needed to delete old left-overs from
+# versions <1.2.9
+_old_capture_format_re = re.compile("^tmp_\d{5}.jpg$")
+
 # valid timelapses
 _valid_timelapse_types = ["off", "timed", "zchange"]
 
@@ -129,14 +135,30 @@ def delete_old_unrendered_timelapses():
 	clean_after_days = settings().getInt(["webcam", "cleanTmpAfterDays"])
 	cutoff = time.time() - clean_after_days * 24 * 60 * 60
 
+	prefixes_to_clean = []
 	for filename in os.listdir(basedir):
 		try:
 			path = os.path.join(basedir, filename)
+
+			prefix = _extract_prefix(filename)
+			if prefix is None:
+				# might be an old tmp_00000.jpg kinda frame. we can't
+				# render those easily anymore, so delete that stuff
+				if _old_capture_format_re.match(filename):
+					os.remove(path)
+				continue
+
+			if prefix in prefixes_to_clean:
+				continue
+
 			if os.path.getmtime(path) < cutoff:
-				os.remove(path)
+				prefixes_to_clean.append(prefix)
 		except:
 			logging.getLogger(__name__).exception("Error while processing file {} during cleanup".format(filename))
 
+	for prefix in prefixes_to_clean:
+		delete_unrendered_timelapse(prefix)
+
 
 def _create_render_start_handler(name, gcode=None):
 	def f(movie):