From 4594743baf6e6fd4b50f68665d009084f605bc78 Mon Sep 17 00:00:00 2001
From: BonfaceKilz
Date: Mon, 2 Aug 2021 21:45:17 +0300
Subject: wqflask: views: After approval, rename the file at the right point

---
 wqflask/wqflask/views.py | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

(limited to 'wqflask')

diff --git a/wqflask/wqflask/views.py b/wqflask/wqflask/views.py
index 1a8536ac..c7582782 100644
--- a/wqflask/wqflask/views.py
+++ b/wqflask/wqflask/views.py
@@ -1434,10 +1434,11 @@ def approve_data(name):
                        dataset_id=name.split(".")[0],  # use the dataset name
                        editor=sample_data.get("author"),
                        json_data=json.dumps(sample_data)))
-            # Once data is approved, rename it!
-            os.rename(os.path.join("/tmp/sample-data/diffs", name),
-                      os.path.join("/tmp/sample-data/diffs",
-                                   f"{name}.approved"))
+    if modifications:
+        # Once data is approved, rename it!
+        os.rename(os.path.join("/tmp/sample-data/diffs", name),
+                  os.path.join("/tmp/sample-data/diffs",
+                               f"{name}.approved"))
     return redirect("/admin/data-sample/diffs/")
 
 
-- 
cgit v1.2.3